[EXT] Re: [PATCH] net/cnxk: fix deadlock in security session creation

[Nithin Kumar Dabilpuram]

> -----Original Message-----
> From: Zhang, Fan <fanzhang.oss at gmail.com>
> Sent: Thursday, January 5, 2023 7:35 PM
> To: David Marchand <david.marchand at redhat.com>; dev at dpdk.org
> Cc: stable at dpdk.org; Nithin Kumar Dabilpuram <ndabilpuram at marvell.com>; Kiran Kumar
> Kokkilagadda <kirankumark at marvell.com>; Sunil Kumar Kori <skori at marvell.com>; Satha
> Koteswara Rao Kottidi <skoteshwar at marvell.com>; Vidya Sagar Velumuri
> <vvelumuri at marvell.com>
> Subject: [EXT] Re: [PATCH] net/cnxk: fix deadlock in security session creation
> 
> External Email
> 
> ----------------------------------------------------------------------
> 
> On 1/5/2023 1:57 PM, David Marchand wrote:
> > Releasing the lock was missing in this branch.
> >
> > Fixes: 4440eb88ddfc ("net/cnxk: use full context IPsec structures")
> > Cc: stable at dpdk.org
> >
> > Signed-off-by: David Marchand <david.marchand at redhat.com>
> > ---
> >   drivers/net/cnxk/cn9k_ethdev_sec.c | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/net/cnxk/cn9k_ethdev_sec.c b/drivers/net/cnxk/cn9k_ethdev_sec.c
> > index 67966a4e49..327f221e38 100644
> > --- a/drivers/net/cnxk/cn9k_ethdev_sec.c
> > +++ b/drivers/net/cnxk/cn9k_ethdev_sec.c
> > @@ -556,6 +556,7 @@ cn9k_eth_sec_session_create(void *device,
> >
> >   	if (!dev->outb.lf_base) {
> >   		plt_err("Could not allocate security session private data");
> > +		rte_spinlock_unlock(lock);
> >   		return -ENOMEM;
> >   	}
> >
> Acked-by: Fan Zhang <fanzhang.oss at gmail.com>

Acked-by: Nithin Dabilpuram <ndabilpuram at marvell.com>