On 10/19/21 11:09 PM, David Marchand wrote: > On Tue, Oct 19, 2021 at 7:40 PM Andrew Rybchenko > <andrew.rybche...@oktetlabs.ru> 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_F_NON_IO >> - fix deprecation usage >> - add Fixes tag the patch which deprecates unused macros > > 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 wait tomorrow, to see if Olivier wants to send some acks.
Olivier has just added missing Acks. Do you need v4 from me with patch 2 fixes? Your changes LGTM and I don't mind if you fix it on apply.