Hi David, > -----Original Message----- > From: David Marchand <david.march...@redhat.com> > Sent: Friday, April 17, 2020 2:23 PM > To: Trahe, Fiona <fiona.tr...@intel.com> > Cc: dev <dev@dpdk.org>; Jerin Jacob <jerinjac...@gmail.com>; Pavan Nikhilesh > <pbhagavat...@marvell.com>; Richardson, Bruce <bruce.richard...@intel.com>; > Thomas Monjalon > <tho...@monjalon.net>; Yigit, Ferruh <ferruh.yi...@intel.com>; Hemant Agrawal > <hemant.agra...@nxp.com> > Subject: Re: [dpdk-dev] [PATCH v5] build: disable experimental API check > internally > > On Fri, Apr 17, 2020 at 12:21 PM Trahe, Fiona <fiona.tr...@intel.com> wrote: > > I see this is already applied. > > > > However, > > rte_cryptodev_queue_pair_setup() calls > > rte_cryptodev_sym_get_existing_header_session_size() > > The former is a stable API, the latter is experimental. > > So I expect the build to break when ALLOW_EXPERIMENTAL_API is disabled. [Fiona] Thanks for confirming where the flag is. But I think you've missed my point. What about this problem? How was this tested with the flag disabled?
> > I tried to build the crypto_perf tool to check this, wasn't clear where it > > gets it, seems like it inherits > from here: > > mk/target/generic/rte.vars.mk. So I disabled it there and got many build > > errors. > > I suppose you mean dpdk-test-crypto-perf. > This tool is compiled inside dpdk, so the flag is inherited from > rte.vars.mk as intended. > > > -- > David Marchand