> From: Ferruh Yigit <ferruh.yigit@intel.com>

> Date: Tuesday, 16 November 2021 at 19:00

> To: Elena Agostini <eagostini@nvidia.com>

> Cc: dev@dpdk.org <dev@dpdk.org>, Slava Ovsiienko <viacheslavo@nvidia.com>

> Subject: Re: [PATCH v2 1/1] app/testpmd: add GPU memory option in iofwd engine

> External email: Use caution opening links or attachments>

>

 

> On 11/11/2021 9:41 PM, eagostini@nvidia.com wrote:

> > --- a/app/test-pmd/meson.build

> > +++ b/app/test-pmd/meson.build

> > @@ -32,7 +32,7 @@ if dpdk_conf.has('RTE_HAS_JANSSON')

> >       ext_deps += jansson_dep

> >   endif

> >

> > -deps += ['ethdev', 'gro', 'gso', 'cmdline', 'metrics', 'bus_pci']

> > +deps += ['ethdev', 'gro', 'gso', 'cmdline', 'metrics', 'bus_pci', 'gpudev']>

 

> I didn't review the set, but in a very high level do we want to add

> 'gpudev' as dependency? Isn't this like adding 'rawdev' as dependency.

 

gpudev is a library that can be built without a gpu driver as all the other libraries

and it is actually used only in case of GPU memory mempool.

 

Reasons for this patch are:

 

- Have an upstreamed benchmark tool to measure network metrics using GPU memory

- Test some DPDK features not really tested anywhere like the external memory mempool feature