event/dlb2: fix missing queue ordering capability flag

Message ID 20231109174420.788106-1-bruce.richardson@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Jerin Jacob
Headers
Series event/dlb2: fix missing queue ordering capability flag |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/github-robot: build success github build: passed
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS

Commit Message

Bruce Richardson Nov. 9, 2023, 5:44 p.m. UTC
  The dlb2 driver did not advertise the fact that events could be enqueued
to it for any queues, not just those in numerical sequence. Add the
missing bit to the capabilities flag returned from the info_get()
function.

Fixes: d39e23f26e1e ("event/dlb2: fix advertized capabilities")
Fixes: e7c9971a857a ("event/dlb2: add probe-time hardware init")
Cc: stable@dpdk.org

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/event/dlb2/dlb2.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Jerin Jacob Nov. 10, 2023, 7:25 a.m. UTC | #1
On Thu, Nov 9, 2023 at 11:14 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> The dlb2 driver did not advertise the fact that events could be enqueued
> to it for any queues, not just those in numerical sequence. Add the
> missing bit to the capabilities flag returned from the info_get()
> function.
>
> Fixes: d39e23f26e1e ("event/dlb2: fix advertized capabilities")
> Fixes: e7c9971a857a ("event/dlb2: add probe-time hardware init")
> Cc: stable@dpdk.org
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Waiting for Ack/Review from @Abdullah Sevincer   to merge.


> ---
>  drivers/event/dlb2/dlb2.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c
> index e645f7595a..050ace0904 100644
> --- a/drivers/event/dlb2/dlb2.c
> +++ b/drivers/event/dlb2/dlb2.c
> @@ -72,6 +72,7 @@ static struct rte_event_dev_info evdev_dlb2_default_info = {
>         .max_single_link_event_port_queue_pairs =
>                 DLB2_MAX_NUM_DIR_PORTS(DLB2_HW_V2),
>         .event_dev_cap = (RTE_EVENT_DEV_CAP_EVENT_QOS |
> +                         RTE_EVENT_DEV_CAP_NONSEQ_MODE |
>                           RTE_EVENT_DEV_CAP_DISTRIBUTED_SCHED |
>                           RTE_EVENT_DEV_CAP_QUEUE_ALL_TYPES |
>                           RTE_EVENT_DEV_CAP_BURST_MODE |
> --
> 2.39.2
>
  
Sevincer, Abdullah Nov. 12, 2023, 8:35 p.m. UTC | #2
Acked-by: Abdullah Sevincer <abdullah.sevincer@intel.com>
  
Jerin Jacob Nov. 13, 2023, 5:36 a.m. UTC | #3
On Mon, Nov 13, 2023 at 2:05 AM Sevincer, Abdullah
<abdullah.sevincer@intel.com> wrote:
>
> Acked-by: Abdullah Sevincer <abdullah.sevincer@intel.com>

Applied to dpdk-next-eventdev/for-main. Thanks
  

Patch

diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c
index e645f7595a..050ace0904 100644
--- a/drivers/event/dlb2/dlb2.c
+++ b/drivers/event/dlb2/dlb2.c
@@ -72,6 +72,7 @@  static struct rte_event_dev_info evdev_dlb2_default_info = {
 	.max_single_link_event_port_queue_pairs =
 		DLB2_MAX_NUM_DIR_PORTS(DLB2_HW_V2),
 	.event_dev_cap = (RTE_EVENT_DEV_CAP_EVENT_QOS |
+			  RTE_EVENT_DEV_CAP_NONSEQ_MODE |
 			  RTE_EVENT_DEV_CAP_DISTRIBUTED_SCHED |
 			  RTE_EVENT_DEV_CAP_QUEUE_ALL_TYPES |
 			  RTE_EVENT_DEV_CAP_BURST_MODE |