[dpdk-test-report] |WARNING| pw49072 [v6, 30/31] net/ice: support statistics

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Dec 18 09:50:12 CET 2018


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

_coding style issues_


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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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


More information about the test-report mailing list