PING. Ashwin Sekhar T K
> -----Original Message----- > From: Ashwin Sekhar T K <[email protected]> > Sent: Thursday, April 28, 2022 3:30 PM > To: [email protected] > Cc: Jerin Jacob Kollanukkaran <[email protected]>; Sunil Kumar Kori > <[email protected]>; Satha Koteswara Rao Kottidi > <[email protected]>; Pavan Nikhilesh Bhagavatula > <[email protected]>; Kiran Kumar Kokkilagadda > <[email protected]>; Satheesh Paul Antonysamy > <[email protected]>; Ashwin Sekhar Thalakalath Kottilveetil > <[email protected]>; Anoob Joseph <[email protected]>; Akhil > Goyal <[email protected]>; Nithin Kumar Dabilpuram > <[email protected]>; Harman Kalra <[email protected]> > Subject: [PATCH] mempool/cnxk: avoid batch op free for empty mempools > > Batch op data is initialized inside mempool alloc. But in case of empty > mempools, the alloc function is not called and hence the initialization of > batch > op data is also not done. So ensure the validity of batch op data inside > mempool free. > > Signed-off-by: Ashwin Sekhar T K <[email protected]> > --- > drivers/mempool/cnxk/cn10k_mempool_ops.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/mempool/cnxk/cn10k_mempool_ops.c > b/drivers/mempool/cnxk/cn10k_mempool_ops.c > index 6ebbf91de5..a02e01cea0 100644 > --- a/drivers/mempool/cnxk/cn10k_mempool_ops.c > +++ b/drivers/mempool/cnxk/cn10k_mempool_ops.c > @@ -109,6 +109,12 @@ batch_op_fini(struct rte_mempool *mp) > int i; > > op_data = batch_op_data_get(mp->pool_id); > + if (!op_data) { > + /* Batch op data can be uninitialized in case of empty > + * mempools. > + */ > + return; > + } > > rte_wmb(); > for (i = 0; i < RTE_MAX_LCORE; i++) { > -- > 2.32.0
