Re: [PATCH] app/test: invoke all telemetry commands

[David Marchand <david.marchand@redhat.com>]

On Fri, Jul 29, 2022 at 12:20 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
> > +if not is_windows and dpdk_conf.has('RTE_LIB_TELEMETRY')
> > +    test_args = [dpdk_test]
> > +    test_args += test_no_huge_args
> > +    if get_option('default_library') == 'shared'
> > +        foreach drv:dpdk_drivers
> > +            test_args += ['-d', drv.full_path().split('.a')[0] + '.so']
> > +        endforeach
>
> Rather than looping for each driver and building a huge cmdline, we should
> just be able to pass in the path to the drivers directory, and have DPDK
> auto-load all .so files there. Passing in "meson.project_build_root() +
> '/drivers'" should probably work. If we want a more correct way to get the
> drivers build directory, we can call meson.current_build_dir() when
> processing the drivers and save off that value to be reused here.

I copied this loop from an existing block added with 207b1c813f39
("test: fix build without ring PMD").
On the need for this loop at the time... I think it was to avoid
loading non .so stuff from a build directory.
And you probably handled this with 49b536fc3060 ("eal: load only
shared libs from driver plugin directories").

I'll try your suggestion and, if it works, I'll update the other existing test.


-- 
David Marchand