On Mon, 8 Nov 2021 18:39:36 +0000
Elena Agostini <eagost...@nvidia.com> wrote:
> > From: Stephen Hemminger <step...@networkplumber.org>
> > Date: Monday, 8 November 2021 at 19:35
> > To: Elena Agostini <eagost...@nvidia.com>
> > Cc: dev@dpdk.org <dev@dpdk.org>
> > Subject: Re: [dpdk-dev] [PATCH v2 1/1] gpu/cuda: introduce CUDA driver
> > External email: Use caution opening links or attachments>
> >
> > On Thu, 4 Nov 2021 02:01:28 +0000
> > <eagost...@nvidia.com> wrote:>
> > > +/* Single entry of the memory list */
> > > +struct mem_entry {
> > > + CUdeviceptr ptr_d;
> > > + void *ptr_h;
> > > + size_t size;
> > > + struct rte_gpu *dev;
> > > + CUcontext ctx;>
> > Not sure where these types CUdeviceptr and CUcontext are coming
> > from, but the code looks like Windows style not DPDK or Linux.>
> > Please don't introduce CamelCase typedef's and never typedefs
> > for pointers.
>
> These are CUDA Driver API specific types, I can’t change them.
Could you at least avoid using the pointer typedefs.
When a pointer is typedef'd it leads to confusion to readers and
some dumb static analyzers.
