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

[Elena Agostini <eagostini@nvidia.com>]

[-- Attachment #1: Type: text/plain, Size: 1260 bytes --]

> 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

[-- Attachment #2: Type: text/html, Size: 3658 bytes --]