Re: [PATCH v2 5/6] event/cnxk: support to set runtime queue attributes

[Jerin Jacob <jerinjacobk@gmail.com>]

On Tue, Apr 5, 2022 at 11:12 AM Shijith Thotton <sthotton@marvell.com> wrote:
>
> Added API to set queue attributes at runtime and API to get weight and
> affinity.
>
> Signed-off-by: Shijith Thotton <sthotton@marvell.com>
> ---
>  doc/guides/eventdevs/features/cnxk.ini |  1 +
>  drivers/event/cnxk/cn10k_eventdev.c    |  4 ++
>  drivers/event/cnxk/cn9k_eventdev.c     |  4 ++
>  drivers/event/cnxk/cnxk_eventdev.c     | 91 ++++++++++++++++++++++++--
>  drivers/event/cnxk/cnxk_eventdev.h     | 16 +++++
>  5 files changed, 110 insertions(+), 6 deletions(-)
>
> diff --git a/doc/guides/eventdevs/features/cnxk.ini b/doc/guides/eventdevs/features/cnxk.ini
> index 7633c6e3a2..bee69bf8f4 100644
> --- a/doc/guides/eventdevs/features/cnxk.ini
> +++ b/doc/guides/eventdevs/features/cnxk.ini
> @@ -12,6 +12,7 @@ runtime_port_link          = Y
>  multiple_queue_port        = Y
>  carry_flow_id              = Y
>  maintenance_free           = Y
> +runtime_queue_attr         = y
> +
>         .port_def_conf = cnxk_sso_port_def_conf,
>         .port_setup = cn9k_sso_port_setup,
>         .port_release = cn9k_sso_port_release,
> diff --git a/drivers/event/cnxk/cnxk_eventdev.c b/drivers/event/cnxk/cnxk_eventdev.c
> index be021d86c9..e07cb589f2 100644
> --- a/drivers/event/cnxk/cnxk_eventdev.c
> +++ b/drivers/event/cnxk/cnxk_eventdev.c
> @@ -120,7 +120,8 @@ cnxk_sso_info_get(struct cnxk_sso_evdev *dev,
>                                   RTE_EVENT_DEV_CAP_MULTIPLE_QUEUE_PORT |
>                                   RTE_EVENT_DEV_CAP_NONSEQ_MODE |
>                                   RTE_EVENT_DEV_CAP_CARRY_FLOW_ID |
> -                                 RTE_EVENT_DEV_CAP_MAINTENANCE_FREE;
> +                                 RTE_EVENT_DEV_CAP_MAINTENANCE_FREE |
> +                                 RTE_EVENT_DEV_CAP_RUNTIME_QUEUE_ATTR;

Please swap 6/6 and 5/6 as to avoid the runtime failure at this point.