Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-15 Thread Thomas Monjalon
15/06/2021 20:24, Ferruh Yigit: > On 6/8/2021 7:34 AM, Thomas Monjalon wrote: > > 08/06/2021 06:10, Jerin Jacob: > >> On Mon, Jun 7, 2021 at 10:17 PM Thomas Monjalon > >> wrote: > >>> > >>> 07/06/2021 15:54, Jerin Jacob: > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon > wrote: >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-15 Thread Ferruh Yigit
On 6/8/2021 7:34 AM, Thomas Monjalon wrote: > 08/06/2021 06:10, Jerin Jacob: >> On Mon, Jun 7, 2021 at 10:17 PM Thomas Monjalon wrote: >>> >>> 07/06/2021 15:54, Jerin Jacob: On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon wrote: > 07/06/2021 09:20, Wang, Haiyue: >> From: Honnappa Naga

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-08 Thread Thomas Monjalon
08/06/2021 09:09, Jerin Jacob: > On Tue, Jun 8, 2021 at 12:05 PM Thomas Monjalon wrote: > > > > 08/06/2021 06:10, Jerin Jacob: > > > On Mon, Jun 7, 2021 at 10:17 PM Thomas Monjalon > > > wrote: > > > > > > > > 07/06/2021 15:54, Jerin Jacob: > > > > > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalo

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-08 Thread Jerin Jacob
On Tue, Jun 8, 2021 at 12:05 PM Thomas Monjalon wrote: > > 08/06/2021 06:10, Jerin Jacob: > > On Mon, Jun 7, 2021 at 10:17 PM Thomas Monjalon wrote: > > > > > > 07/06/2021 15:54, Jerin Jacob: > > > > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon > > > > wrote: > > > > > 07/06/2021 09:20, Wang,

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Thomas Monjalon
08/06/2021 06:10, Jerin Jacob: > On Mon, Jun 7, 2021 at 10:17 PM Thomas Monjalon wrote: > > > > 07/06/2021 15:54, Jerin Jacob: > > > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon > > > wrote: > > > > 07/06/2021 09:20, Wang, Haiyue: > > > > > From: Honnappa Nagarahalli > > > > > > If we keep CX

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Jerin Jacob
On Mon, Jun 7, 2021 at 10:17 PM Thomas Monjalon wrote: > > 07/06/2021 15:54, Jerin Jacob: > > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon wrote: > > > 07/06/2021 09:20, Wang, Haiyue: > > > > From: Honnappa Nagarahalli > > > > > If we keep CXL in mind, I would imagine that in the future the >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Honnappa Nagarahalli
> -Original Message- > From: Jerin Jacob > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon > wrote: > > > > 07/06/2021 09:20, Wang, Haiyue: > > > From: Honnappa Nagarahalli > > > > If we keep CXL in mind, I would imagine that in the future the > > > > devices on PCIe could have their ow

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Thomas Monjalon
07/06/2021 15:54, Jerin Jacob: > On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon wrote: > > 07/06/2021 09:20, Wang, Haiyue: > > > From: Honnappa Nagarahalli > > > > If we keep CXL in mind, I would imagine that in the future the devices > > > > on PCIe could have their own > > > > local memory. Ma

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Jerin Jacob
On Mon, Jun 7, 2021 at 4:13 PM Thomas Monjalon wrote: > > 07/06/2021 09:20, Wang, Haiyue: > > From: Honnappa Nagarahalli > > > If we keep CXL in mind, I would imagine that in the future the devices on > > > PCIe could have their own > > > local memory. May be some of the APIs could use generic n

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Thomas Monjalon
06/06/2021 03:10, Honnappa Nagarahalli: > > The new library gpudev is for dealing with GPU from a DPDK application in a > > vendor-agnostic way. > > It would be good to explain how the application using GPU+DPDK would look > like. It can be anything to invent. We can give few more words saying i

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Thomas Monjalon
07/06/2021 09:20, Wang, Haiyue: > From: Honnappa Nagarahalli > > If we keep CXL in mind, I would imagine that in the future the devices on > > PCIe could have their own > > local memory. May be some of the APIs could use generic names. For ex: > > instead of calling it as > > "rte_gpu_malloc" ma

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Thomas Monjalon
06/06/2021 07:28, Jerin Jacob: > On Sun, Jun 6, 2021 at 6:44 AM Honnappa Nagarahalli > > This patch does not provide the big picture view of what the processing > > looks like using GPU. It would be good to explain that. > > For ex: > > 1) Will the notion of GPU hidden from the application? i.e. i

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-07 Thread Wang, Haiyue
halli > ; nd > Subject: RE: [dpdk-dev] [PATCH] gpudev: introduce memory API > > > > > > > > > > > 04/06/2021 17:20, Jerin Jacob: > > > > > On Fri, Jun 4, 2021 at 7:39 PM Thomas Monjalon > > wrote: > > > > > > 04/06/2021 15

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-05 Thread Jerin Jacob
On Sun, Jun 6, 2021 at 6:44 AM Honnappa Nagarahalli wrote: > > > > > > > > > > > 04/06/2021 17:20, Jerin Jacob: > > > > > On Fri, Jun 4, 2021 at 7:39 PM Thomas Monjalon > > wrote: > > > > > > 04/06/2021 15:59, Andrew Rybchenko: > > > > > > > On 6/4/21 4:18 PM, Thomas Monjalon wrote: > > > > > >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-05 Thread Honnappa Nagarahalli
> > > > > > 04/06/2021 17:20, Jerin Jacob: > > > > On Fri, Jun 4, 2021 at 7:39 PM Thomas Monjalon > wrote: > > > > > 04/06/2021 15:59, Andrew Rybchenko: > > > > > > On 6/4/21 4:18 PM, Thomas Monjalon wrote: > > > > > > > 04/06/2021 15:05, Andrew Rybchenko: > > > > > > >> On 6/4/21 3:46 PM, Thoma

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-05 Thread Honnappa Nagarahalli
> > From: Elena Agostini > > The new library gpudev is for dealing with GPU from a DPDK application in a > vendor-agnostic way. It would be good to explain how the application using GPU+DPDK would look like. Which parts of the workload need DPDK's support? Any requirements on co-existence of

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-05 Thread Wang, Haiyue
> -Original Message- > From: Thomas Monjalon > Sent: Saturday, June 5, 2021 15:49 > To: Wang, Haiyue > Cc: dev@dpdk.org; Elena Agostini ; > andrew.rybche...@oktetlabs.ru; Yigit, Ferruh > ; jer...@marvell.com > Subject: Re: [dpdk-dev] [PATCH] gpudev: introduce me

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-05 Thread Thomas Monjalon
04/06/2021 20:04, Wang, Haiyue: > From: Thomas Monjalon > > 04/06/2021 15:25, Wang, Haiyue: > > > From: Thomas Monjalon > > > > Another question is about the function rte_gpu_free(). > > > > How do we recognize that a memory chunk is from the CPU and GPU visible, > > > > or just from GPU? > > > >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Jerin Jacob
uh ; > > dpdk-dev ; > > Elena Agostini ; David Marchand > > > > Subject: Re: [dpdk-dev] [PATCH] gpudev: introduce memory API > > > > 04/06/2021 17:20, Jerin Jacob: > > > On Fri, Jun 4, 2021 at 7:39 PM Thomas Monjalon > > > wrote: > > > &g

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Wang, Haiyue
> -Original Message- > From: dev On Behalf Of Thomas Monjalon > Sent: Friday, June 4, 2021 23:51 > To: Jerin Jacob > Cc: Honnappa Nagarahalli ; Andrew Rybchenko > ; Yigit, Ferruh ; > dpdk-dev ; > Elena Agostini ; David Marchand > > Subject: Re: [dpdk-

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Wang, Haiyue
> -Original Message- > From: Thomas Monjalon > Sent: Friday, June 4, 2021 22:06 > To: Wang, Haiyue > Cc: dev@dpdk.org; Elena Agostini ; > andrew.rybche...@oktetlabs.ru; Yigit, Ferruh > ; jer...@marvell.com > Subject: Re: [dpdk-dev] [PATCH] gpudev: introduce memor

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 17:20, Jerin Jacob: > On Fri, Jun 4, 2021 at 7:39 PM Thomas Monjalon wrote: > > 04/06/2021 15:59, Andrew Rybchenko: > > > On 6/4/21 4:18 PM, Thomas Monjalon wrote: > > > > 04/06/2021 15:05, Andrew Rybchenko: > > > >> On 6/4/21 3:46 PM, Thomas Monjalon wrote: > > > >>> 04/06/2021 13:09,

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Jerin Jacob
On Fri, Jun 4, 2021 at 7:39 PM Thomas Monjalon wrote: > > 04/06/2021 15:59, Andrew Rybchenko: > > On 6/4/21 4:18 PM, Thomas Monjalon wrote: > > > 04/06/2021 15:05, Andrew Rybchenko: > > >> On 6/4/21 3:46 PM, Thomas Monjalon wrote: > > >>> 04/06/2021 13:09, Jerin Jacob: > > On Fri, Jun 4, 2021

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Jerin Jacob
On Fri, Jun 4, 2021 at 6:25 PM Thomas Monjalon wrote: > > 03/06/2021 13:38, Jerin Jacob: > > On Thu, Jun 3, 2021 at 4:00 PM Thomas Monjalon wrote: > > > 03/06/2021 12:04, Jerin Jacob: > > > > On Thu, Jun 3, 2021 at 3:06 PM Thomas Monjalon > > > > wrote: > > > > > 03/06/2021 11:20, Jerin Jacob:

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 15:59, Andrew Rybchenko: > On 6/4/21 4:18 PM, Thomas Monjalon wrote: > > 04/06/2021 15:05, Andrew Rybchenko: > >> On 6/4/21 3:46 PM, Thomas Monjalon wrote: > >>> 04/06/2021 13:09, Jerin Jacob: > On Fri, Jun 4, 2021 at 3:58 PM Thomas Monjalon > wrote: > > 03/06/2021 11:33,

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 15:25, Wang, Haiyue: > From: Thomas Monjalon > > Another question is about the function rte_gpu_free(). > > How do we recognize that a memory chunk is from the CPU and GPU visible, > > or just from GPU? > > > > I didn't find the rte_gpu_free_visible definition, and the rte_gpu_free's

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Andrew Rybchenko
On 6/4/21 4:18 PM, Thomas Monjalon wrote: > 04/06/2021 15:05, Andrew Rybchenko: >> On 6/4/21 3:46 PM, Thomas Monjalon wrote: >>> 04/06/2021 13:09, Jerin Jacob: On Fri, Jun 4, 2021 at 3:58 PM Thomas Monjalon wrote: > 03/06/2021 11:33, Ferruh Yigit: >> On 6/3/2021 8:47 AM, Jerin Jacob w

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Wang, Haiyue
> -Original Message- > From: Thomas Monjalon > Sent: Friday, June 4, 2021 20:44 > To: Wang, Haiyue > Cc: dev@dpdk.org; Elena Agostini > Subject: Re: [dpdk-dev] [PATCH] gpudev: introduce memory API > > 04/06/2021 13:07, Wang, Haiyue: > > > From: E

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 15:05, Andrew Rybchenko: > On 6/4/21 3:46 PM, Thomas Monjalon wrote: > > 04/06/2021 13:09, Jerin Jacob: > >> On Fri, Jun 4, 2021 at 3:58 PM Thomas Monjalon wrote: > >>> 03/06/2021 11:33, Ferruh Yigit: > On 6/3/2021 8:47 AM, Jerin Jacob wrote: > > On Thu, Jun 3, 2021 at 2:05 AM

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Andrew Rybchenko
On 6/4/21 3:46 PM, Thomas Monjalon wrote: > 04/06/2021 13:09, Jerin Jacob: >> On Fri, Jun 4, 2021 at 3:58 PM Thomas Monjalon wrote: >>> 03/06/2021 11:33, Ferruh Yigit: On 6/3/2021 8:47 AM, Jerin Jacob wrote: > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon > wrote: >> + [gpudev

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
03/06/2021 13:38, Jerin Jacob: > On Thu, Jun 3, 2021 at 4:00 PM Thomas Monjalon wrote: > > 03/06/2021 12:04, Jerin Jacob: > > > On Thu, Jun 3, 2021 at 3:06 PM Thomas Monjalon > > > wrote: > > > > 03/06/2021 11:20, Jerin Jacob: > > > > > The device needs have a queue kind of structure > > > > > a

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 13:09, Jerin Jacob: > On Fri, Jun 4, 2021 at 3:58 PM Thomas Monjalon wrote: > > 03/06/2021 11:33, Ferruh Yigit: > > > On 6/3/2021 8:47 AM, Jerin Jacob wrote: > > > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon > > > > wrote: > > > >> + [gpudev] (@ref rte_gpudev.h), > >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 13:07, Wang, Haiyue: > > From: Elena Agostini > > +typedef int (*gpu_malloc_t)(struct rte_gpu_dev *dev, size_t size, void > > **ptr); > > +typedef int (*gpu_free_t)(struct rte_gpu_dev *dev, void *ptr); > > + [...] > > + /* FUNCTION: Allocate memory on the GPU. */ > > + gpu_malloc_t

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Jerin Jacob
On Fri, Jun 4, 2021 at 3:58 PM Thomas Monjalon wrote: > > 03/06/2021 11:33, Ferruh Yigit: > > On 6/3/2021 8:47 AM, Jerin Jacob wrote: > > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon > > > wrote: > > >> + [gpudev] (@ref rte_gpudev.h), > > > > > > Since this device does not have

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Wang, Haiyue
> -Original Message- > From: dev On Behalf Of Thomas Monjalon > Sent: Thursday, June 3, 2021 04:36 > To: dev@dpdk.org > Cc: Elena Agostini > Subject: [dpdk-dev] [PATCH] gpudev: introduce memory API > > From: Elena Agostini > > The new library gpudev is fo

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
03/06/2021 11:33, Ferruh Yigit: > On 6/3/2021 8:47 AM, Jerin Jacob wrote: > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon wrote: > >> + [gpudev] (@ref rte_gpudev.h), > > > > Since this device does not have a queue etc? Shouldn't make it a > > library like mempool with vendor-defin

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-04 Thread Thomas Monjalon
04/06/2021 07:51, Wang, Haiyue: > > From: Elena Agostini > > > > The new library gpudev is for dealing with GPU from a DPDK application > > in a vendor-agnostic way. > > > > As a first step, the features are focused on memory management. > > A function allows to allocate memory inside the GPU, >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Wang, Haiyue
> -Original Message- > From: dev On Behalf Of Thomas Monjalon > Sent: Thursday, June 3, 2021 04:36 > To: dev@dpdk.org > Cc: Elena Agostini > Subject: [dpdk-dev] [PATCH] gpudev: introduce memory API > > From: Elena Agostini > > The new library gpudev is fo

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Jerin Jacob
On Thu, Jun 3, 2021 at 4:00 PM Thomas Monjalon wrote: > > 03/06/2021 12:04, Jerin Jacob: > > On Thu, Jun 3, 2021 at 3:06 PM Thomas Monjalon wrote: > > > > > > 03/06/2021 11:20, Jerin Jacob: > > > > On Thu, Jun 3, 2021 at 2:23 PM Thomas Monjalon > > > > wrote: > > > > > > > > > > 03/06/2021 10:4

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 12:04, Jerin Jacob: > On Thu, Jun 3, 2021 at 3:06 PM Thomas Monjalon wrote: > > > > 03/06/2021 11:20, Jerin Jacob: > > > On Thu, Jun 3, 2021 at 2:23 PM Thomas Monjalon > > > wrote: > > > > > > > > 03/06/2021 10:47, Jerin Jacob: > > > > > On Thu, Jun 3, 2021 at 2:13 PM Thomas Monjalon

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Jerin Jacob
On Thu, Jun 3, 2021 at 3:06 PM Thomas Monjalon wrote: > > 03/06/2021 11:20, Jerin Jacob: > > On Thu, Jun 3, 2021 at 2:23 PM Thomas Monjalon wrote: > > > > > > 03/06/2021 10:47, Jerin Jacob: > > > > On Thu, Jun 3, 2021 at 2:13 PM Thomas Monjalon > > > > wrote: > > > > > > > > > > 03/06/2021 10:4

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 11:20, Jerin Jacob: > On Thu, Jun 3, 2021 at 2:23 PM Thomas Monjalon wrote: > > > > 03/06/2021 10:47, Jerin Jacob: > > > On Thu, Jun 3, 2021 at 2:13 PM Thomas Monjalon > > > wrote: > > > > > > > > 03/06/2021 10:41, Jerin Jacob: > > > > > On Thu, Jun 3, 2021 at 1:58 PM Thomas Monjalon

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Ferruh Yigit
On 6/3/2021 8:47 AM, Jerin Jacob wrote: > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon wrote: >> >> From: Elena Agostini >> >> The new library gpudev is for dealing with GPU from a DPDK application >> in a vendor-agnostic way. >> >> As a first step, the features are focused on memory management

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Jerin Jacob
On Thu, Jun 3, 2021 at 2:23 PM Thomas Monjalon wrote: > > 03/06/2021 10:47, Jerin Jacob: > > On Thu, Jun 3, 2021 at 2:13 PM Thomas Monjalon wrote: > > > > > > 03/06/2021 10:41, Jerin Jacob: > > > > On Thu, Jun 3, 2021 at 1:58 PM Thomas Monjalon > > > > wrote: > > > > > > > > > > 03/06/2021 09:4

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Andrew Rybchenko
On 6/3/21 11:26 AM, Thomas Monjalon wrote: > 03/06/2021 09:49, Andrew Rybchenko: >> On 6/3/21 10:26 AM, Thomas Monjalon wrote: >>> 03/06/2021 09:06, Andrew Rybchenko: On 6/2/21 11:35 PM, Thomas Monjalon wrote: > + * Allocate a chunk of memory on the GPU. Looking a below function

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 10:47, Jerin Jacob: > On Thu, Jun 3, 2021 at 2:13 PM Thomas Monjalon wrote: > > > > 03/06/2021 10:41, Jerin Jacob: > > > On Thu, Jun 3, 2021 at 1:58 PM Thomas Monjalon > > > wrote: > > > > > > > > 03/06/2021 09:47, Jerin Jacob: > > > > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Jerin Jacob
On Thu, Jun 3, 2021 at 2:13 PM Thomas Monjalon wrote: > > 03/06/2021 10:41, Jerin Jacob: > > On Thu, Jun 3, 2021 at 1:58 PM Thomas Monjalon wrote: > > > > > > 03/06/2021 09:47, Jerin Jacob: > > > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon > > > > wrote: > > > > > --- a/doc/api/doxy-api-in

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 10:41, Jerin Jacob: > On Thu, Jun 3, 2021 at 1:58 PM Thomas Monjalon wrote: > > > > 03/06/2021 09:47, Jerin Jacob: > > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon > > > wrote: > > > > --- a/doc/api/doxy-api-index.md > > > > +++ b/doc/api/doxy-api-index.md > > > > @@ -21,6 +21,7 @

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Jerin Jacob
On Thu, Jun 3, 2021 at 1:58 PM Thomas Monjalon wrote: > > 03/06/2021 09:47, Jerin Jacob: > > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon wrote: > > > --- a/doc/api/doxy-api-index.md > > > +++ b/doc/api/doxy-api-index.md > > > @@ -21,6 +21,7 @@ The public API headers are grouped by topics: > >

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 09:47, Jerin Jacob: > On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon wrote: > > --- a/doc/api/doxy-api-index.md > > +++ b/doc/api/doxy-api-index.md > > @@ -21,6 +21,7 @@ The public API headers are grouped by topics: > >[compressdev](@ref rte_compressdev.h), > >[compress]

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 09:49, Andrew Rybchenko: > On 6/3/21 10:26 AM, Thomas Monjalon wrote: > > 03/06/2021 09:06, Andrew Rybchenko: > >> On 6/2/21 11:35 PM, Thomas Monjalon wrote: > >>> + * Allocate a chunk of memory on the GPU. > >> > >> Looking a below function it is required to clarify here if > >> the mem

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Andrew Rybchenko
On 6/3/21 10:26 AM, Thomas Monjalon wrote: > 03/06/2021 09:06, Andrew Rybchenko: >> On 6/2/21 11:35 PM, Thomas Monjalon wrote: >>> + * Allocate a chunk of memory on the GPU. >> >> Looking a below function it is required to clarify here if >> the memory is visible or invisible to GPU (or both allowe

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Jerin Jacob
On Thu, Jun 3, 2021 at 2:05 AM Thomas Monjalon wrote: > > From: Elena Agostini > > The new library gpudev is for dealing with GPU from a DPDK application > in a vendor-agnostic way. > > As a first step, the features are focused on memory management. > A function allows to allocate memory inside t

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 09:18, David Marchand: > Quick pass: > > On Wed, Jun 2, 2021 at 10:36 PM Thomas Monjalon wrote: > > diff --git a/lib/gpudev/gpu_driver.h b/lib/gpudev/gpu_driver.h > > new file mode 100644 > > index 00..5ff609e49d > > --- /dev/null > > +++ b/lib/gpudev/gpu_driver.h [...] > > +st

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Thomas Monjalon
03/06/2021 09:06, Andrew Rybchenko: > On 6/2/21 11:35 PM, Thomas Monjalon wrote: > > From: Elena Agostini > > > > The new library gpudev is for dealing with GPU from a DPDK application > > in a vendor-agnostic way. > > > > As a first step, the features are focused on memory management. > > A fun

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread David Marchand
Quick pass: On Wed, Jun 2, 2021 at 10:36 PM Thomas Monjalon wrote: > diff --git a/lib/gpudev/gpu_driver.h b/lib/gpudev/gpu_driver.h > new file mode 100644 > index 00..5ff609e49d > --- /dev/null > +++ b/lib/gpudev/gpu_driver.h > @@ -0,0 +1,44 @@ > +/* SPDX-License-Identifier: BSD-3-Clause

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-03 Thread Andrew Rybchenko
On 6/2/21 11:35 PM, Thomas Monjalon wrote: > From: Elena Agostini > > The new library gpudev is for dealing with GPU from a DPDK application > in a vendor-agnostic way. > > As a first step, the features are focused on memory management. > A function allows to allocate memory inside the GPU, > wh

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-02 Thread Thomas Monjalon
02/06/2021 22:46, Stephen Hemminger: > On Wed, 2 Jun 2021 22:35:31 +0200 > Thomas Monjalon wrote: > > > +/** Store a list of info for a given GPU. */ > > +struct rte_gpu_info { > > + /** GPU device ID. */ > > + uint16_t gpu_id; > > + /** Unique identifier name. */ > > + char name[RTE_GPU

Re: [dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-02 Thread Stephen Hemminger
On Wed, 2 Jun 2021 22:35:31 +0200 Thomas Monjalon wrote: > +/** Store a list of info for a given GPU. */ > +struct rte_gpu_info { > + /** GPU device ID. */ > + uint16_t gpu_id; > + /** Unique identifier name. */ > + char name[RTE_GPU_NAME_MAX_LEN]; > + /** Total memory availa

[dpdk-dev] [PATCH] gpudev: introduce memory API

2021-06-02 Thread Thomas Monjalon
From: Elena Agostini The new library gpudev is for dealing with GPU from a DPDK application in a vendor-agnostic way. As a first step, the features are focused on memory management. A function allows to allocate memory inside the GPU, while another one allows to use main (CPU) memory from the GP