[dpdk-test-report] |WARNING| pw48298 [14/19] net/ice: support statistics

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Nov 23 07:53:00 CET 2018


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/48298

_coding style issues_


CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#254: FILE: drivers/net/ice/ice_ethdev.c:2909:
+	PMD_DRV_LOG(DEBUG, "rx_bytes:            %"PRIu64"", nes->rx_bytes);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#254: FILE: drivers/net/ice/ice_ethdev.c:2909:
+	PMD_DRV_LOG(DEBUG, "rx_bytes:            %"PRIu64"", nes->rx_bytes);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#255: FILE: drivers/net/ice/ice_ethdev.c:2910:
+	PMD_DRV_LOG(DEBUG, "rx_unicast:          %"PRIu64"", nes->rx_unicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#256: FILE: drivers/net/ice/ice_ethdev.c:2911:
+	PMD_DRV_LOG(DEBUG, "rx_multicast:        %"PRIu64"", nes->rx_multicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#257: FILE: drivers/net/ice/ice_ethdev.c:2912:
+	PMD_DRV_LOG(DEBUG, "rx_broadcast:        %"PRIu64"", nes->rx_broadcast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#258: FILE: drivers/net/ice/ice_ethdev.c:2913:
+	PMD_DRV_LOG(DEBUG, "rx_discards:         %"PRIu64"", nes->rx_discards);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#259: FILE: drivers/net/ice/ice_ethdev.c:2914:
+	PMD_DRV_LOG(DEBUG, "rx_unknown_protocol: %"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#261: FILE: drivers/net/ice/ice_ethdev.c:2916:
+	PMD_DRV_LOG(DEBUG, "tx_bytes:            %"PRIu64"", nes->tx_bytes);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#262: FILE: drivers/net/ice/ice_ethdev.c:2917:
+	PMD_DRV_LOG(DEBUG, "tx_unicast:          %"PRIu64"", nes->tx_unicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#263: FILE: drivers/net/ice/ice_ethdev.c:2918:
+	PMD_DRV_LOG(DEBUG, "tx_multicast:        %"PRIu64"", nes->tx_multicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#264: FILE: drivers/net/ice/ice_ethdev.c:2919:
+	PMD_DRV_LOG(DEBUG, "tx_broadcast:        %"PRIu64"", nes->tx_broadcast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#265: FILE: drivers/net/ice/ice_ethdev.c:2920:
+	PMD_DRV_LOG(DEBUG, "tx_discards:         %"PRIu64"", nes->tx_discards);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#266: FILE: drivers/net/ice/ice_ethdev.c:2921:
+	PMD_DRV_LOG(DEBUG, "tx_errors:           %"PRIu64"", nes->tx_errors);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#475: FILE: drivers/net/ice/ice_ethdev.c:3130:
+	PMD_DRV_LOG(DEBUG, "rx_bytes:	%"PRIu64"", ns->eth.rx_bytes);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#476: FILE: drivers/net/ice/ice_ethdev.c:3131:
+	PMD_DRV_LOG(DEBUG, "rx_unicast:	%"PRIu64"", ns->eth.rx_unicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#477: FILE: drivers/net/ice/ice_ethdev.c:3132:
+	PMD_DRV_LOG(DEBUG, "rx_multicast:%"PRIu64"", ns->eth.rx_multicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#478: FILE: drivers/net/ice/ice_ethdev.c:3133:
+	PMD_DRV_LOG(DEBUG, "rx_broadcast:%"PRIu64"", ns->eth.rx_broadcast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#479: FILE: drivers/net/ice/ice_ethdev.c:3134:
+	PMD_DRV_LOG(DEBUG, "rx_discards:%"PRIu64"", ns->eth.rx_discards);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#480: FILE: drivers/net/ice/ice_ethdev.c:3135:
+	PMD_DRV_LOG(DEBUG, "vsi rx_discards:%"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#482: FILE: drivers/net/ice/ice_ethdev.c:3137:
+	PMD_DRV_LOG(DEBUG, "rx_unknown_protocol:  %"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#484: FILE: drivers/net/ice/ice_ethdev.c:3139:
+	PMD_DRV_LOG(DEBUG, "tx_bytes:	%"PRIu64"", ns->eth.tx_bytes);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#485: FILE: drivers/net/ice/ice_ethdev.c:3140:
+	PMD_DRV_LOG(DEBUG, "tx_unicast:	%"PRIu64"", ns->eth.tx_unicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#486: FILE: drivers/net/ice/ice_ethdev.c:3141:
+	PMD_DRV_LOG(DEBUG, "tx_multicast:%"PRIu64"", ns->eth.tx_multicast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#487: FILE: drivers/net/ice/ice_ethdev.c:3142:
+	PMD_DRV_LOG(DEBUG, "tx_broadcast:%"PRIu64"", ns->eth.tx_broadcast);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#488: FILE: drivers/net/ice/ice_ethdev.c:3143:
+	PMD_DRV_LOG(DEBUG, "tx_discards:%"PRIu64"", ns->eth.tx_discards);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#489: FILE: drivers/net/ice/ice_ethdev.c:3144:
+	PMD_DRV_LOG(DEBUG, "vsi tx_discards:%"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#491: FILE: drivers/net/ice/ice_ethdev.c:3146:
+	PMD_DRV_LOG(DEBUG, "tx_errors:		%"PRIu64"", ns->eth.tx_errors);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#493: FILE: drivers/net/ice/ice_ethdev.c:3148:
+	PMD_DRV_LOG(DEBUG, "tx_dropped_link_down:	%"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#495: FILE: drivers/net/ice/ice_ethdev.c:3150:
+	PMD_DRV_LOG(DEBUG, "crc_errors:	%"PRIu64"", ns->crc_errors);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#496: FILE: drivers/net/ice/ice_ethdev.c:3151:
+	PMD_DRV_LOG(DEBUG, "illegal_bytes:	%"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#498: FILE: drivers/net/ice/ice_ethdev.c:3153:
+	PMD_DRV_LOG(DEBUG, "error_bytes:	%"PRIu64"", ns->error_bytes);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#499: FILE: drivers/net/ice/ice_ethdev.c:3154:
+	PMD_DRV_LOG(DEBUG, "mac_local_faults:	%"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#501: FILE: drivers/net/ice/ice_ethdev.c:3156:
+	PMD_DRV_LOG(DEBUG, "mac_remote_faults:	%"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#503: FILE: drivers/net/ice/ice_ethdev.c:3158:
+	PMD_DRV_LOG(DEBUG, "link_xon_rx:	%"PRIu64"", ns->link_xon_rx);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#504: FILE: drivers/net/ice/ice_ethdev.c:3159:
+	PMD_DRV_LOG(DEBUG, "link_xoff_rx:	%"PRIu64"", ns->link_xoff_rx);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#505: FILE: drivers/net/ice/ice_ethdev.c:3160:
+	PMD_DRV_LOG(DEBUG, "link_xon_tx:	%"PRIu64"", ns->link_xon_tx);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#506: FILE: drivers/net/ice/ice_ethdev.c:3161:
+	PMD_DRV_LOG(DEBUG, "link_xoff_tx:	%"PRIu64"", ns->link_xoff_tx);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#507: FILE: drivers/net/ice/ice_ethdev.c:3162:
+	PMD_DRV_LOG(DEBUG, "rx_size_64:		%"PRIu64"", ns->rx_size_64);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#508: FILE: drivers/net/ice/ice_ethdev.c:3163:
+	PMD_DRV_LOG(DEBUG, "rx_size_127:	%"PRIu64"", ns->rx_size_127);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#509: FILE: drivers/net/ice/ice_ethdev.c:3164:
+	PMD_DRV_LOG(DEBUG, "rx_size_255:	%"PRIu64"", ns->rx_size_255);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#510: FILE: drivers/net/ice/ice_ethdev.c:3165:
+	PMD_DRV_LOG(DEBUG, "rx_size_511:	%"PRIu64"", ns->rx_size_511);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#511: FILE: drivers/net/ice/ice_ethdev.c:3166:
+	PMD_DRV_LOG(DEBUG, "rx_size_1023:	%"PRIu64"", ns->rx_size_1023);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#512: FILE: drivers/net/ice/ice_ethdev.c:3167:
+	PMD_DRV_LOG(DEBUG, "rx_size_1522:	%"PRIu64"", ns->rx_size_1522);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#513: FILE: drivers/net/ice/ice_ethdev.c:3168:
+	PMD_DRV_LOG(DEBUG, "rx_size_big:	%"PRIu64"", ns->rx_size_big);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#514: FILE: drivers/net/ice/ice_ethdev.c:3169:
+	PMD_DRV_LOG(DEBUG, "rx_undersize:	%"PRIu64"", ns->rx_undersize);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#515: FILE: drivers/net/ice/ice_ethdev.c:3170:
+	PMD_DRV_LOG(DEBUG, "rx_fragments:	%"PRIu64"", ns->rx_fragments);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#516: FILE: drivers/net/ice/ice_ethdev.c:3171:
+	PMD_DRV_LOG(DEBUG, "rx_oversize:	%"PRIu64"", ns->rx_oversize);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#517: FILE: drivers/net/ice/ice_ethdev.c:3172:
+	PMD_DRV_LOG(DEBUG, "rx_jabber:		%"PRIu64"", ns->rx_jabber);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#518: FILE: drivers/net/ice/ice_ethdev.c:3173:
+	PMD_DRV_LOG(DEBUG, "tx_size_64:		%"PRIu64"", ns->tx_size_64);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#519: FILE: drivers/net/ice/ice_ethdev.c:3174:
+	PMD_DRV_LOG(DEBUG, "tx_size_127:	%"PRIu64"", ns->tx_size_127);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#520: FILE: drivers/net/ice/ice_ethdev.c:3175:
+	PMD_DRV_LOG(DEBUG, "tx_size_255:	%"PRIu64"", ns->tx_size_255);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#521: FILE: drivers/net/ice/ice_ethdev.c:3176:
+	PMD_DRV_LOG(DEBUG, "tx_size_511:	%"PRIu64"", ns->tx_size_511);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#522: FILE: drivers/net/ice/ice_ethdev.c:3177:
+	PMD_DRV_LOG(DEBUG, "tx_size_1023:	%"PRIu64"", ns->tx_size_1023);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#523: FILE: drivers/net/ice/ice_ethdev.c:3178:
+	PMD_DRV_LOG(DEBUG, "tx_size_1522:	%"PRIu64"", ns->tx_size_1522);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#524: FILE: drivers/net/ice/ice_ethdev.c:3179:
+	PMD_DRV_LOG(DEBUG, "tx_size_big:	%"PRIu64"", ns->tx_size_big);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#525: FILE: drivers/net/ice/ice_ethdev.c:3180:
+	PMD_DRV_LOG(DEBUG, "rx_len_errors:	%"PRIu64"", ns->rx_len_errors);

total: 0 errors, 0 warnings, 56 checks, 591 lines checked


More information about the test-report mailing list