[dpdk-dev] [PATCH] net/fm10k: remove RSS restriction with num of queues

[Zhang, Qi Z]

Hi:

> -----Original Message-----
> From: zhouyangchao [mailto:zhouyates at gmail.com]
> Sent: Wednesday, November 29, 2017 2:12 PM
> To: dev at dpdk.org
> Cc: qi.z.zhang at intel.comqi.z.zhang@intel.com
> Subject: [PATCH] net/fm10k: remove RSS restriction with num of queues
> 
> FM10K HW does not have such restrictions.
> 
> Signed-off-by: Yangchao Zhou <zhouyates at gmail.com>
> ---
>  drivers/net/fm10k/fm10k_ethdev.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/fm10k/fm10k_ethdev.c
> b/drivers/net/fm10k/fm10k_ethdev.c
> index 2d05a46..403f6c5 100644
> --- a/drivers/net/fm10k/fm10k_ethdev.c
> +++ b/drivers/net/fm10k/fm10k_ethdev.c
> @@ -533,9 +533,8 @@ fm10k_dev_rss_configure(struct rte_eth_dev *dev)
>  		0x6A, 0x42, 0xB7, 0x3B, 0xBE, 0xAC, 0x01, 0xFA,
>  	};
> 
> -	if (dev->data->nb_rx_queues == 1 ||
> -	    dev_conf->rxmode.mq_mode != ETH_MQ_RX_RSS ||
> -	    dev_conf->rx_adv_conf.rss_conf.rss_hf == 0) {

Yes, you can enable RSS with single queue, but it is not necessary.

> +	if (dev_conf->rxmode.mq_mode != ETH_MQ_RX_RSS ||
> +		dev_conf->rx_adv_conf.rss_conf.rss_hf == 0) {
>  		FM10K_WRITE_REG(hw, FM10K_MRQC(0), 0);
>  		return;
>  	}
> --
> 2.9.0.windows.1
> 
> 

Regards
Qi