On Sun, Jan 21, 2018 at 07:50:08PM +0100, Thomas Monjalon wrote: > 13/12/2017 16:17, Neil Horman: > > --- a/lib/librte_eal/linuxapp/eal/Makefile > > +++ b/lib/librte_eal/linuxapp/eal/Makefile > > +CFLAGS += -DALLOW_EXPERIMENTAL_APIS > I think I addressed this in an earlier thread, but I'm opposed to doing this. I'm strongly in favor of granting an exception to any developer who wants to use experimental apis for in-tree code, but I would like for them to have to opt into that decision, rather than just have a blanket allowance. I think theres value in a developer having to make tha conscious decision when writing new code.
> We can define this flag for the whole DPDK libraries, > in mk/rte.vars.mk > inside the block ifneq ($(BUILDING_RTE_SDK),) > > > --- a/mk/internal/rte.compile-pre.mk > > +++ b/mk/internal/rte.compile-pre.mk > > +EXPERIMENTAL_CHECK = $(RTE_SDK)/buildtools/experimentalsyms.sh > > +CHECK_EXPERIMENTAL = $(EXPERIMENTAL_CHECK) $(SRCDIR)/$(EXPORT_MAP) $@ > > > echo $(C_TO_O_DISP); \ > > $(C_TO_O) && \ > > $(PMDINFO_TO_O) && \ > > + $(CHECK_EXPERIMENTAL) && \ > > Inserting this check looks good. > >
