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

[Bruce Richardson]

On Wed, Sep 09, 2020 at 07:52:39PM +0000, Dumitrescu, Cristian wrote:
> 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.
> 
Yes, we also have -1 in DPDK for an unspecified NUMA node, e.g. where a PCI
device is just attached to the chipset rather than direct to the CPU
socket.

/Bruce