[dpdk-test-report] |WARNING| pw48690 [v3, 28/34] net/ice: support statistics

automatic DPDK test reports
 help / color / mirror / Atom feed

From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Wenzhuo Lu <wenzhuo.lu@intel.com>
Subject: [dpdk-test-report] |WARNING| pw48690 [v3, 28/34] net/ice: support statistics
Date: Wed, 12 Dec 2018 08:02:05 +0100 (CET)	[thread overview]
Message-ID: <20181212070205.64F3C1B4B0@dpdk.org> (raw)
In-Reply-To: <1544598004-27099-29-git-send-email-wenzhuo.lu@intel.com>

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

_coding style issues_


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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

          parent reply	other threads:[~2018-12-12  7:02 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <1544598004-27099-29-git-send-email-wenzhuo.lu@intel.com>]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181212070205.64F3C1B4B0@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=test-report@dpdk.org \
    --cc=wenzhuo.lu@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Be sure your reply has a Subject: header at the top and a blank line before the message body.

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).