Re: [dpdk-dev] [PATCH v3 3/3] test/devargs: add devargs test cases

[Bruce Richardson <bruce.richardson@intel.com>]

On Wed, Oct 20, 2021 at 09:40:55AM +0000, Xueming(Steven) Li wrote:
> On Wed, 2021-10-20 at 11:08 +0200, David Marchand wrote:
> > On Wed, Oct 20, 2021 at 10:22 AM Xueming Li <xuemingl@nvidia.com> wrote:
> > > 
> > > Initial version to test Global devargs syntax.
> > > 
> > > Signed-off-by: Xueming Li <xuemingl@nvidia.com>
> > 
> > 10/95 DPDK:fast-tests / devargs_autotest      FAIL     0.17 s (exit
> > status 255 or signal 127 SIGinvalid)
> > 
> > --- command ---
> > DPDK_TEST='devargs_autotest'
> > /home/runner/work/dpdk/dpdk/build/app/test/dpdk-test -l 0-1
> > --file-prefix=devargs_autotest
> > --- stdout ---
> > RTE>>devargs_autotest
> > == test valid case ==
> > rte_devargs_parse(net_virtio_user0,iface=test,path=/dev/vhost-net,queues=1)
> > returned -14 (but should not)
> > rte_devargs_parse(net_virtio_user0,iface=test,path=/class/bus/,queues=1)
> > returned -14 (but should not)
> > Test Failed
> > RTE>>
> > --- stderr ---
> > EAL: Detected CPU lcores: 2
> > EAL: Detected NUMA nodes: 1
> > EAL: Detected shared linkage of DPDK
> > EAL: Multi-process socket /var/run/dpdk/devargs_autotest/mp_socket
> > EAL: Selected IOVA mode 'PA'
> > EAL: No available 1048576 kB hugepages reported
> > EAL: VFIO support initialized
> > APP: HPET is not enabled, using TSC as default timer
> > EAL: failed to parse device "net_virtio_user0"
> > EAL: failed to parse device "net_virtio_user0"
> 
> Yes, noticed that, seems virtio driver not enabled. Tried to add
> "net_virtio" to test_deps in meson file, but failed with:
> ../app/test/meson.build:444:4: ERROR: Tried to get unknown variable
> "static_rte_net_virtio".
> 
> Seems meson scripts only lookup dependency on libs, I'm not good at
> meson, any suggestion? 
>
"net_virtio" is correct for the virtio driver. However, you probably need
to do like is done for the other network drivers linked into the autotest,
and make it conditional on the support being present, since not every build
enables the virtio driver. For example, as is currently in the meson.build
for the autotests:

# The following linkages of drivers are required because
# they are used via a driver-specific API.
if dpdk_conf.has('RTE_NET_BOND')
    test_deps += 'net_bond'
    test_sources += ['test_link_bonding.c', 'test_link_bonding_rssconf.c']
    ...
endif

/Bruce