netdev - Re: [net-next,v5,05/13] octeontx2-af: npc: cn20k: Allocate default MCAM indexes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20260128022921.4152787-1-kuba@kernel.org>
Date: Tue, 27 Jan 2026 18:29:21 -0800
From: Jakub Kicinski <kuba@...nel.org>
To: rkannoth@...vell.com
Cc: Jakub Kicinski <kuba@...nel.org>,
	sbhatta@...vell.com,
	edumazet@...gle.com,
	andrew+netdev@...n.ch,
	netdev@...r.kernel.org,
	sgoutham@...vell.com,
	sumang@...vell.com,
	pabeni@...hat.com,
	davem@...emloft.net,
	linux-kernel@...r.kernel.org
Subject: Re: [net-next,v5,05/13] octeontx2-af: npc: cn20k: Allocate default MCAM indexes

This is an AI-generated review of your patch. The human sending this
email has considered the AI review valid, or at least plausible.

For local reproduction steps see https://netdev-ai.bots.linux.dev/ai-local.html
---
octeontx2-af: npc: cn20k: Allocate default MCAM indexes

This patch allocates MCAM indexes for promiscuous, multicast, broadcast,
and unicast traffic when NIX LF is attached to PF/VF, improving
allocation efficiency over the previous reservation method.

> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.c b/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.c
> index dd5352bd6dbf..6ae462d6ebaa 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.c

[ ... ]

> +chk_sanity:
> +	dev_dbg(rvu->dev,
> +		"%s: Default index for pcifunc=%#x, bcast=%u mcast=%u promise=%u ucast=%u cnt=%u\n",
                                                                       ^^^^^^^

Should "promise" be "promisc" here? This appears to be a typo in the debug
message.

[ ... ]

> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.h b/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.h
> index aadee502bac1..f1734435db4d 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.h
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/cn20k/npc.h

[ ... ]

> +/**
> + * enum npc_dft_rule_id - Default rule type
> + *
> + * Mcam default rule type.
> + *
> + * @NPC_DFT_RULE_START_ID:	Not used
> + * @NPC_DFT_RULE_PROMISC_ID:	promiscuous rule
> + * @NPC_DFT_RULE_MCAST_ID:	promiscuous rule
                                ^^^^^^^^^^^^^^^^

Should this say "multicast rule" instead of "promiscuous rule"? It appears
to be copied from the PROMISC line above.

> + * @NPC_DFT_RULE_BCAST_ID:	broadcast rule
> + * @NPC_DFT_RULE_UCAST_ID:	unicast rule
> + * @NPC_DFT_RULE_MAX_ID:	Maximum index.
> + */
-- 
pw-bot: cr