Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-15 Thread Wang, Haiyue
che...@oktetlabs.ru; olivier.m...@6wind.com > Subject: Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus > > 15/04/2021 09:55, Xueming(Steven) Li: > > From: Wang, Haiyue > > > From: Xueming(Steven) Li > > > > From: Wang, Haiyue > > >

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-15 Thread Thomas Monjalon
15/04/2021 09:55, Xueming(Steven) Li: > From: Wang, Haiyue > > From: Xueming(Steven) Li > > > From: Wang, Haiyue > > > > From: dev On Behalf Of Xueming Li > > > > > +/** > > > > > + * A structure describing an auxiliary driver. > > > > > + */ > > > > > +struct rte_auxiliary_driver { > > > > > +

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-15 Thread Xueming(Steven) Li
> -Original Message- > From: Wang, Haiyue > Sent: Thursday, April 15, 2021 3:52 PM > To: Xueming(Steven) Li ; NBU-Contact-Thomas Monjalon > > Cc: dev@dpdk.org; Asaf Penso ; Parav Pandit > ; Ray Kinsella ; Neil Horman > > Subject: RE: [dpdk-dev] [PATCH v

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-15 Thread Wang, Haiyue
> -Original Message- > From: Xueming(Steven) Li > Sent: Thursday, April 15, 2021 15:46 > To: Wang, Haiyue ; NBU-Contact-Thomas Monjalon > > Cc: dev@dpdk.org; Asaf Penso ; Parav Pandit > ; Ray Kinsella > ; Neil Horman > Subject: RE: [dpdk-dev] [PATCH v

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-15 Thread Xueming(Steven) Li
> -Original Message- > From: Wang, Haiyue > Sent: Thursday, April 15, 2021 3:36 PM > To: Xueming(Steven) Li ; NBU-Contact-Thomas Monjalon > > Cc: dev@dpdk.org; Asaf Penso ; Parav Pandit > ; Ray Kinsella ; Neil Horman > > Subject: RE: [dpdk-dev] [PATCH v

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-15 Thread Wang, Haiyue
> -Original Message- > From: dev On Behalf Of Xueming Li > Sent: Tuesday, April 13, 2021 11:23 > To: Thomas Monjalon > Cc: dev@dpdk.org; xuemi...@nvidia.com; Asaf Penso ; Parav > Pandit ; > Ray Kinsella ; Neil Horman > Subject: [dpdk-dev] [PATCH v1] bus/auxili

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-14 Thread Wang, Haiyue
> -Original Message- > From: Xueming(Steven) Li > Sent: Wednesday, April 14, 2021 23:39 > To: Wang, Haiyue ; NBU-Contact-Thomas Monjalon > > Cc: dev@dpdk.org; Asaf Penso ; Parav Pandit > ; Ray Kinsella > ; Neil Horman > Subject: RE: [dpdk-dev] [PATCH v

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-14 Thread Xueming(Steven) Li
> -Original Message- > From: Thomas Monjalon > Sent: Wednesday, April 14, 2021 4:18 PM > To: Xueming(Steven) Li ; Wang, Haiyue > > Cc: dev@dpdk.org; Asaf Penso ; Parav Pandit > ; Ray Kinsella > Subject: Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-14 Thread Xueming(Steven) Li
Hi Haiyue, > > + > > +/* > > + * Test whether the auxiliary device exist */ bool > > +auxiliary_exists(const char *name) > > is_auxiliary_support() ? This function test whether an auxiliary device exists, name parameter is device name. I think auxiliary_dev_exists(name) make more sense. Agree

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-14 Thread Wang, Haiyue
> -Original Message- > From: Thomas Monjalon > Sent: Wednesday, April 14, 2021 16:18 > To: Xueming Li ; Wang, Haiyue > Cc: dev@dpdk.org; Asaf Penso ; Parav Pandit > ; Ray Kinsella > > Subject: Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus >

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-14 Thread Thomas Monjalon
14/04/2021 04:59, Wang, Haiyue: > From: Xueming Li [...] > > +void > > +auxiliary_on_scan(struct rte_auxiliary_device *dev) > > +{ > > + struct rte_devargs *devargs; > > + > > + devargs = auxiliary_devargs_lookup(dev->name); > > + dev->device.devargs = devargs; > > Can be simple as: > > dev

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-13 Thread Wang, Haiyue
> -Original Message- > From: dev On Behalf Of Xueming Li > Sent: Tuesday, April 13, 2021 11:23 > To: Thomas Monjalon > Cc: dev@dpdk.org; xuemi...@nvidia.com; Asaf Penso ; Parav > Pandit ; > Ray Kinsella ; Neil Horman > Subject: [dpdk-dev] [PATCH v1] bus/auxili

Re: [dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-13 Thread Thomas Monjalon
I would like to Cc those who are interested in the auxiliary bus, but it's difficult to know who will implement it. Can we expect some reviews? 13/04/2021 05:23, Xueming Li: > Auxiliary [1] provides a way to split function into child-devices > representing sub-domains of functionality. Each auxil

[dpdk-dev] [PATCH v1] bus/auxiliary: introduce auxiliary bus

2021-04-12 Thread Xueming Li
Auxiliary [1] provides a way to split function into child-devices representing sub-domains of functionality. Each auxiliary_device represents a part of its parent functionality. Auxiliary device is identified by unique device name, sysfs path: /sys/bus/auxiliary/devices/ [1] kernel auxiliary bu