* [PATCH v2 1/1] net/cnxk: fix RSS config via ethdev configure API [not found] <20240410131127.2482240-1-skori@marvell.com> @ 2024-05-23 14:53 ` skori 2024-05-24 13:40 ` Jerin Jacob 0 siblings, 1 reply; 2+ messages in thread From: skori @ 2024-05-23 14:53 UTC (permalink / raw) To: Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori, Satha Rao, Harman Kalra Cc: dev, stable 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. Adding handling for user passed RSS configuration too via rte_eth_dev_configure(). Fixes: b75e0aca84b0 ("net/cnxk: add device configuration operation") Cc: stable@dpdk.org Signed-off-by: Sunil Kumar Kori <skori@marvell.com> --- v1..v2: - added Fixes tag. 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.34.1 ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2 1/1] net/cnxk: fix RSS config via ethdev configure API 2024-05-23 14:53 ` [PATCH v2 1/1] net/cnxk: fix RSS config via ethdev configure API skori @ 2024-05-24 13:40 ` Jerin Jacob 0 siblings, 0 replies; 2+ messages in thread From: Jerin Jacob @ 2024-05-24 13:40 UTC (permalink / raw) To: skori Cc: Nithin Dabilpuram, Kiran Kumar K, Satha Rao, Harman Kalra, dev, stable On Thu, May 23, 2024 at 8:23 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. > > Adding handling for user passed RSS configuration too via > rte_eth_dev_configure(). > > Fixes: b75e0aca84b0 ("net/cnxk: add device configuration operation") > Cc: stable@dpdk.org > > Signed-off-by: Sunil Kumar Kori <skori@marvell.com> > --- > v1..v2: > - added Fixes tag. Applied to dpdk-next-net-mrvl/for-main. Thanks ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-05-24 13:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20240410131127.2482240-1-skori@marvell.com>
2024-05-23 14:53 ` [PATCH v2 1/1] net/cnxk: fix RSS config via ethdev configure API skori
2024-05-24 13:40 ` Jerin Jacob
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).