Device queue initialization

PATCH Applied Hernan Vargas 3 participants 5 messages Aug 23, 2024 – Sep 13, 2024

lore

Share & tools

lore

Hernan Vargas

author

2024-08-23 15:01 UTC #0 lore raw

v2 cover Device queue initialization

Reworded commit message for cosmetic queue initialization change.

Hernan Vargas (1):
drivers/baseband: device queues initialization

 drivers/baseband/acc/rte_acc100_pmd.c              | 1 +
 drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++
 drivers/baseband/fpga_lte_fec/fpga_lte_fec.c       | 2 ++
 drivers/baseband/la12xx/bbdev_la12xx.c             | 2 ++
 4 files changed, 7 insertions(+)

Signature · 1 line

2.37.1

Hernan Vargas

author

2024-08-23 15:01 UTC #1 lore raw

PATCH v2 1/1 drivers/baseband: device queues initialization

Cosmetic change to keep code consistency in PMD for queue initialization set to zero.
No functional impact.

Signed-off-by Hernan Vargas <[email protected]>

---
 drivers/baseband/acc/rte_acc100_pmd.c              | 1 +
 drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++
 drivers/baseband/fpga_lte_fec/fpga_lte_fec.c       | 2 ++
 drivers/baseband/la12xx/bbdev_la12xx.c             | 2 ++
 4 files changed, 7 insertions(+)

drivers/baseband/acc/rte_acc100_pmd.c +1 -0

index ab69350080f0..9b12fa7048c2 100644
@@ -1030,6 +1030,7 @@ acc100_dev_info_get(struct rte_bbdev *dev,
 	dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = d->acc_conf.q_dl_5g.num_aqs_per_groups *
 			d->acc_conf.q_dl_5g.num_qgroups;
 	dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
 	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = d->acc_conf.q_ul_4g.num_qgroups;
 	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = d->acc_conf.q_dl_4g.num_qgroups;
 	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = d->acc_conf.q_ul_5g.num_qgroups;

drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c +2 -0

index 9b253cde280d..dad16ad422b5 100644
@@ -625,6 +625,8 @@ fpga_5gnr_dev_info_get(struct rte_bbdev *dev, struct rte_bbdev_driver_info *dev_
 	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->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
 	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1;
 	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1;
 }

drivers/baseband/fpga_lte_fec/fpga_lte_fec.c +2 -0

index a4a963f74dbf..c6a3c3805a04 100644
@@ -659,6 +659,8 @@ fpga_dev_info_get(struct rte_bbdev *dev,
 	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->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
 	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = 1;
 	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = 1;
 }

drivers/baseband/la12xx/bbdev_la12xx.c +2 -0

index 1a56e73abdab..c13eabbf4b6e 100644
@@ -108,6 +108,8 @@ la12xx_info_get(struct rte_bbdev *dev __rte_unused,
 	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->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
 	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);

Signature · 1 line

2.37.1

Hemant Agrawal

Acked-by on v2 1/1

2024-08-26 05:14 UTC #2 lore raw

Re: [PATCH v2 1/1] drivers/baseband: device queues initialization

Quoted 55 lines · Hernan Vargas

Cosmetic change to keep code consistency in PMD for queue initialization set to zero.
No functional impact.

Signed-off-by Hernan Vargas <[email protected]>

---
  drivers/baseband/acc/rte_acc100_pmd.c              | 1 +
  drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++
  drivers/baseband/fpga_lte_fec/fpga_lte_fec.c       | 2 ++
  drivers/baseband/la12xx/bbdev_la12xx.c             | 2 ++
  4 files changed, 7 insertions(+)

drivers/baseband/acc/rte_acc100_pmd.c +1 -0

index ab69350080f0..9b12fa7048c2 100644
@@ -1030,6 +1030,7 @@ acc100_dev_info_get(struct rte_bbdev *dev,
  	dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = d->acc_conf.q_dl_5g.num_aqs_per_groups *
  			d->acc_conf.q_dl_5g.num_qgroups;
  	dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
  	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = d->acc_conf.q_ul_4g.num_qgroups;
  	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = d->acc_conf.q_dl_4g.num_qgroups;
  	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = d->acc_conf.q_ul_5g.num_qgroups;

drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c +2 -0

index 9b253cde280d..dad16ad422b5 100644
@@ -625,6 +625,8 @@ fpga_5gnr_dev_info_get(struct rte_bbdev *dev, struct rte_bbdev_driver_info *dev_
  	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->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
  	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1;
  	dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1;
  }

drivers/baseband/fpga_lte_fec/fpga_lte_fec.c +2 -0

index a4a963f74dbf..c6a3c3805a04 100644
@@ -659,6 +659,8 @@ fpga_dev_info_get(struct rte_bbdev *dev,
  	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->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
  	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = 1;
  	dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = 1;
  }

drivers/baseband/la12xx/bbdev_la12xx.c +0 -0

index 1a56e73abdab..c13eabbf4b6e 100644

drivers/baseband/la12xx/bbdev_la12xx.c la12xx_info_get(struct rte_bbdev *dev __rte_unused, · line 115

Quoted 8 lines

drivers/baseband/la12xx/bbdev_la12xx.c +2 -0

@@ -108,6 +108,8 @@ la12xx_info_get(struct rte_bbdev *dev __rte_unused,
  	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->num_queues[RTE_BBDEV_OP_FFT] = 0;
+	dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0;
  	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);

Acked-by Hemant Agrawal <[email protected]>

Maxime Coquelin

Reviewed-by on v2 1/1

2024-09-10 15:35 UTC #3 lore raw

Re: [PATCH v2 1/1] drivers/baseband: device queues initialization

In reply to Hernan Vargas

Quoted 4 lines · Hernan Vargas

Cosmetic change to keep code consistency in PMD for queue initialization set to zero.
No functional impact.

Quoted 8 lines

Signed-off-by Hernan Vargas <[email protected]>

---
  drivers/baseband/acc/rte_acc100_pmd.c              | 1 +
  drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++
  drivers/baseband/fpga_lte_fec/fpga_lte_fec.c       | 2 ++
  drivers/baseband/la12xx/bbdev_la12xx.c             | 2 ++
  4 files changed, 7 insertions(+)

Reviewed-by Maxime Coquelin <[email protected]>

Thanks,
Maxime

Maxime Coquelin

Applied

2024-09-13 13:50 UTC #4 lore raw

Re: [PATCH v2 1/1] drivers/baseband: device queues initialization

In reply to Hernan Vargas

Quoted 4 lines · Hernan Vargas

Cosmetic change to keep code consistency in PMD for queue initialization set to zero.
No functional impact.

Quoted 8 lines

Signed-off-by Hernan Vargas <[email protected]>

---
  drivers/baseband/acc/rte_acc100_pmd.c              | 1 +
  drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++
  drivers/baseband/fpga_lte_fec/fpga_lte_fec.c       | 2 ++
  drivers/baseband/la12xx/bbdev_la12xx.c             | 2 ++
  4 files changed, 7 insertions(+)

Applied to dpdk-next-baseband/for-main

Thanks,
Maxime