> -----Original Message----- > From: Zhang, Fan <fanzhang....@gmail.com> > Sent: Thursday, January 5, 2023 7:35 PM > To: David Marchand <david.march...@redhat.com>; dev@dpdk.org > Cc: sta...@dpdk.org; Nithin Kumar Dabilpuram <ndabilpu...@marvell.com>; Kiran > Kumar > Kokkilagadda <kirankum...@marvell.com>; Sunil Kumar Kori <sk...@marvell.com>; > Satha > Koteswara Rao Kottidi <skotesh...@marvell.com>; Vidya Sagar Velumuri > <vvelum...@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: sta...@dpdk.org > > > > Signed-off-by: David Marchand <david.march...@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....@gmail.com>
Acked-by: Nithin Dabilpuram <ndabilpu...@marvell.com>