mbox series

[v1,0/2] bbdev: add device info on queue topology

Message ID 1646785355-168133-1-git-send-email-nicolas.chautru@intel.com (mailing list archive)
Headers
Series bbdev: add device info on queue topology |

Message

Chautru, Nicolas March 9, 2022, 12:22 a.m. UTC
  Addressing an historical concern that the device info struct only
imperfectly captured what queues are available on the device
(number of operation and priority). This ended up being an iterative
process for application to find each queue could be configured.

ie. the gap was captured as technical debt previously  in comments
/* 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
 */

This is now being exposed explictly based on the what the device actually
supports using the existing info_get api

Note: did not update the release yet notes since it will go in next release

Nicolas Chautru (2):
  bbdev: add device info on queue topology
  drivers/baseband: update PMDs to expose queue per operation

 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             |  8 +++++-
 lib/bbdev/rte_bbdev.h                              |  4 +++
 5 files changed, 44 insertions(+), 13 deletions(-)
  

Comments

Chautru, Nicolas June 6, 2022, 4:15 p.m. UTC | #1
Hi Hemant, 
Could you review this serie, this is targeting 22.11 release. Let me know if unclear. 
Thanks
Nic

> -----Original Message-----
> From: Chautru, Nicolas <nicolas.chautru@intel.com>
> Sent: Tuesday, March 8, 2022 4:23 PM
> To: dev@dpdk.org; gakhil@marvell.com
> Cc: trix@redhat.com; hemant.agrawal@nxp.com; Zhang, Mingshan
> <mingshan.zhang@intel.com>; Chautru, Nicolas
> <nicolas.chautru@intel.com>
> Subject: [PATCH v1 0/2] bbdev: add device info on queue topology
> 
> Addressing an historical concern that the device info struct only imperfectly
> captured what queues are available on the device (number of operation and
> priority). This ended up being an iterative process for application to find each
> queue could be configured.
> 
> ie. the gap was captured as technical debt previously  in comments
> /* 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
>  */
> 
> This is now being exposed explictly based on the what the device actually
> supports using the existing info_get api
> 
> Note: did not update the release yet notes since it will go in next release
> 
> Nicolas Chautru (2):
>   bbdev: add device info on queue topology
>   drivers/baseband: update PMDs to expose queue per operation
> 
>  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             |  8 +++++-
>  lib/bbdev/rte_bbdev.h                              |  4 +++
>  5 files changed, 44 insertions(+), 13 deletions(-)
> 
> --
> 1.8.3.1