[dpdk-test-report] |WARNING| pw48483 [v2, 14/20] net/ice: support statistics

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Dec 6 18:41:08 CET 2018


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

_coding style issues_


CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#255: FILE: drivers/net/ice/ice_ethdev.c:2924:
+	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:2924:
+	PMD_DRV_LOG(DEBUG, "rx_bytes:            %"PRIu64"", nes->rx_bytes);

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#526: FILE: drivers/net/ice/ice_ethdev.c:3195:
+	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