From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id 64F3C1B4B0; Wed, 12 Dec 2018 08:02:05 +0100 (CET) In-Reply-To: <1544598004-27099-29-git-send-email-wenzhuo.lu@intel.com> References: <1544598004-27099-29-git-send-email-wenzhuo.lu@intel.com> To: test-report@dpdk.org Cc: Wenzhuo Lu Message-Id: <20181212070205.64F3C1B4B0@dpdk.org> Date: Wed, 12 Dec 2018 08:02:05 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw48690 [v3, 28/34] net/ice: support statistics X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Dec 2018 07:02:05 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/48690 _coding style issues_ CHECK:CAMELCASE: Avoid CamelCase: #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