> -----Original Message-----
> From: David Marchand <[email protected]>
> Sent: Sunday, June 28, 2026 3:57 PM
> To: Dariusz Sosnowski <[email protected]>
> Cc: Bruce Richardson <[email protected]>; NBU-Contact-Thomas
> Monjalon (EXTERNAL) <[email protected]>; Andrew Rybchenko
> <[email protected]>; Slava Ovsiienko
> <[email protected]>; Bing Zhao <[email protected]>; Ori Kam
> <[email protected]>; Suanming Mou <[email protected]>; Matan
> Azrad <[email protected]>; [email protected]; Yu Jiang <[email protected]>
> Subject: Re: [PATCH v4 0/4] add versioned symbols for recently stabilized APIs
> 
> External email: Use caution opening links or attachments
> 
> 
> Hello Dariusz,
> 
> On Thu, 25 Jun 2026 at 18:08, Dariusz Sosnowski <[email protected]>
> wrote:
> >
> > Main goal of this patchset is to address
> https://bugs.dpdk.org/show_bug.cgi?id=1957
> > but it also handles other recently stabilized symbols and has some minor
> fixes:
> >
> > - Patch 1 - Fix RTE_VERSION_EXPERIMENTAL_SYMBOL macro on clang.
> > - Patch 2 - Allow function versioning inside drivers.
> > - Patch 3 - Version the function symbols stabilized in
> >
> https://git.dpdk.org/dpdk/commit/?id=e8cab133645f5466ef75e511629add43
> b68a5027
> > - Patch 4 - Version the rte_flow_dynf_metadata_register() function 
> > stabilized
> in
> >
> https://git.dpdk.org/dpdk/commit/?id=4ee2f5c1cedf9ee7f39afa667f71b07f40
> 04ba5c
> >   Restore EXPERIMENTAL version on global variable symbols
> >   rte_flow_dynf_metadata_offs and rte_flow_dynf_metadata_mask.
> >
> > v4:
> > - Fixed build with older meson versions.
> >
> > v3:
> > - https://inbox.dpdk.org/dev/20260625133311.1299705-1-
> [email protected]/
> > - Added rebuilding of drivers with -DRTE_BUILD_SHARED_LIB
> >   whenever function versioning is enabled.
> >
> > v2:
> > - https://inbox.dpdk.org/dev/20260624131337.1127323-1-
> [email protected]/
> > - Drop patches introducing versioning macros for symbol aliases
> >   and their usage (patch 4 and 5 from v1)
> > - EXPERIMENTAL version on global variable symbols
> >   rte_flow_dynf_metadata_offs and rte_flow_dynf_metadata_mask,
> >   as discussed under v1.
> > - Change commit title prefix in patch (2) from "drivers" to "build".
> >
> > v1: https://inbox.dpdk.org/dev/20260623113752.1100072-1-
> [email protected]/
> >
> > Dariusz Sosnowski (4):
> >   eal: fix macro for versioned experimental symbol
> >   build: support function versioning for drivers
> >   net/mlx5: fix stabilized function versions
> >   ethdev: fix promoted flow metadata symbols
> >
> >  drivers/meson.build                  | 21 ++++++++++++++++++++-
> >  drivers/net/mlx5/meson.build         |  2 ++
> >  drivers/net/mlx5/mlx5_driver_event.c | 22 ++++++++++++++++------
> >  drivers/net/mlx5/mlx5_flow.c         | 18 ++++++++++++------
> >  lib/eal/common/eal_export.h          |  2 +-
> >  lib/ethdev/meson.build               |  2 ++
> >  lib/ethdev/rte_flow.c                | 13 ++++++++-----
> >  7 files changed, 61 insertions(+), 19 deletions(-)
> 
> LGTM.
> Series applied, thanks Dariusz.
> 
> Please send the patch to drop this compat in 26.11.

Patches for dropping these compatibility aliases are on the mailing list:

- 
https://patches.dpdk.org/project/dpdk/patch/[email protected]/
- 
https://patches.dpdk.org/project/dpdk/patch/[email protected]/

Best regards,
Dariusz Sosnowski

Reply via email to