> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Wednesday, 22 February 2023 09.01 > > 21/02/2023 21:18, Kamalakshitha Aligeri: > > Hi Thomas, > > > > Do you want me to squash the unit tests in the mempool lib patch or do I > have to wait for the reviews from mempool maintainers > > Yes I think you can do the squash if Morten agrees.
Yes, I agree. And if there are any more changes required before the code is accepted by the maintainers, I will let Kamalakshitha make those changes. How should the different acks/review tags be handled when squashing two patches into one? The library patch has: Signed-off-by: Morten Brørup <m...@smartsharesystems.com> Acked-by: Konstantin Ananyev <konstantin.v.anan...@yandex.ru> Acked-by: Chengwen Feng <fengcheng...@huawei.com> Acked-by: Kamalakshitha Aligeri <kamalakshitha.alig...@arm.com> And the test patch has: Signed-off-by: Kamalakshitha Aligeri <kamalakshitha.alig...@arm.com> Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com> Reviewed-by: Feifei Wang <feifei.wa...@arm.com> Acked-by: Morten Brørup <m...@smartsharesystems.com> > > > > From: Thomas Monjalon <tho...@monjalon.net> > > > 10/02/2023 08:33, Morten Brørup: > > > > > From: Kamalakshitha Aligeri [mailto:kamalakshitha.alig...@arm.com] > > > > > Sent: Friday, 10 February 2023 07.54 > > > > > > > > > > Added mempool test cases with zero-copy get and put API's > > > > > > > > > > Signed-off-by: Kamalakshitha Aligeri <kamalakshitha.alig...@arm.com> > > > > > Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com> > > > > > Reviewed-by: Feifei Wang <feifei.wa...@arm.com> > > > > > > > > I already acked v1 of this patch, but here it is again for Patchwork: > > > > > > > > Acked-by: Morten Brørup <m...@smartsharesystems.com> > > > > > > > > > --- > > > > > Link: > > > > > > > > https://patchwork.dpdk.org/project/dpdk/patch/20230209145833.129986-1- > > > > > m...@smartsharesystems.com/ > > > > > > > > @David, here's the zero-copy mempool cache API test cases you were > > > asking for. > > > > > > The unit tests should be squashed in the mempool lib patch. > > > > > > Also I would to see a review from the mempool maintainers.
