DPDK patches and discussions
 help / color / mirror / Atom feed
From: "Chautru, Nicolas" <nicolas.chautru@intel.com>
To: Tom Rix <trix@redhat.com>, "dev@dpdk.org" <dev@dpdk.org>,
	"thomas@monjalon.net" <thomas@monjalon.net>,
	"gakhil@marvell.com" <gakhil@marvell.com>,
	"hemant.agrawal@nxp.com" <hemant.agrawal@nxp.com>
Cc: "maxime.coquelin@redhat.com" <maxime.coquelin@redhat.com>,
	"mdr@ashroe.eu" <mdr@ashroe.eu>,
	"Richardson, Bruce" <bruce.richardson@intel.com>,
	"david.marchand@redhat.com" <david.marchand@redhat.com>,
	"stephen@networkplumber.org" <stephen@networkplumber.org>
Subject: RE: [PATCH v4 4/7] drivers/baseband: update PMDs to expose queue per operation
Date: Mon, 15 Aug 2022 17:28:50 +0000	[thread overview]
Message-ID: <BY5PR11MB4451B32179333F13E5AF4D0EF8689@BY5PR11MB4451.namprd11.prod.outlook.com> (raw)
In-Reply-To: <934b8642-c52d-c1f1-6de4-728e110ab26b@redhat.com>

Hi Tom, 

Back from time off, replying to that previous email.

