From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Jasvinder Singh <jasvinder.singh@intel.com>
Subject: [dpdk-test-report] |WARNING| pw40842 [PATCH 16/21] net/softnic: add cli for pipeline table entries
Date: Fri, 8 Jun 2018 16:19:31 +0200 (CEST) [thread overview]
Message-ID: <20180608141931.200ED1BA96@dpdk.org> (raw)
In-Reply-To: <20180608124155.140663-17-jasvinder.singh@intel.com>
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/40842
_coding style issues_
CHECK:SPACING: No space is necessary after a cast
#437: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1781:
+ (struct pkt_key_ipv4_5tuple *) m->match.hash.key;
CHECK:SPACING: No space is necessary after a cast
#519: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1829:
+ (struct pkt_key_ipv6_5tuple *) m->match.hash.key;
CHECK:SPACING: No space is necessary after a cast
#572: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1877:
+ (struct pkt_key_ipv4_addr *) m->match.hash.key;
CHECK:SPACING: No space is necessary after a cast
#594: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1899:
+ (struct pkt_key_ipv6_addr *) m->match.hash.key;
CHECK:SPACING: No space is necessary after a cast
#616: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1921:
+ (struct pkt_key_qinq *) m->match.hash.key;
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'svlan > 0xFFF'
#625: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1930:
+ if ((parser_read_uint16(&svlan, tokens[3]) != 0) ||
+ (svlan > 0xFFF)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'cvlan > 0xFFF'
#632: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:1937:
+ if ((parser_read_uint16(&cvlan, tokens[4]) != 0) ||
+ (cvlan > 0xFFF)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#740: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2045:
+ if ((n_tokens == 0) || (strcmp(tokens[0], "fwd") != 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 2'
#755: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2060:
+ if ((n_tokens < 2) ||
+ parser_read_uint32(&id, tokens[1]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 2'
#774: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2079:
+ if ((n_tokens < 2) ||
+ parser_read_uint32(&id, tokens[1]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#794: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2099:
+ if ((n_tokens == 0) || (strcmp(tokens[0], "balance") != 0))
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#810: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2115:
+
+}
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 9'
#843: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2148:
+ if ((n_tokens < 9) ||
+ strcmp(tokens[0], "meter") ||
+ parser_read_uint32(&mtr->meter_profile_id, tokens[1]) ||
+ strcmp(tokens[2], "policer") ||
+ strcmp(tokens[3], "g") ||
+ parse_policer_action(tokens[4], &mtr->policer[e_RTE_METER_GREEN]) ||
+ strcmp(tokens[5], "y") ||
+ parse_policer_action(tokens[6], &mtr->policer[e_RTE_METER_YELLOW]) ||
+ strcmp(tokens[7], "r") ||
+ parse_policer_action(tokens[8], &mtr->policer[e_RTE_METER_RED]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#863: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2168:
+ if ((n_tokens == 0) || strcmp(tokens[0], "meter"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 10'
#869: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2174:
+ if ((n_tokens < 10) ||
+ strcmp(tokens[0], "tc0") ||
+ (parse_table_action_meter_tc(tokens + 1,
+ n_tokens - 1,
+ &a->mtr.mtr[0]) == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#879: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2184:
+ if ((n_tokens == 0) || strcmp(tokens[0], "tc1")) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 30'
#885: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2190:
+ if ((n_tokens < 30) ||
+ (parse_table_action_meter_tc(tokens + 1,
+ n_tokens - 1, &a->mtr.mtr[1]) == 0) ||
+ strcmp(tokens[10], "tc2") ||
+ (parse_table_action_meter_tc(tokens + 11,
+ n_tokens - 11, &a->mtr.mtr[2]) == 0) ||
+ strcmp(tokens[20], "tc3") ||
+ (parse_table_action_meter_tc(tokens + 21,
+ n_tokens - 21, &a->mtr.mtr[3]) == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 5'
#908: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2213:
+ if ((n_tokens < 5) ||
+ strcmp(tokens[0], "tm") ||
+ strcmp(tokens[1], "subport") ||
+ parser_read_uint32(&subport_id, tokens[2]) ||
+ strcmp(tokens[3], "pipe") ||
+ parser_read_uint32(&pipe_id, tokens[4]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#927: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2232:
+ if ((n_tokens == 0) || strcmp(tokens[0], "encap"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 3'
#935: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2240:
+ if ((n_tokens < 3) ||
+ parse_mac_addr(tokens[1], &a->encap.ether.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.ether.ether.sa))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 6'
#949: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2254:
+ if ((n_tokens < 6) ||
+ parse_mac_addr(tokens[1], &a->encap.vlan.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.vlan.ether.sa) ||
+ parser_read_uint32(&pcp, tokens[3]) ||
+ (pcp > 0x7) ||
+ parser_read_uint32(&dei, tokens[4]) ||
+ (dei > 0x1) ||
+ parser_read_uint32(&vid, tokens[5]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pcp > 0x7'
#949: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2254:
+ if ((n_tokens < 6) ||
+ parse_mac_addr(tokens[1], &a->encap.vlan.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.vlan.ether.sa) ||
+ parser_read_uint32(&pcp, tokens[3]) ||
+ (pcp > 0x7) ||
+ parser_read_uint32(&dei, tokens[4]) ||
+ (dei > 0x1) ||
+ parser_read_uint32(&vid, tokens[5]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'dei > 0x1'
#949: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2254:
+ if ((n_tokens < 6) ||
+ parse_mac_addr(tokens[1], &a->encap.vlan.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.vlan.ether.sa) ||
+ parser_read_uint32(&pcp, tokens[3]) ||
+ (pcp > 0x7) ||
+ parser_read_uint32(&dei, tokens[4]) ||
+ (dei > 0x1) ||
+ parser_read_uint32(&vid, tokens[5]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'vid > 0xFFF'
#949: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2254:
+ if ((n_tokens < 6) ||
+ parse_mac_addr(tokens[1], &a->encap.vlan.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.vlan.ether.sa) ||
+ parser_read_uint32(&pcp, tokens[3]) ||
+ (pcp > 0x7) ||
+ parser_read_uint32(&dei, tokens[4]) ||
+ (dei > 0x1) ||
+ parser_read_uint32(&vid, tokens[5]) ||
+ (vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 9'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'svlan_pcp > 0x7'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'svlan_dei > 0x1'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'svlan_vid > 0xFFF'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'cvlan_pcp > 0x7'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'cvlan_dei > 0x1'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'cvlan_vid > 0xFFF'
#973: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2278:
+ if ((n_tokens < 9) ||
+ parse_mac_addr(tokens[1], &a->encap.qinq.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.qinq.ether.sa) ||
+ parser_read_uint32(&svlan_pcp, tokens[3]) ||
+ (svlan_pcp > 0x7) ||
+ parser_read_uint32(&svlan_dei, tokens[4]) ||
+ (svlan_dei > 0x1) ||
+ parser_read_uint32(&svlan_vid, tokens[5]) ||
+ (svlan_vid > 0xFFF) ||
+ parser_read_uint32(&cvlan_pcp, tokens[6]) ||
+ (cvlan_pcp > 0x7) ||
+ parser_read_uint32(&cvlan_dei, tokens[7]) ||
+ (cvlan_dei > 0x1) ||
+ parser_read_uint32(&cvlan_vid, tokens[8]) ||
+ (cvlan_vid > 0xFFF))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'label > 0xFFFFF'
#1015: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2320:
+ if (parse_mac_addr(tokens[2], &a->encap.mpls.ether.da) ||
+ parse_mac_addr(tokens[3], &a->encap.mpls.ether.sa) ||
+ strcmp(tokens[4], "label0") ||
+ parser_read_uint32(&label, tokens[5]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[6]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[7]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tc > 0x7'
#1015: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2320:
+ if (parse_mac_addr(tokens[2], &a->encap.mpls.ether.da) ||
+ parse_mac_addr(tokens[3], &a->encap.mpls.ether.sa) ||
+ strcmp(tokens[4], "label0") ||
+ parser_read_uint32(&label, tokens[5]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[6]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[7]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ttl > 0x3F'
#1015: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2320:
+ if (parse_mac_addr(tokens[2], &a->encap.mpls.ether.da) ||
+ parse_mac_addr(tokens[3], &a->encap.mpls.ether.sa) ||
+ strcmp(tokens[4], "label0") ||
+ parser_read_uint32(&label, tokens[5]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[6]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[7]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#1033: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2338:
+ if ((n_tokens == 0) || strcmp(tokens[0], "label1")) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 4'
#1040: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2345:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'label > 0xFFFFF'
#1040: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2345:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tc > 0x7'
#1040: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2345:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ttl > 0x3F'
#1040: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2345:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#1056: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2361:
+ if ((n_tokens == 0) || strcmp(tokens[0], "label2")) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 4'
#1063: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2368:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'label > 0xFFFFF'
#1063: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2368:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tc > 0x7'
#1063: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2368:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ttl > 0x3F'
#1063: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2368:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens == 0'
#1079: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2384:
+ if ((n_tokens == 0) || strcmp(tokens[0], "label3")) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 4'
#1086: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2391:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'label > 0xFFFFF'
#1086: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2391:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tc > 0x7'
#1086: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2391:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ttl > 0x3F'
#1086: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2391:
+ if ((n_tokens < 4) ||
+ parser_read_uint32(&label, tokens[1]) ||
+ (label > 0xFFFFF) ||
+ parser_read_uint32(&tc, tokens[2]) ||
+ (tc > 0x7) ||
+ parser_read_uint32(&ttl, tokens[3]) ||
+ (ttl > 0x3F))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 4'
#1107: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2412:
+ if ((n_tokens < 4) ||
+ parse_mac_addr(tokens[1], &a->encap.pppoe.ether.da) ||
+ parse_mac_addr(tokens[2], &a->encap.pppoe.ether.sa) ||
+ parser_read_uint16(&a->encap.pppoe.pppoe.session_id,
+ tokens[3]))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 4'
#1127: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2432:
+ if ((n_tokens < 4) ||
+ strcmp(tokens[0], "nat"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 2'
#1169: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2474:
+ if ((n_tokens < 2) ||
+ strcmp(tokens[0], "ttl"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 1'
#1189: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2494:
+ if ((n_tokens < 1) ||
+ strcmp(tokens[0], "stats"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 1'
#1204: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2509:
+ if ((n_tokens < 1) ||
+ strcmp(tokens[0], "time"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens < 2'
#1224: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2529:
+ if ((n_tokens < 2) ||
+ strcmp(tokens[0], "action"))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens != 11'
#1474: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2779:
+ if ((n_tokens != 11) && (n_tokens != 12)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens != 12'
#1474: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2779:
+ if ((n_tokens != 11) && (n_tokens != 12)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_rules == 0'
#1648: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2953:
+ if ((parser_read_uint32(&n_rules, tokens[8]) != 0) ||
+ (n_rules == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match == NULL'
#1658: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2963:
+ if ((match == NULL) || (action == NULL) || (data == NULL)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action == NULL'
#1658: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2963:
+ if ((match == NULL) || (action == NULL) || (data == NULL)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'data == NULL'
#1658: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:2963:
+ if ((match == NULL) || (action == NULL) || (data == NULL)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 3'
#1983: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3288:
+ if ((n_tokens >= 3) &&
+ (strcmp(tokens[2], "period") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 5'
#1989: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3294:
+ if ((n_tokens >= 5) &&
+ (strcmp(tokens[2], "port") == 0) &&
+ (strcmp(tokens[3], "in") == 0) &&
+ (strcmp(tokens[4], "bsz") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 5'
#1997: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3302:
+ if ((n_tokens >= 5) &&
+ (strcmp(tokens[2], "port") == 0) &&
+ (strcmp(tokens[3], "out") == 0) &&
+ (strcmp(tokens[4], "bsz") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 4'
#2005: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3310:
+ if ((n_tokens >= 4) &&
+ (strcmp(tokens[2], "table") == 0) &&
+ (strcmp(tokens[3], "match") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 6'
#2012: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3317:
+ if ((n_tokens >= 6) &&
+ (strcmp(tokens[2], "port") == 0) &&
+ (strcmp(tokens[3], "in") == 0) &&
+ (strcmp(tokens[5], "table") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 6'
#2021: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3326:
+ if ((n_tokens >= 6) &&
+ (strcmp(tokens[2], "port") == 0) &&
+ (strcmp(tokens[3], "in") == 0) &&
+ (strcmp(tokens[5], "enable") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 6'
#2030: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3335:
+ if ((n_tokens >= 6) &&
+ (strcmp(tokens[2], "port") == 0) &&
+ (strcmp(tokens[3], "in") == 0) &&
+ (strcmp(tokens[5], "disable") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 7'
#2039: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3344:
+ if ((n_tokens >= 7) &&
+ (strcmp(tokens[2], "table") == 0) &&
+ (strcmp(tokens[4], "rule") == 0) &&
+ (strcmp(tokens[5], "add") == 0) &&
+ (strcmp(tokens[6], "match") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 8'
#2044: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3349:
+ if ((n_tokens >= 8) &&
+ (strcmp(tokens[7], "default") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 7'
#2056: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3361:
+ if ((n_tokens >= 7) &&
+ (strcmp(tokens[2], "table") == 0) &&
+ (strcmp(tokens[4], "rule") == 0) &&
+ (strcmp(tokens[5], "add") == 0) &&
+ (strcmp(tokens[6], "bulk") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 7'
#2066: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3371:
+ if ((n_tokens >= 7) &&
+ (strcmp(tokens[2], "table") == 0) &&
+ (strcmp(tokens[4], "rule") == 0) &&
+ (strcmp(tokens[5], "delete") == 0) &&
+ (strcmp(tokens[6], "match") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 8'
#2071: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3376:
+ if ((n_tokens >= 8) &&
+ (strcmp(tokens[7], "default") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 5'
#2085: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3390:
+ if ((n_tokens >= 5) &&
+ (strcmp(tokens[4], "enable") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_tokens >= 5'
#2092: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3397:
+ if ((n_tokens >= 5) &&
+ (strcmp(tokens[4], "disable") == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'file_name == NULL'
#2113: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3418:
+ if ((file_name == NULL) ||
+ (strlen(file_name) == 0) ||
+ (msg_in_len_max == 0) ||
+ (msg_out_len_max == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'msg_in_len_max == 0'
#2113: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3418:
+ if ((file_name == NULL) ||
+ (strlen(file_name) == 0) ||
+ (msg_in_len_max == 0) ||
+ (msg_out_len_max == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'msg_out_len_max == 0'
#2113: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3418:
+ if ((file_name == NULL) ||
+ (strlen(file_name) == 0) ||
+ (msg_in_len_max == 0) ||
+ (msg_out_len_max == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'msg_in == NULL'
#2121: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3426:
+ if ((msg_in == NULL) ||
+ (msg_out == NULL)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'msg_out == NULL'
#2121: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3426:
+ if ((msg_in == NULL) ||
+ (msg_out == NULL)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'file_name == NULL'
#2176: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3481:
+ if ((file_name == NULL) ||
+ (strlen(file_name) == 0) ||
+ (line_len_max == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'line_len_max == 0'
#2176: FILE: drivers/net/softnic/rte_eth_softnic_cli.c:3481:
+ if ((file_name == NULL) ||
+ (strlen(file_name) == 0) ||
+ (line_len_max == 0)) {
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match == NULL'
#2418: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:694:
+ if ((match == NULL) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2418: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:694:
+ if ((match == NULL) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2418: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:694:
+ if ((match == NULL) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 't->ip_version == 0'
#2433: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:709:
+ if ((r->ip_version && (t->ip_version == 0)) ||
+ ((r->ip_version == 0) && t->ip_version))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'r->ip_version == 0'
#2433: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:709:
+ if ((r->ip_version && (t->ip_version == 0)) ||
+ ((r->ip_version == 0) && t->ip_version))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'r->sa_depth > 32'
#2438: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:714:
+ if ((r->sa_depth > 32) ||
+ (r->da_depth > 32))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'r->da_depth > 32'
#2438: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:714:
+ if ((r->sa_depth > 32) ||
+ (r->da_depth > 32))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'r->sa_depth > 128'
#2442: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:718:
+ if ((r->sa_depth > 128) ||
+ (r->da_depth > 128))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'r->da_depth > 128'
#2442: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:718:
+ if ((r->sa_depth > 128) ||
+ (r->da_depth > 128))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 't->key_size != 4'
#2460: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:736:
+ if ((r->ip_version && (t->key_size != 4)) ||
+ ((r->ip_version == 0) && (t->key_size != 16)))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'r->ip_version == 0'
#2460: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:736:
+ if ((r->ip_version && (t->key_size != 4)) ||
+ ((r->ip_version == 0) && (t->key_size != 16)))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 't->key_size != 16'
#2460: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:736:
+ if ((r->ip_version && (t->key_size != 4)) ||
+ ((r->ip_version == 0) && (t->key_size != 16)))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action == NULL'
#2489: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:765:
+ if ((action == NULL) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2489: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:765:
+ if ((action == NULL) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2489: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:765:
+ if ((action == NULL) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.action == RTE_PIPELINE_ACTION_PORT'
#2499: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:775:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_PORT) &&
+ (action->fwd.id >= p->n_ports_out))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.id >= p->n_ports_out'
#2499: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:775:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_PORT) &&
+ (action->fwd.id >= p->n_ports_out))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.action == RTE_PIPELINE_ACTION_TABLE'
#2503: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:779:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_TABLE) &&
+ (action->fwd.id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.id >= p->n_tables'
#2503: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:779:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_TABLE) &&
+ (action->fwd.id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'subport_id >= n_subports_per_port'
#2524: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:800:
+ if ((subport_id >= n_subports_per_port) ||
+ (pipe_id >= n_pipes_per_subport))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pipe_id >= n_pipes_per_subport'
#2524: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:800:
+ if ((subport_id >= n_subports_per_port) ||
+ (pipe_id >= n_pipes_per_subport))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ip_version0 == 0'
#2541: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:817:
+ if ((ip_version1 && (ip_version0 == 0)) ||
+ ((ip_version1 == 0) && ip_version0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ip_version1 == 0'
#2541: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:817:
+ if ((ip_version1 && (ip_version0 == 0)) ||
+ ((ip_version1 == 0) && ip_version0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action == NULL'
#2554: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:830:
+ if ((action == NULL) ||
+ (action->action_mask != (1LLU << RTE_TABLE_ACTION_FWD)) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2554: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:830:
+ if ((action == NULL) ||
+ (action->action_mask != (1LLU << RTE_TABLE_ACTION_FWD)) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2554: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:830:
+ if ((action == NULL) ||
+ (action->action_mask != (1LLU << RTE_TABLE_ACTION_FWD)) ||
+ (p == NULL) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.action == RTE_PIPELINE_ACTION_PORT'
#2561: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:837:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_PORT) &&
+ (action->fwd.id >= p->n_ports_out))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.id >= p->n_ports_out'
#2561: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:837:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_PORT) &&
+ (action->fwd.id >= p->n_ports_out))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.action == RTE_PIPELINE_ACTION_TABLE'
#2565: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:841:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_TABLE) &&
+ (action->fwd.id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action->fwd.id >= p->n_tables'
#2565: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:841:
+ if ((action->fwd.action == RTE_PIPELINE_ACTION_TABLE) &&
+ (action->fwd.id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pipeline_name == NULL'
#2587: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:863:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match == NULL'
#2587: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:863:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action == NULL'
#2587: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:863:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'data == NULL'
#2587: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:863:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2594: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:870:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ match_check(match, p, table_id) ||
+ action_check(action, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p->enabled == 0'
#2594: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:870:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ match_check(match, p, table_id) ||
+ action_check(action, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2594: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:870:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ match_check(match, p, table_id) ||
+ action_check(action, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pipeline_name == NULL'
#2641: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:917:
+ if ((pipeline_name == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action == NULL'
#2641: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:917:
+ if ((pipeline_name == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'data == NULL'
#2641: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:917:
+ if ((pipeline_name == NULL) ||
+ (action == NULL) ||
+ (data == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2647: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:923:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ action_default_check(action, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p->enabled == 0'
#2647: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:923:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ action_default_check(action, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2647: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:923:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ action_default_check(action, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pipeline_name == NULL'
#2695: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:971:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL) ||
+ (n_rules == NULL) ||
+ (*n_rules == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match == NULL'
#2695: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:971:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL) ||
+ (n_rules == NULL) ||
+ (*n_rules == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action == NULL'
#2695: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:971:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL) ||
+ (n_rules == NULL) ||
+ (*n_rules == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'data == NULL'
#2695: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:971:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL) ||
+ (n_rules == NULL) ||
+ (*n_rules == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'n_rules == NULL'
#2695: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:971:
+ if ((pipeline_name == NULL) ||
+ (match == NULL) ||
+ (action == NULL) ||
+ (data == NULL) ||
+ (n_rules == NULL) ||
+ (*n_rules == 0))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2704: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:980:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p->enabled == 0'
#2704: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:980:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2704: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:980:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'pipeline_name == NULL'
#2757: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1033:
+ if ((pipeline_name == NULL) ||
+ (match == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match == NULL'
#2757: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1033:
+ if ((pipeline_name == NULL) ||
+ (match == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2762: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1038:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ match_check(match, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p->enabled == 0'
#2762: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1038:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ match_check(match, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2762: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1038:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables) ||
+ match_check(match, p, table_id))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p == NULL'
#2807: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1083:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p->enabled == 0'
#2807: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1083:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'table_id >= p->n_tables'
#2807: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1083:
+ if ((p == NULL) ||
+ (p->enabled == 0) ||
+ (table_id >= p->n_tables))
CHECK:SPACING: No space is necessary after a cast
#2935: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1257:
+ (int32_t) mh->match.acl.priority;
CHECK:SPACING: No space is necessary after a cast
#2965: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1287:
+ (uint32_t *) mh->match.acl.ipv6.sa;
CHECK:SPACING: No space is necessary after a cast
#2967: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1289:
+ (uint32_t *) mh->match.acl.ipv6.da;
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#2971: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1293:
+ status = match_convert_ipv6_depth(
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#2977: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1299:
+ status = match_convert_ipv6_depth(
CHECK:SPACING: No space is necessary after a cast
#3025: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1347:
+ (int32_t) mh->match.acl.priority;
CHECK:SPACING: No space is necessary after a cast
#3028: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1350:
+ (uint32_t *) mh->match.acl.ipv6.sa;
CHECK:SPACING: No space is necessary after a cast
#3030: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1352:
+ (uint32_t *) mh->match.acl.ipv6.da;
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#3034: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1356:
+ status = match_convert_ipv6_depth(
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#3040: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1362:
+ status = match_convert_ipv6_depth(
CHECK:SPACING: No space is necessary after a cast
#3127: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1449:
+ struct pipeline_msg_rsp *rsp = (struct pipeline_msg_rsp *) req;
CHECK:SPACING: No space is necessary after a cast
#3137: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1459:
+ data_in = (struct rte_pipeline_table_entry *) p->buffer;
CHECK:SPACING: No space is necessary after a cast
#3276: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1598:
+ struct pipeline_msg_rsp *rsp = (struct pipeline_msg_rsp *) req;
CHECK:SPACING: No space is necessary after a cast
#3284: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1606:
+ data_in = (struct rte_pipeline_table_entry *) p->buffer;
CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#3313: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1635:
+{
+
CHECK:SPACING: No space is necessary after a cast
#3314: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1636:
+ struct pipeline_msg_rsp *rsp = (struct pipeline_msg_rsp *) req;
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match_ll == NULL'
#3340: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1662:
+ if ((match_ll == NULL) ||
+ (action_ll == NULL) ||
+ (match_ll_ptr == NULL) ||
+ (action_ll_ptr == NULL) ||
+ (found == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action_ll == NULL'
#3340: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1662:
+ if ((match_ll == NULL) ||
+ (action_ll == NULL) ||
+ (match_ll_ptr == NULL) ||
+ (action_ll_ptr == NULL) ||
+ (found == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'match_ll_ptr == NULL'
#3340: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1662:
+ if ((match_ll == NULL) ||
+ (action_ll == NULL) ||
+ (match_ll_ptr == NULL) ||
+ (action_ll_ptr == NULL) ||
+ (found == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'action_ll_ptr == NULL'
#3340: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1662:
+ if ((match_ll == NULL) ||
+ (action_ll == NULL) ||
+ (match_ll_ptr == NULL) ||
+ (action_ll_ptr == NULL) ||
+ (found == NULL))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'found == NULL'
#3340: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1662:
+ if ((match_ll == NULL) ||
+ (action_ll == NULL) ||
+ (match_ll_ptr == NULL) ||
+ (action_ll_ptr == NULL) ||
+ (found == NULL))
WARNING:LONG_LINE: line over 90 characters
#3350: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1672:
+ (struct rte_pipeline_table_entry *)&action_ll[i * TABLE_RULE_ACTION_SIZE_MAX];
CHECK:BRACES: Unbalanced braces around else statement
#3467: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1789:
+ } else
CHECK:SPACING: No space is necessary after a cast
#3511: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1833:
+ struct pipeline_msg_rsp *rsp = (struct pipeline_msg_rsp *) req;
CHECK:SPACING: No space is necessary after a cast
#3535: FILE: drivers/net/softnic/rte_eth_softnic_thread.c:1857:
+ struct pipeline_msg_rsp *rsp = (struct pipeline_msg_rsp *) req;
total: 0 errors, 1 warnings, 165 checks, 3529 lines checked
parent reply other threads:[~2018-06-08 14:19 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <20180608124155.140663-17-jasvinder.singh@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=20180608141931.200ED1BA96@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=jasvinder.singh@intel.com \
--cc=test-report@dpdk.org \
/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).