RE: [EXTERNAL] Re: [PATCH 22.11] net/cnxk: fix lock for security session ops

[Rahul Bhansali <rbhansali@marvell.com>]

> -----Original Message-----
> From: Luca Boccassi <luca.boccassi@gmail.com>
> Sent: Friday, July 4, 2025 6:32 AM
> To: Rahul Bhansali <rbhansali@marvell.com>
> Cc: stable@dpdk.org
> Subject: [EXTERNAL] Re: [PATCH 22.11] net/cnxk: fix lock for security session
> ops
> 
> On Thu, 3 Jul 2025 at 12: 24, Rahul Bhansali <rbhansali@ marvell. com> wrote:
> > > [ upstream commit 9bebc33703df999a405ed7103dc45230d0f1fbda ] > >
> Add fixes to have lock on security session update, write and read to > prevent
> ZjQcmQRYFpfptBannerStart Prioritize security for external emails:
> Confirm sender and content safety before clicking links or opening
> attachments <https://us-phishalarm-
> ewt.proofpoint.com/EWT/v1/CRVmXkqW!uK3X-
> 9D8Q9iROC93VDO7ZZV175I0spRBIzgtaAMKCcgV14TBwRsdHtiQf9se748hO6
> Zmwbq8WUgLG4SNJCqYgfGLGBoJIe8NLMbE8-
> PbkLpwx0MhnTDuTO8ukI2X$>
> Report Suspicious
> 
> ZjQcmQRYFpfptBannerEnd
> On Thu, 3 Jul 2025 at 12:24, Rahul Bhansali <rbhansali@marvell.com> wrote:
> >
> > [ upstream commit 9bebc33703df999a405ed7103dc45230d0f1fbda ]
> >
> > Add fixes to have lock on security session update, write and read to
> > prevent corruption.
> >
> > Fixes: 8efa348e8160 ("net/cnxk: support custom SA index")
> >
> > Signed-off-by: Rahul Bhansali <rbhansali@marvell.com>
> > ---
> >  drivers/net/cnxk/cn10k_ethdev_sec.c | 107
> > +++++++++++++++++++++++-----
> >  1 file changed, 90 insertions(+), 17 deletions(-)
> 
> Thanks for the backport, but unfortunately it doesn't build, please double
> check it:
Ack. There was an API arg change, I missed it. I will send v2.
Thanks.
> 
> ../drivers/net/cnxk/cn10k_ethdev_sec.c: In function
> ‘rte_pmd_cnxk_hw_sa_read’:
> ../drivers/net/cnxk/cn10k_ethdev_sec.c:1013:16: error: ‘inb’
> undeclared (first use in this function); did you mean ‘int’?
>  1013 |         lock = inb ? &dev->inb.lock : &dev->outb.lock;
>       |                ^~~
>       |                int
> ../drivers/net/cnxk/cn10k_ethdev_sec.c:1013:16: note: each undeclared
> identifier is reported only once for each function it appears in
> ../drivers/net/cnxk/cn10k_ethdev_sec.c: In function
> ‘rte_pmd_cnxk_hw_sa_write’:
> ../drivers/net/cnxk/cn10k_ethdev_sec.c:1057:16: error: ‘inb’
> undeclared (first use in this function); did you mean ‘int’?
>  1057 |         lock = inb ? &dev->inb.lock : &dev->outb.lock;
>       |                ^~~
>       |                int