> -----Original Message-----
> From: Tom Rix <trix@redhat.com>
> Sent: Monday, July 18, 2022 6:21 AM
> To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org;
> thomas@monjalon.net; gakhil@marvell.com; hemant.agrawal@nxp.com
> Cc: maxime.coquelin@redhat.com; mdr@ashroe.eu; Richardson, Bruce
> <bruce.richardson@intel.com>; david.marchand@redhat.com;
> stephen@networkplumber.org
> Subject: Re: [PATCH v4 4/7] drivers/baseband: update PMDs to expose queue
> per operation
> 
> 
> On 7/7/22 10:19 AM, Chautru, Nicolas wrote:
> > Hi Tom,
> >
> >> -----Original Message-----
> >> From: Tom Rix <trix@redhat.com>
> >> Sent: Thursday, July 7, 2022 6:21 AM
> >> To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org;
> >> thomas@monjalon.net; gakhil@marvell.com; hemant.agrawal@nxp.com
> >> Cc: maxime.coquelin@redhat.com; mdr@ashroe.eu; Richardson, Bruce
> >> <bruce.richardson@intel.com>; david.marchand@redhat.com;
> >> stephen@networkplumber.org
> >> Subject: Re: [PATCH v4 4/7] drivers/baseband: update PMDs to expose
> >> queue per operation
> >>
> >>
> >> On 7/6/22 2:10 PM, Chautru, Nicolas wrote:
> >>> Hi Tom,
> >>>
> >>>> -----Original Message-----
> >>>> From: Tom Rix <trix@redhat.com>
> >>>> Sent: Wednesday, July 6, 2022 9:15 AM
> >>>> To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org;
> >>>> thomas@monjalon.net; gakhil@marvell.com;
> hemant.agrawal@nxp.com
> >>>> Cc: maxime.coquelin@redhat.com; mdr@ashroe.eu; Richardson, Bruce
> >>>> <bruce.richardson@intel.com>; david.marchand@redhat.com;
> >>>> stephen@networkplumber.org
> >>>> Subject: Re: [PATCH v4 4/7] drivers/baseband: update PMDs to expose
> >>>> queue per operation
> >>>>
> >>>>
> >>>> On 7/5/22 5:23 PM, Nicolas Chautru wrote:
> >>>>> Add support in existing bbdev PMDs for the explicit number of
> >>>>> queue and priority for each operation type configured on the device.
> >>>>>
> >>>>> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
> >>>>> ---
> >>>>>     drivers/baseband/acc100/rte_acc100_pmd.c           | 29
> +++++++++++++--
> >> ----
> >>>> ---
> >>>>>     drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c |  8 ++++++
> >>>>>     drivers/baseband/fpga_lte_fec/fpga_lte_fec.c       |  8 ++++++
> >>>>>     drivers/baseband/la12xx/bbdev_la12xx.c             |  7 ++++++
> >>>>>     drivers/baseband/turbo_sw/bbdev_turbo_software.c   | 11
> ++++++++
> >>>>>     5 files changed, 51 insertions(+), 12 deletions(-)
> >>>>>
> >>>>> diff --git a/drivers/baseband/acc100/rte_acc100_pmd.c
> >>>>> b/drivers/baseband/acc100/rte_acc100_pmd.c
> >>>>> index 17ba798..d568d0d 100644
> >>>>> --- a/drivers/baseband/acc100/rte_acc100_pmd.c
> >>>>> +++ b/drivers/baseband/acc100/rte_acc100_pmd.c
> >>>>> @@ -966,6 +966,7 @@
> >>>>>     		struct rte_bbdev_driver_info *dev_info)
> >>>>>     {
> >>>>>     	struct acc100_device *d = dev->data->dev_private;
> >>>>> +	int i;
> >>>>>
> >>>>>     	static const struct rte_bbdev_op_cap bbdev_capabilities[] = {
> >>>>>     		{
> >>>>> @@ -1062,19 +1063,23 @@
> >>>>>     	fetch_acc100_config(dev);
> >>>>>     	dev_info->device_status =
> RTE_BBDEV_DEV_NOT_SUPPORTED;
> >>>>>
> >>>>> -	/* This isn't ideal because it reports the maximum number of queues
> >>>> but
> >>>>> -	 * does not provide info on how many can be uplink/downlink or
> >>>> different
> >>>>> -	 * priorities
> >>>>> -	 */
> >>>>> -	dev_info->max_num_queues =
> >>>>> -			d->acc100_conf.q_dl_5g.num_aqs_per_groups *
> >>>>> -			d->acc100_conf.q_dl_5g.num_qgroups +
> >>>>> -			d->acc100_conf.q_ul_5g.num_aqs_per_groups *
> >>>>> -			d->acc100_conf.q_ul_5g.num_qgroups +
> >>>>> -			d->acc100_conf.q_dl_4g.num_aqs_per_groups *
> >>>>> -			d->acc100_conf.q_dl_4g.num_qgroups +
> >>>>> -			d->acc100_conf.q_ul_4g.num_aqs_per_groups *
> >>>>> +	/* Expose number of queues */
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_NONE] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_DEC] =
> >>>>> +d->acc100_conf.q_ul_4g.num_aqs_per_groups *
> >>>>>     			d->acc100_conf.q_ul_4g.num_qgroups;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = d-
> >>>>> acc100_conf.q_dl_4g.num_aqs_per_groups *
> >>>>> +			d->acc100_conf.q_dl_4g.num_qgroups;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = d-
> >>>>> acc100_conf.q_ul_5g.num_aqs_per_groups *
> >>>>> +			d->acc100_conf.q_ul_5g.num_qgroups;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = d-
> >>>>> acc100_conf.q_dl_5g.num_aqs_per_groups *
> >>>>> +			d->acc100_conf.q_dl_5g.num_qgroups;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] =
> d-
> >>>>> acc100_conf.q_ul_4g.num_qgroups;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] =
> d-
> >>>>> acc100_conf.q_dl_4g.num_qgroups;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = d-
> >>>>> acc100_conf.q_ul_5g.num_qgroups;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = d-
> >>>>> acc100_conf.q_dl_5g.num_qgroups;
> >>>>> +	dev_info->max_num_queues = 0;
> >>>>> +	for (i = RTE_BBDEV_OP_TURBO_DEC; i <
> RTE_BBDEV_OP_LDPC_ENC;
> >>>> i++)
> >>>>
> >>>> should this be i <=  ?
> >>>>
> >>> Thanks
> >>>
> >>>>> +		dev_info->max_num_queues += dev_info-
> >num_queues[i];
> >>>>>     	dev_info->queue_size_lim = ACC100_MAX_QUEUE_DEPTH;
> >>>>>     	dev_info->hardware_accelerated = true;
> >>>>>     	dev_info->max_dl_queue_priority = diff --git
> >>>>> a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c
> >>>>> b/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c
> >>>>> index 57b12af..b4982af 100644
> >>>>> --- a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c
> >>>>> +++ b/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c
> >>>>> @@ -379,6 +379,14 @@
> >>>>>     		if (hw_q_id != FPGA_INVALID_HW_QUEUE_ID)
> >>>>>     			dev_info->max_num_queues++;
> >>>>>     	}
> >>>>> +	/* Expose number of queue per operation type */
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_NONE] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_DEC] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] =
> dev_info-
> >>>>> max_num_queues / 2;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] =
> dev_info-
> >>>>> max_num_queues / 2;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1;
> >>>>>     }
> >>>>>
> >>>>>     /**
> >>>>> diff --git a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
> >>>>> b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
> >>>>> index 2a330c4..dc7f479 100644
> >>>>> --- a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
> >>>>> +++ b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
> >>>>> @@ -655,6 +655,14 @@ struct __rte_cache_aligned fpga_queue {
> >>>>>     		if (hw_q_id != FPGA_INVALID_HW_QUEUE_ID)
> >>>>>     			dev_info->max_num_queues++;
> >>>>>     	}
> >>>>> +	/* Expose number of queue per operation type */
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_NONE] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_DEC] =
> dev_info-
> >>>>> max_num_queues / 2;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] =
> dev_info-
> >>>>> max_num_queues / 2;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = 0;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = 1;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] =
> 1;
> >>>>>     }
> >>>>>
> >>>>>     /**
> >>>>> diff --git a/drivers/baseband/la12xx/bbdev_la12xx.c
> >>>>> b/drivers/baseband/la12xx/bbdev_la12xx.c
> >>>>> index c1f88c6..e99ea9a 100644
> >>>>> --- a/drivers/baseband/la12xx/bbdev_la12xx.c
> >>>>> +++ b/drivers/baseband/la12xx/bbdev_la12xx.c
> >>>>> @@ -102,6 +102,13 @@ struct bbdev_la12xx_params {
> >>>>>     	dev_info->min_alignment = 64;
> >>>>>     	dev_info->device_status =
> RTE_BBDEV_DEV_NOT_SUPPORTED;
> >>>>>
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_NONE] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_DEC] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = 0;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] =
> >>>> LA12XX_MAX_QUEUES / 2;
> >>>>> +	dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] =
> >>>> LA12XX_MAX_QUEUES / 2;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1;
> >>>>> +	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1;
> >>>>>     	rte_bbdev_log_debug("got device info from %u", dev->data-
> >dev_id);
> >>>>>     }
> >>>>>
> >>>>> diff --git a/drivers/baseband/turbo_sw/bbdev_turbo_software.c
> >>>>> b/drivers/baseband/turbo_sw/bbdev_turbo_software.c
> >>>>> index dbc5524..647e706 100644
> >>>>> --- a/drivers/baseband/turbo_sw/bbdev_turbo_software.c
> >>>>> +++ b/drivers/baseband/turbo_sw/bbdev_turbo_software.c
> >>>>> @@ -256,6 +256,17 @@ struct turbo_sw_queue {
> >>>>>     	dev_info->data_endianness = RTE_LITTLE_ENDIAN;
> >>>>>     	dev_info->device_status =
> RTE_BBDEV_DEV_NOT_SUPPORTED;
> >>>>>
> >>>>> +	const struct rte_bbdev_op_cap *op_cap =
> bbdev_capabilities;
> >>>> Should this be done through dev instead of assigning directly ?
> >>> I am not sure I follow your suggestion. Do you mind clarifying?
> >> bbdev_capabilites is a const defined in this function, do you really
> >> need to loop over it to find information that is constant ?
> > I still miss your point. Note that this constant is not always the same at
> build time (based on what SDK it can links to).
> > What would suggest?
> 
> Operations that can be done at compile time, should be.  Useless there is a
> good reason.
> 
> You need to provide a good reason or make the change.

I believe this is more graceful, scalable and maintainable this way. 
At build time we already define the list of capability then we just process that information without risk of disconnect between two pieces of code. 
The drawback is execution time but this function is not time sensititive (enumeration).
I could change it but the code would be poorer with risk of breaking stuff in the future (redundant information in the code hence bug prone).
Ie. defining the number of operations and queues using another serie of  #define. 
From my point of view that would be something that we would not do internally for the reasons above, but if you insist I will just change accordingly so that to move on. 
Not a big deal, let us know. 

Thanks
Nic

> 
> Tom
> 
> >
> > Thanks
> > Nic
> >
> >
> >> Tom
> >>
> >>>> Tom
> >>>>
> >>>>> +	int num_op_type = 0;
> >>>>> +	for (; op_cap->type != RTE_BBDEV_OP_NONE; ++op_cap)
> >>>>> +		num_op_type++;
> >>>>> +	op_cap = bbdev_capabilities;
> >>>>> +	if (num_op_type > 0) {
> >>>>> +		int num_queue_per_type = dev_info-
> >max_num_queues /
> >>>> num_op_type;
> >>>>> +		for (; op_cap->type != RTE_BBDEV_OP_NONE;
> ++op_cap)
> >>>>> +			dev_info->num_queues[op_cap->type] =
> >>>> num_queue_per_type;
> >>>>> +	}
> >>>>> +
> >>>>>     	rte_bbdev_log_debug("got device info from %u\n", dev-
> >data-
> >>>>> dev_id);
> >>>>>     }
> >>>>>


  reply	other threads:[~2022-08-15 17:29 UTC|newest]

Thread overview: 174+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-09  0:22 [PATCH v1 0/2] bbdev: add device info on queue topology Nicolas Chautru
2022-03-09  0:22 ` [PATCH v1 1/2] " Nicolas Chautru
2022-03-09  1:28   ` Stephen Hemminger
2022-03-09  0:22 ` [PATCH v1 2/2] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-06-17 18:37   ` [PATCH v2 0/5] bbdev changes for 22.11 Nicolas Chautru
2022-06-17 18:37     ` [PATCH v2 1/5] bbdev: allow operation type enum for growth Nicolas Chautru
2022-06-17 18:37     ` [PATCH v2 2/5] bbdev: add device status info Nicolas Chautru
2022-06-17 18:37     ` [PATCH v2 3/5] bbdev: add device info on queue topology Nicolas Chautru
2022-06-17 18:37     ` [PATCH v2 4/5] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-06-17 18:37     ` [PATCH v2 5/5] bbdev: add new operation for FFT processing Nicolas Chautru
2022-06-28  1:35       ` [PATCH v3 0/7] bbdev changes for 22.11 Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 2/7] bbdev: add device status info Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-06-28  1:35         ` [PATCH v3 7/7] bbdev: add a lock option for enqueue/dequeue operation Nicolas Chautru
2022-07-06  0:23       ` [PATCH v4 0/7] bbdev changes for 22.11 Nicolas Chautru
2022-07-06  0:23         ` [PATCH v4 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-07-06 12:50           ` Tom Rix
2022-07-06 21:20             ` Chautru, Nicolas
2022-07-06  0:23         ` [PATCH v4 2/7] bbdev: add device status info Nicolas Chautru
2022-07-06 15:38           ` Tom Rix
2022-07-06 21:16             ` Chautru, Nicolas
2022-07-07 13:37               ` Tom Rix
2022-07-07 17:15                 ` Chautru, Nicolas
2022-07-18 13:09                   ` Tom Rix
2022-08-25 14:08               ` Maxime Coquelin
2022-07-06  0:23         ` [PATCH v4 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-07-06 16:06           ` Tom Rix
2022-07-06 21:12             ` Chautru, Nicolas
2022-07-07 13:34               ` Tom Rix
2022-07-07 17:13                 ` Chautru, Nicolas
2022-07-18 13:04                   ` Tom Rix
2022-07-06  0:23         ` [PATCH v4 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-07-06 16:15           ` Tom Rix
2022-07-06 21:10             ` Chautru, Nicolas
2022-07-07 13:20               ` Tom Rix
2022-07-07 17:19                 ` Chautru, Nicolas
2022-07-18 13:21                   ` Tom Rix
2022-08-15 17:28                     ` Chautru, Nicolas [this message]
2022-07-06  0:23         ` [PATCH v4 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-07-06 18:47           ` Tom Rix
2022-07-06 21:04             ` Chautru, Nicolas
2022-07-07 13:09               ` Tom Rix
2022-07-07 16:57                 ` Chautru, Nicolas
2022-07-18 22:38                   ` Tom Rix
2022-07-06  0:23         ` [PATCH v4 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-07-06 18:57           ` Tom Rix
2022-07-06 20:34             ` Chautru, Nicolas
2022-07-06  0:23         ` [PATCH v4 7/7] bbdev: add a lock option for enqueue/dequeue operation Nicolas Chautru
2022-07-06 19:01           ` Tom Rix
2022-07-06 19:20             ` Stephen Hemminger
2022-07-06 20:21               ` Chautru, Nicolas
2022-07-07 12:47                 ` Tom Rix
2022-07-06 23:28       ` [PATCH v5 0/7] bbdev changes for 22.11 Nicolas Chautru
2022-07-06 23:28         ` [PATCH v5 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-08-25 13:54           ` Maxime Coquelin
2022-07-06 23:28         ` [PATCH v5 2/7] bbdev: add device status info Nicolas Chautru
2022-08-25 14:18           ` Maxime Coquelin
2022-08-25 18:30             ` Chautru, Nicolas
2022-08-26 10:12               ` Maxime Coquelin
2022-08-29 16:10                 ` Chautru, Nicolas
2022-08-30  7:08                   ` Maxime Coquelin
2022-08-30 19:38                     ` Chautru, Nicolas
2022-07-06 23:28         ` [PATCH v5 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-08-25 15:23           ` Maxime Coquelin
2022-07-06 23:28         ` [PATCH v5 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-07-06 23:28         ` [PATCH v5 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-07-06 23:28         ` [PATCH v5 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-07-06 23:28         ` [PATCH v5 7/7] bbdev: remove unnecessary if-check Nicolas Chautru
2022-08-15 17:54         ` [PATCH v5 0/7] bbdev changes for 22.11 Chautru, Nicolas
2022-08-25 18:24       ` [PATCH v6 " Nicolas Chautru
2022-08-25 18:24         ` [PATCH v6 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-08-25 18:24         ` [PATCH v6 2/7] bbdev: add device status info Nicolas Chautru
2022-08-25 18:24         ` [PATCH v6 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-08-25 18:24         ` [PATCH v6 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-08-26 11:53           ` Maxime Coquelin
2022-08-25 18:24         ` [PATCH v6 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-08-26 12:07           ` Maxime Coquelin
2022-08-29 18:18             ` Chautru, Nicolas
2022-08-25 18:24         ` [PATCH v6 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-08-26 19:51           ` Maxime Coquelin
2022-08-25 18:24         ` [PATCH v6 7/7] bbdev: remove unnecessary if-check Nicolas Chautru
2022-08-26 19:52           ` Maxime Coquelin
2022-08-29 18:07       ` [PATCH v7 0/7] bbdev changes for 22.11 Nicolas Chautru
2022-08-29 18:07         ` [PATCH v7 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-08-29 18:07         ` [PATCH v7 2/7] bbdev: add device status info Nicolas Chautru
2022-08-30  2:19           ` Zhang, Mingshan
2022-08-30  4:43           ` Hemant Agrawal
2022-09-21 18:54           ` [EXT] " Akhil Goyal
2022-09-21 20:53             ` Chautru, Nicolas
2022-08-29 18:07         ` [PATCH v7 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-08-29 18:07         ` [PATCH v7 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-08-30  4:44           ` Hemant Agrawal
2022-09-21 19:00           ` [EXT] " Akhil Goyal
2022-09-21 20:53             ` Chautru, Nicolas
2022-08-29 18:07         ` [PATCH v7 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-09-21 19:14           ` [EXT] " Akhil Goyal
2022-09-21 20:56             ` Chautru, Nicolas
2022-09-22 14:19               ` Akhil Goyal
2022-09-22 16:39                 ` Chautru, Nicolas
2022-09-22 16:48                   ` Akhil Goyal
2022-09-22 17:25                     ` Chautru, Nicolas
2022-08-29 18:07         ` [PATCH v7 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-09-21 19:21           ` [EXT] " Akhil Goyal
2022-09-21 20:57             ` Chautru, Nicolas
2022-09-23 10:57             ` Ferruh Yigit
     [not found]               ` <CO6PR18MB44848717BA4EA2FF8967D7CBD8509@CO6PR18MB4484.namprd18.prod.outlook.com>
2022-09-24 16:34                 ` Chautru, Nicolas
2022-09-27  9:43                   ` Ferruh Yigit
2022-09-27 20:59                   ` Chautru, Nicolas
2022-09-29 18:10                     ` Ferruh Yigit
2022-09-29 18:32                       ` Akhil Goyal
2022-09-29 19:48                         ` Chautru, Nicolas
2022-09-30  7:54                           ` Maxime Coquelin
2022-08-29 18:07         ` [PATCH v7 7/7] bbdev: remove unnecessary if-check Nicolas Chautru
2022-09-21 19:25           ` [EXT] " Akhil Goyal
2022-09-21 20:58             ` Chautru, Nicolas
2022-08-30  4:45         ` [PATCH v7 0/7] bbdev changes for 22.11 Hemant Agrawal
2022-09-06 16:47         ` Chautru, Nicolas
2022-09-21 21:02       ` [PATCH v8 " Nic Chautru
2022-09-21 21:02         ` [PATCH v8 1/7] bbdev: allow operation type enum for growth Nic Chautru
2022-09-21 21:02         ` [PATCH v8 2/7] bbdev: add device status info Nic Chautru
2022-09-21 21:02         ` [PATCH v8 3/7] bbdev: add device info on queue topology Nic Chautru
2022-09-21 21:02         ` [PATCH v8 4/7] drivers/baseband: update PMDs to expose queue per operation Nic Chautru
2022-09-21 21:02         ` [PATCH v8 5/7] bbdev: add new operation for FFT processing Nic Chautru
2022-09-21 21:02         ` [PATCH v8 6/7] bbdev: add queue related warning and status information Nic Chautru
2022-09-21 21:02         ` [PATCH v8 7/7] bbdev: remove unnecessary if-check Nic Chautru
2022-09-22 17:45       ` [PATCH v9 0/7] bbdev changes for 22.11 Nic Chautru
2022-09-22 17:45         ` [PATCH v9 1/7] bbdev: allow operation type enum for growth Nic Chautru
2022-09-22 17:45         ` [PATCH v9 2/7] bbdev: add device status info Nic Chautru
2022-09-22 17:45         ` [PATCH v9 3/7] bbdev: add device info on queue topology Nic Chautru
2022-09-22 17:45         ` [PATCH v9 4/7] drivers/baseband: update PMDs to expose queue per operation Nic Chautru
2022-09-22 17:45         ` [PATCH v9 5/7] bbdev: add new operation for FFT processing Nic Chautru
2022-09-22 17:45         ` [PATCH v9 6/7] bbdev: add queue related warning and status information Nic Chautru
2022-09-22 17:45         ` [PATCH v9 7/7] bbdev: remove unnecessary if-check Nic Chautru
2022-09-22 18:17         ` [EXT] [PATCH v9 0/7] bbdev changes for 22.11 Akhil Goyal
2022-09-22 20:59           ` Chautru, Nicolas
2022-09-30 18:45       ` [PATCH v10 " Nicolas Chautru
2022-09-30 18:45         ` [PATCH v10 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-09-30 18:46         ` [PATCH v10 2/7] bbdev: add device status info Nicolas Chautru
2022-09-30 18:46         ` [PATCH v10 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-09-30 18:46         ` [PATCH v10 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-09-30 18:46         ` [PATCH v10 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-09-30 18:46         ` [PATCH v10 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-10-03  8:28           ` Thomas Monjalon
2022-10-03 16:39             ` Chautru, Nicolas
2022-10-03 17:21               ` Thomas Monjalon
2022-09-30 18:46         ` [PATCH v10 7/7] bbdev: remove unnecessary if-check Nicolas Chautru
2022-09-30 20:38         ` [EXT] [PATCH v10 0/7] bbdev changes for 22.11 Akhil Goyal
2022-10-03 18:00       ` [PATCH v11 " Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 2/7] bbdev: add device status info Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-10-03 18:00         ` [PATCH v11 7/7] bbdev: remove unnecessary if-check Nicolas Chautru
2022-10-04 17:16       ` [PATCH v12 0/7] bbdev changes for 22.11 Nicolas Chautru
2022-10-04 17:16         ` [PATCH v12 1/7] bbdev: allow operation type enum for growth Nicolas Chautru
2022-10-04 17:16         ` [PATCH v12 2/7] bbdev: add device status info Nicolas Chautru
2022-10-05  7:16           ` Maxime Coquelin
2022-10-04 17:16         ` [PATCH v12 3/7] bbdev: add device info on queue topology Nicolas Chautru
2022-10-04 17:16         ` [PATCH v12 4/7] drivers/baseband: update PMDs to expose queue per operation Nicolas Chautru
2022-10-04 17:16         ` [PATCH v12 5/7] bbdev: add new operation for FFT processing Nicolas Chautru
2022-10-04 17:16         ` [PATCH v12 6/7] bbdev: add queue related warning and status information Nicolas Chautru
2022-10-04 17:16         ` [PATCH v12 7/7] bbdev: remove unnecessary if-check Nicolas Chautru
2022-10-06 17:31         ` [EXT] [PATCH v12 0/7] bbdev changes for 22.11 Akhil Goyal
2022-10-06 22:28           ` Chautru, Nicolas
2022-10-07  4:46             ` Akhil Goyal
2022-10-10  7:35           ` Thomas Monjalon
2022-10-10 17:07             ` Chautru, Nicolas
2022-06-06 16:15 ` [PATCH v1 0/2] bbdev: add device info on queue topology Chautru, Nicolas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BY5PR11MB4451B32179333F13E5AF4D0EF8689@BY5PR11MB4451.namprd11.prod.outlook.com \
    --to=nicolas.chautru@intel.com \
    --cc=bruce.richardson@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=gakhil@marvell.com \
    --cc=hemant.agrawal@nxp.com \
    --cc=maxime.coquelin@redhat.com \
    --cc=mdr@ashroe.eu \
    --cc=stephen@networkplumber.org \
    --cc=thomas@monjalon.net \
    --cc=trix@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).