Re: [dpdk-dev] [PATCH v3 0/6] mempool: cleanup namespace

2021-10-20 Thread David Marchand
On Wed, Oct 20, 2021 at 9:52 AM Andrew Rybchenko wrote: > > Thanks for the quick rebase. > > I had rebased v2 before Olivier comments. > > I spotted a little issue diffing with your v3 (see comment on patch > > 2), and fixed your v3 in a local branch of mine. > > It passes my checks. > > > > I'll

Re: [dpdk-dev] [PATCH v3 0/6] mempool: cleanup namespace

2021-10-20 Thread Andrew Rybchenko
On 10/20/21 10:52 AM, David Marchand wrote: > On Tue, Oct 19, 2021 at 10:09 PM David Marchand > wrote: >> On Tue, Oct 19, 2021 at 7:40 PM Andrew Rybchenko >> wrote: >>> >>> Add RTE_ prefix to mempool API including internal. Keep old public API >>> with fallback to new defines. Internal API is jus

Re: [dpdk-dev] [PATCH v3 0/6] mempool: cleanup namespace

2021-10-20 Thread Andrew Rybchenko
On 10/19/21 11:09 PM, David Marchand wrote: > On Tue, Oct 19, 2021 at 7:40 PM Andrew Rybchenko > wrote: >> >> Add RTE_ prefix to mempool API including internal. Keep old public API >> with fallback to new defines. Internal API is just renamed. >> >> v3: >> - fix typo >> - rebase on top of

Re: [dpdk-dev] [PATCH v3 0/6] mempool: cleanup namespace

2021-10-20 Thread David Marchand
On Tue, Oct 19, 2021 at 10:09 PM David Marchand wrote: > On Tue, Oct 19, 2021 at 7:40 PM Andrew Rybchenko > wrote: > > > > Add RTE_ prefix to mempool API including internal. Keep old public API > > with fallback to new defines. Internal API is just renamed. > > > > v3: > > - fix typo > >

Re: [dpdk-dev] [PATCH v3 0/6] mempool: cleanup namespace

2021-10-19 Thread David Marchand
On Tue, Oct 19, 2021 at 7:40 PM Andrew Rybchenko wrote: > > Add RTE_ prefix to mempool API including internal. Keep old public API > with fallback to new defines. Internal API is just renamed. > > v3: > - fix typo > - rebase on top of current main > - add prefix to newly added MEMPOOL_

[dpdk-dev] [PATCH v3 0/6] mempool: cleanup namespace

2021-10-19 Thread Andrew Rybchenko
Add RTE_ prefix to mempool API including internal. Keep old public API with fallback to new defines. Internal API is just renamed. v3: - fix typo - rebase on top of current main - add prefix to newly added MEMPOOL_F_NON_IO - fix deprecation usage - add Fixes tag the patch which