* [linux-next:master 2314/12403] drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:575:9-32: duplicated argument to & or | (fwd)
@ 2024-01-05 8:15 Julia Lawall
0 siblings, 0 replies; only message in thread
From: Julia Lawall @ 2024-01-05 8:15 UTC (permalink / raw)
To: Geetha sowjanya; +Cc: Linux Memory Management List, oe-kbuild-all
Hello,
Please check lines 575 and 576, which are identical.
julia
---------- Forwarded message ----------
Date: Fri, 5 Jan 2024 12:29:52 +0800
From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Julia Lawall <julia.lawall@inria.fr>
Subject: [linux-next:master 2314/12403]
drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:575:9-32: duplicated
argument to & or |
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: Linux Memory Management List <linux-mm@kvack.org>
TO: Geetha sowjanya <gakula@marvell.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: d0b3c8aa5e37775cd7c3ac07b256218df0fd6678
commit: a8d4879d5f1fac060719567d1a8e8f8e68a127fc [2314/12403] octeontx2-pf: TC flower offload support for ICMP type and code
:::::: branch date: 23 hours ago
:::::: commit date: 6 weeks ago
config: arm64-randconfig-r053-20240104 (https://download.01.org/0day-ci/archive/20240105/202401051229.BdWGEvJ9-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 13.2.0
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Julia Lawall <julia.lawall@inria.fr>
| Closes: https://lore.kernel.org/r/202401051229.BdWGEvJ9-lkp@intel.com/
cocci warnings: (new ones prefixed by >>)
>> drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:575:9-32: duplicated argument to & or |
drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:575:35-58: duplicated argument to & or |
vim +575 drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 557
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 558 static void npc_set_features(struct rvu *rvu, int blkaddr, u8 intf)
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 559 {
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 560 struct npc_mcam *mcam = &rvu->hw->mcam;
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 561 u64 *features = &mcam->rx_features;
a8d4879d5f1fac Geetha sowjanya 2023-11-22 562 u64 proto_flags;
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 563 int hdr;
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 564
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 565 if (is_npc_intf_tx(intf))
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 566 features = &mcam->tx_features;
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 567
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 568 for (hdr = NPC_DMAC; hdr < NPC_HEADER_FIELDS_MAX; hdr++) {
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 569 if (npc_check_field(rvu, blkaddr, hdr, intf))
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 570 *features |= BIT_ULL(hdr);
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 571 }
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 572
a8d4879d5f1fac Geetha sowjanya 2023-11-22 573 proto_flags = BIT_ULL(NPC_SPORT_TCP) | BIT_ULL(NPC_SPORT_UDP) |
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 574 BIT_ULL(NPC_DPORT_TCP) | BIT_ULL(NPC_DPORT_UDP) |
a8d4879d5f1fac Geetha sowjanya 2023-11-22 @575 BIT_ULL(NPC_SPORT_SCTP) | BIT_ULL(NPC_DPORT_SCTP) |
a8d4879d5f1fac Geetha sowjanya 2023-11-22 576 BIT_ULL(NPC_SPORT_SCTP) | BIT_ULL(NPC_DPORT_SCTP) |
a8d4879d5f1fac Geetha sowjanya 2023-11-22 577 BIT_ULL(NPC_TYPE_ICMP) | BIT_ULL(NPC_CODE_ICMP);
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 578
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 579 /* for tcp/udp/sctp corresponding layer type should be in the key */
a8d4879d5f1fac Geetha sowjanya 2023-11-22 580 if (*features & proto_flags) {
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 581 if (!npc_check_field(rvu, blkaddr, NPC_LD, intf))
a8d4879d5f1fac Geetha sowjanya 2023-11-22 582 *features &= ~proto_flags;
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 583 else
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 584 *features |= BIT_ULL(NPC_IPPROTO_TCP) |
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 585 BIT_ULL(NPC_IPPROTO_UDP) |
a8d4879d5f1fac Geetha sowjanya 2023-11-22 586 BIT_ULL(NPC_IPPROTO_SCTP) |
a8d4879d5f1fac Geetha sowjanya 2023-11-22 587 BIT_ULL(NPC_IPPROTO_ICMP);
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 588 }
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 589
2b9cef66790209 Naveen Mamindlapalli 2021-03-18 590 /* for AH/ICMP/ICMPv6/, check if corresponding layer type is present in the key */
2b9cef66790209 Naveen Mamindlapalli 2021-03-18 591 if (npc_check_field(rvu, blkaddr, NPC_LD, intf)) {
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 592 *features |= BIT_ULL(NPC_IPPROTO_AH);
2b9cef66790209 Naveen Mamindlapalli 2021-03-18 593 *features |= BIT_ULL(NPC_IPPROTO_ICMP);
2b9cef66790209 Naveen Mamindlapalli 2021-03-18 594 *features |= BIT_ULL(NPC_IPPROTO_ICMP6);
2b9cef66790209 Naveen Mamindlapalli 2021-03-18 595 }
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 596
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 597 /* for ESP, check if corresponding layer type is present in the key */
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 598 if (npc_check_field(rvu, blkaddr, NPC_LE, intf))
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 599 *features |= BIT_ULL(NPC_IPPROTO_ESP);
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 600
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 601 /* for vlan corresponding layer type should be in the key */
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 602 if (*features & BIT_ULL(NPC_OUTER_VID))
b7cf966126eb16 Naveen Mamindlapalli 2021-01-11 603 if (!npc_check_field(rvu, blkaddr, NPC_LB, intf))
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 604 *features &= ~BIT_ULL(NPC_OUTER_VID);
dce677da57c0ed Subbaraya Sundeep 2021-08-27 605
12aa0a3b93f3ad Ratheesh Kannoth 2023-08-09 606 /* Set SPI flag only if AH/ESP and IPSEC_SPI are in the key */
12aa0a3b93f3ad Ratheesh Kannoth 2023-08-09 607 if (npc_check_field(rvu, blkaddr, NPC_IPSEC_SPI, intf) &&
12aa0a3b93f3ad Ratheesh Kannoth 2023-08-09 608 (*features & (BIT_ULL(NPC_IPPROTO_ESP) | BIT_ULL(NPC_IPPROTO_AH))))
73b4c04e2e9af8 Ratheesh Kannoth 2023-08-01 609 *features |= BIT_ULL(NPC_IPSEC_SPI);
73b4c04e2e9af8 Ratheesh Kannoth 2023-08-01 610
dce677da57c0ed Subbaraya Sundeep 2021-08-27 611 /* for vlan ethertypes corresponding layer type should be in the key */
dce677da57c0ed Subbaraya Sundeep 2021-08-27 612 if (npc_check_field(rvu, blkaddr, NPC_LB, intf))
dce677da57c0ed Subbaraya Sundeep 2021-08-27 613 *features |= BIT_ULL(NPC_VLAN_ETYPE_CTAG) |
dce677da57c0ed Subbaraya Sundeep 2021-08-27 614 BIT_ULL(NPC_VLAN_ETYPE_STAG);
2cee6401c4eaa5 Suman Ghosh 2022-10-31 615
2cee6401c4eaa5 Suman Ghosh 2022-10-31 616 /* for L2M/L2B/L3M/L3B, check if the type is present in the key */
2cee6401c4eaa5 Suman Ghosh 2022-10-31 617 if (npc_check_field(rvu, blkaddr, NPC_LXMB, intf))
2cee6401c4eaa5 Suman Ghosh 2022-10-31 618 *features |= BIT_ULL(NPC_LXMB);
a63df366d0915a Hariprasad Kelam 2023-09-21 619
a63df366d0915a Hariprasad Kelam 2023-09-21 620 for (hdr = NPC_MPLS1_LBTCBOS; hdr <= NPC_MPLS4_TTL; hdr++) {
a63df366d0915a Hariprasad Kelam 2023-09-21 621 if (npc_check_field(rvu, blkaddr, hdr, intf))
a63df366d0915a Hariprasad Kelam 2023-09-21 622 *features |= BIT_ULL(hdr);
a63df366d0915a Hariprasad Kelam 2023-09-21 623 }
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 624 }
9b179a960a96d5 Subbaraya Sundeep 2020-11-15 625
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2024-01-05 8:15 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-05 8:15 [linux-next:master 2314/12403] drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:575:9-32: duplicated argument to & or | (fwd) Julia Lawall
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox