Re: [PATCH] net/cnxk: add RSS config via ethdev configure API

[Jerin Jacob <jerinjacobk@gmail.com>]

On Wed, Apr 10, 2024 at 6:41 PM <skori@marvell.com> wrote:
>
> From: Sunil Kumar Kori <skori@marvell.com>
>
> Currently user passed RSS configuration is ignored via
> rte_eth_dev_configure() API. Instead default RSS setup is done by
> driver.

Rewrite the git commit as fix and add Fixes: tag


>
> Adding handling for user passed RSS configuration too via
> rte_eth_dev_configure().
>
> Signed-off-by: Sunil Kumar Kori <skori@marvell.com>
> ---
>  drivers/net/cnxk/cnxk_ethdev.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/drivers/net/cnxk/cnxk_ethdev.c b/drivers/net/cnxk/cnxk_ethdev.c
> index 6b37bd877f..95a3d8aaf9 100644
> --- a/drivers/net/cnxk/cnxk_ethdev.c
> +++ b/drivers/net/cnxk/cnxk_ethdev.c
> @@ -1384,6 +1384,13 @@ cnxk_nix_configure(struct rte_eth_dev *eth_dev)
>                 goto free_nix_lf;
>         }
>
> +       /* Overwrite default RSS setup if requested by user */
> +       rc = cnxk_nix_rss_hash_update(eth_dev, &conf->rx_adv_conf.rss_conf);
> +       if (rc) {
> +               plt_err("Failed to configure rss rc=%d", rc);
> +               goto free_nix_lf;
> +       }
> +
>         /* Init the default TM scheduler hierarchy */
>         rc = roc_nix_tm_init(nix);
>         if (rc) {
> --
> 2.25.1
>