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| pw48483 [v2, 14/20] net/ice: support statistics
Date: Thu,  6 Dec 2018 18:41:08 +0100 (CET)	[thread overview]
Message-ID: <20181206174108.E4AA01B101@dpdk.org> (raw)
In-Reply-To: <1543820821-108122-15-git-send-email-wenzhuo.lu@intel.com>

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

           reply	other threads:[~2018-12-06 17:41 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <1543820821-108122-15-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=20181206174108.E4AA01B101@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).