RE: [EXTERNAL] Re: [PATCH v6 7/8] app: reduce app dependencies

["Morten Brørup" <mb@smartsharesystems.com>]

> From: Akhil Goyal [mailto:gakhil@marvell.com]
> Sent: Wednesday, 19 February 2025 09.16
> 
> Hi David,
> > Hello,
> >
> > Cc: Akhil.
> >
> > On Tue, Feb 18, 2025 at 12:16 PM Anatoly Burakov
> > <anatoly.burakov@intel.com> wrote:
> > > diff --git a/app/test-crypto-perf/meson.build b/app/test-crypto-
> > perf/meson.build
> > > index fb48d9ec29..f056431bf4 100644
> > > --- a/app/test-crypto-perf/meson.build
> > > +++ b/app/test-crypto-perf/meson.build
> > > @@ -19,7 +19,7 @@ sources = files(
> > >          'cperf_test_verify.c',
> > >          'main.c',
> > >  )
> > > -deps += ['cryptodev', 'net', 'security']
> > > +deps += ['cryptodev']
> > >  optional_deps += 'crypto_scheduler'
> > >
> > >  cflags += no_wvla_cflag
> >
> > - This app does seem to rely on lib/net.
> >
> > [989/1266] Compiling C object
> > app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o
> > FAILED: app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o
> > ccache aarch64-linux-gnu-gcc -Iapp/dpdk-test-crypto-perf.p -Iapp
> > -I../app -Ilib/cryptodev -I../lib/cryptodev -Ilib/mbuf -I../lib/mbuf
> > -Ilib/mempool -I../lib/mempool -Ilib/ring -I../lib/ring -I. -I..
> > -Iconfig -I../config -Ilib/eal/include -I../lib/eal/include
> > -Ilib/eal/linux/include -I../lib/eal/linux/include
> > -Ilib/eal/arm/include -I../lib/eal/arm/include -I../kernel/linux
> > -Ilib/eal/common -I../lib/eal/common -Ilib/eal -I../lib/eal
> > -Ilib/kvargs -I../lib/kvargs -Ilib/log -I../lib/log -Ilib/metrics
> > -I../lib/metrics -Ilib/telemetry -I../lib/telemetry -Ilib/rcu
> > -I../lib/rcu -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64
> > -Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include
> rte_config.h
> > -Wvla -Wcast-qual -Wdeprecated -Wformat -Wformat-nonliteral
> > -Wformat-security -Wmissing-declarations -Wmissing-prototypes
> > -Wnested-externs -Wold-style-definition -Wpointer-arith -Wsign-
> compare
> > -Wstrict-prototypes -Wundef -Wwrite-strings -Wno-packed-not-aligned
> > -Wno-missing-field-initializers -D_GNU_SOURCE -mcpu=cortex-a72
> > -DALLOW_EXPERIMENTAL_API -Wno-vla -MD -MQ
> > app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o -MF
> > app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o.d -o
> > app/dpdk-test-crypto-perf.p/test-crypto-perf_cperf_ops.c.o -c
> > ../app/test-crypto-perf/cperf_ops.c
> > ../app/test-crypto-perf/cperf_ops.c:6:10: fatal error: rte_ether.h:
> No
> > such file or directory
> >     6 | #include <rte_ether.h>
> >       |          ^~~~~~~~~~~~~
> > compilation terminated.
> > ninja: build stopped: subcommand failed.
> >
> >
> > - Btw, rte_security.h is included in app/test-crypto-
> perf/cperf_ops.c.
> > However, given there is a #ifdef, I suspect the intention was to have
> > lib/security as an optional dep.
> 
> Yes it was initially meant to be optional when rte_security was
> experimental.
> But now, it is kind of mandatory library. I am not sure if it will
> compile without rte_security.
> We can remove the ifdefs over rte_security and
> we should not remove security dependency from crypto-perf.
> 
> -Akhil

Not sure what you mean here, but:
The "security" library is optional, not always enabled, when building DPDK.
Ref: /source/lib/meson.build

Not all applications use the security library (or stuff that depends on it), so don't make it mandatory!

-Morten