[dpdk-dev] [PATCH v3 01/41] pipeline: add new SWX pipeline type

Dumitrescu, Cristian cristian.dumitrescu at intel.com
Wed Sep 9 21:52:39 CEST 2020


Hi Stephen,

> -----Original Message-----
> From: Stephen Hemminger <stephen at networkplumber.org>
> Sent: Wednesday, September 9, 2020 8:06 PM
> To: Dumitrescu, Cristian <cristian.dumitrescu at intel.com>
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v3 01/41] pipeline: add new SWX pipeline
> type
> 
> On Tue,  8 Sep 2020 21:17:50 +0100
> Cristian Dumitrescu <cristian.dumitrescu at intel.com> wrote:
> 
> > +/*
> > + * Pipeline.
> > + */
> > +struct rte_swx_pipeline {
> > +	int build_done;
> > +	int numa_node;
> > +};
> > +
> > +
> 
> Is int the right type for these.
> build_done seems like a Boolean

Isn't the difference between int and bool mostly cosmetic?
AFAIK we don't have a hard rule in DPDK about bool vs. int.

> and numa_node should be unsigned?

All the functions in libnuma use int as the numa_node type, please see: man 3 numa.

Regards,
Cristian


More information about the dev mailing list