* [dpdk-test-report] |WARNING| pw46720 net/softnic: add support for vxlan encap
[not found] <1539345127-179889-1-git-send-email-cristian.dumitrescu@intel.com>
@ 2018-10-19 9:25 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2018-10-19 9:25 UTC (permalink / raw)
To: test-report; +Cc: Cristian Dumitrescu
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/46720
_coding style issues_
CHECK:BRACES: braces {} should be used on all arms of this statement
#70: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1522:
+ if (strcmp(tokens[t0 + 2 + 2], "ipv4") == 0)
[...]
+ else if (strcmp(tokens[t0 + 2 + 2], "ipv6") == 0)
[...]
+ else {
[...]
CHECK:BRACES: Unbalanced braces around else statement
#74: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1526:
+ else {
CHECK:BRACES: braces {} should be used on all arms of this statement
#86: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1538:
+ if (strcmp(tokens[t0 + 2 + 4], "on") == 0)
[...]
+ else if (strcmp(tokens[t0 + 2 + 4], "off") == 0)
[...]
+ else {
[...]
CHECK:BRACES: Unbalanced braces around else statement
#90: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1542:
+ else {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 3'
#136: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3656:
+ if ((n_tokens < 3) ||
+ strcmp(tokens[0], "ether") ||
+ softnic_parse_mac_addr(tokens[1], &a->encap.vxlan.ether.da) ||
+ softnic_parse_mac_addr(tokens[2], &a->encap.vxlan.ether.sa))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 4'
#150: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3670:
+ if ((n_tokens < 4) ||
+ softnic_parser_read_uint32(&pcp, tokens[1]) ||
+ (pcp > 7) ||
+ softnic_parser_read_uint32(&dei, tokens[2]) ||
+ (dei > 1) ||
+ softnic_parser_read_uint32(&vid, tokens[3]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pcp > 7'
#150: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3670:
+ if ((n_tokens < 4) ||
+ softnic_parser_read_uint32(&pcp, tokens[1]) ||
+ (pcp > 7) ||
+ softnic_parser_read_uint32(&dei, tokens[2]) ||
+ (dei > 1) ||
+ softnic_parser_read_uint32(&vid, tokens[3]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'dei > 1'
#150: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3670:
+ if ((n_tokens < 4) ||
+ softnic_parser_read_uint32(&pcp, tokens[1]) ||
+ (pcp > 7) ||
+ softnic_parser_read_uint32(&dei, tokens[2]) ||
+ (dei > 1) ||
+ softnic_parser_read_uint32(&vid, tokens[3]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'vid > 0xFFF'
#150: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3670:
+ if ((n_tokens < 4) ||
+ softnic_parser_read_uint32(&pcp, tokens[1]) ||
+ (pcp > 7) ||
+ softnic_parser_read_uint32(&dei, tokens[2]) ||
+ (dei > 1) ||
+ softnic_parser_read_uint32(&vid, tokens[3]) ||
+ (vid > 0xFFF))
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#169: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3689:
+ /* ipv4 <sa> <da> <dscp> <ttl>
+ | ipv6 <sa> <da> <flow_label> <dscp> <hop_limit> */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#169: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3689:
+ | ipv6 <sa> <da> <flow_label> <dscp> <hop_limit> */
CHECK:BRACES: braces {} should be used on all arms of this statement
#170: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3690:
+ if (strcmp(tokens[0], "ipv4") == 0) {
[...]
+ } else if (strcmp(tokens[0], "ipv6") == 0) {
[...]
+ } else
[...]
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 5'
#174: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3694:
+ if ((n_tokens < 5) ||
+ softnic_parse_ipv4_addr(tokens[1], &sa) ||
+ softnic_parse_ipv4_addr(tokens[2], &da) ||
+ softnic_parser_read_uint8(&dscp, tokens[3]) ||
+ (dscp > 64) ||
+ softnic_parser_read_uint8(&ttl, tokens[4]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'dscp > 64'
#174: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3694:
+ if ((n_tokens < 5) ||
+ softnic_parse_ipv4_addr(tokens[1], &sa) ||
+ softnic_parse_ipv4_addr(tokens[2], &da) ||
+ softnic_parser_read_uint8(&dscp, tokens[3]) ||
+ (dscp > 64) ||
+ softnic_parser_read_uint8(&ttl, tokens[4]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 6'
#195: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3715:
+ if ((n_tokens < 6) ||
+ softnic_parse_ipv6_addr(tokens[1], &sa) ||
+ softnic_parse_ipv6_addr(tokens[2], &da) ||
+ softnic_parser_read_uint32(&flow_label, tokens[3]) ||
+ softnic_parser_read_uint8(&dscp, tokens[4]) ||
+ (dscp > 64) ||
+ softnic_parser_read_uint8(&hop_limit, tokens[5]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'dscp > 64'
#195: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3715:
+ if ((n_tokens < 6) ||
+ softnic_parse_ipv6_addr(tokens[1], &sa) ||
+ softnic_parse_ipv6_addr(tokens[2], &da) ||
+ softnic_parser_read_uint32(&flow_label, tokens[3]) ||
+ softnic_parser_read_uint8(&dscp, tokens[4]) ||
+ (dscp > 64) ||
+ softnic_parser_read_uint8(&hop_limit, tokens[5]))
CHECK:BRACES: Unbalanced braces around else statement
#213: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3733:
+ } else
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 3'
#217: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3737:
+ if ((n_tokens < 3) ||
+ strcmp(tokens[0], "udp") ||
+ softnic_parser_read_uint16(&a->encap.vxlan.udp.sp, tokens[1]) ||
+ softnic_parser_read_uint16(&a->encap.vxlan.udp.dp, tokens[2]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 2'
#228: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3748:
+ if ((n_tokens < 2) ||
+ strcmp(tokens[0], "vxlan") ||
+ softnic_parser_read_uint32(&a->encap.vxlan.vxlan.vni, tokens[1]) ||
+ (a->encap.vxlan.vxlan.vni > 0xFFFFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'a->encap.vxlan.vxlan.vni > 0xFFFFFF'
#228: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3748:
+ if ((n_tokens < 2) ||
+ strcmp(tokens[0], "vxlan") ||
+ softnic_parser_read_uint32(&a->encap.vxlan.vxlan.vni, tokens[1]) ||
+ (a->encap.vxlan.vxlan.vni > 0xFFFFFF))
WARNING:LONG_LINE: line over 90 characters
#230: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3750:
+ softnic_parser_read_uint32(&a->encap.vxlan.vxlan.vni, tokens[1]) ||
total: 0 errors, 3 warnings, 18 checks, 214 lines checked
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2018-10-19 9:25 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1539345127-179889-1-git-send-email-cristian.dumitrescu@intel.com>
2018-10-19 9:25 ` [dpdk-test-report] |WARNING| pw46720 net/softnic: add support for vxlan encap checkpatch
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).