[dpdk-test-report] |WARNING| pw40074 [PATCH] dev Digest Vol 195 Issue 45

checkpatch at dpdk.org checkpatch at dpdk.org
Wed May 16 04:09:57 CEST 2018


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#31: 
When replying, please edit your Subject line so it is more specific than "Re: Contents of dev digest..."

ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 0123456789ab ("commit description")'
#95: 
commit 55fbc92d7800100628579643c9ee2770614fef10

WARNING:BAD_SIGN_OFF: Do not use whitespace before Reported-by:
#114: 
    Reported-by: Andrew Rybchenko <arybchenko at solarflare.com>

WARNING:BAD_SIGN_OFF: Do not use whitespace before Signed-off-by:
#115: 
    Signed-off-by: Pavan Nikhilesh <pbhagavatula at caviumnetworks.com>

WARNING:BAD_SIGN_OFF: Do not use whitespace before Acked-by:
#116: 
    Acked-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>

WARNING:BAD_SIGN_OFF: Duplicate signature
#234: 
To: Gavin Hu <gavin.hu at arm.com>

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#334: FILE: drivers/net/tap/rte_eth_tap.c:506:
+if (txq->type == ETH_TUNTAP_TYPE_TUN) {
[...]
+char *buff_data = rte_pktmbuf_mtod(seg, void *);

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#378: FILE: drivers/net/tap/rte_eth_tap.c:1433:
+if (pmd->type == ETH_TUNTAP_TYPE_TAP) {
 if (is_zero_ether_addr(mac_addr))

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#387: FILE: drivers/net/tap/rte_eth_tap.c:1452:
+if (pmd->type == ETH_TUNTAP_TYPE_TAP) {
 memset(&ifr, 0, sizeof(struct ifreq));

total: 1 errors, 8 warnings, 0 checks, 130 lines checked


More information about the test-report mailing list