[dpdk-dev] [PATCH v3 1/4] eal: disable function versioning on Windows

[Fady Bader]

> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Tuesday, June 30, 2020 2:26 AM
> To: Fady Bader <fady at mellanox.com>
> Cc: dev at dpdk.org; Tasnim Bashar <tbashar at mellanox.com>; Tal Shnaiderman
> <talshn at mellanox.com>; Yohad Tor <yohadt at mellanox.com>;
> dmitry.kozliuk at gmail.com; harini.ramakrishnan at microsoft.com;
> ocardona at microsoft.com; pallavi.kadam at intel.com; ranjit.menon at intel.com;
> olivier.matz at 6wind.com; arybchenko at solarflare.com; mdr at ashroe.eu;
> nhorman at tuxdriver.com
> Subject: Re: [dpdk-dev] [PATCH v3 1/4] eal: disable function versioning on
> Windows
> 
> 22/06/2020 13:55, Fady Bader:
> > Function versioning is not needed on Windows, also the function
> > versioning implementation is not supported by Windows.
> 
> I think it was said that "not needed" is not right.
> Let's just say that it is not supported for now.

Ok, I'll change.

> 
> > Function versioning was disabled on Windows.
> >
> > Signed-off-by: Fady Bader <fady at mellanox.com>
> > ---
> > --- a/lib/librte_eal/include/rte_function_versioning.h
> > +++ b/lib/librte_eal/include/rte_function_versioning.h
> > +#ifdef RTE_EXEC_ENV_WINDOWS
> > +#undef RTE_BUILD_SHARED_LIB
> > +#endif
> 
> This trick is a bit weird but works.
> A comment is missing to explain that function versioning is disabled but shared
> libraries are allowed for Windows.

Sure, I'll add a comment to explain.

> 
>