[dpdk-dev] [PATCH] net/ixgbe: fix rss max rxtx queue num

[dpdk-dev] [PATCH] net/ixgbe: fix rss max rxtx queue num

[Qiming Chen]

Through the local actual test, it is found that the maximum number of
RSS queues is 16, not 128. Check the chip manual, the register MRQE
has the following description: RSS only-Single set of RSS 16 queues.
It also coincides with the actual test value.

Fixes: abf7275bbaa2 ("ixgbe: move to drivers/net/")
Cc: stable@dpdk.org

Signed-off-by: Qiming Chen <chenqiming_huawei@163.com>
---
 drivers/net/ixgbe/base/ixgbe_82599.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ixgbe/base/ixgbe_82599.c b/drivers/net/ixgbe/base/ixgbe_82599.c
index 69fd4cd3fb..edd63018d1 100644
--- a/drivers/net/ixgbe/base/ixgbe_82599.c
+++ b/drivers/net/ixgbe/base/ixgbe_82599.c
@@ -8,8 +8,8 @@
 #include "ixgbe_common.h"
 #include "ixgbe_phy.h"
 
-#define IXGBE_82599_MAX_TX_QUEUES 128
-#define IXGBE_82599_MAX_RX_QUEUES 128
+#define IXGBE_82599_MAX_TX_QUEUES 16
+#define IXGBE_82599_MAX_RX_QUEUES 16
 #define IXGBE_82599_RAR_ENTRIES   128
 #define IXGBE_82599_MC_TBL_SIZE   128
 #define IXGBE_82599_VFT_TBL_SIZE  128
-- 
2.30.1.windows.1

Re: [dpdk-dev] [PATCH] net/ixgbe: fix rss max rxtx queue num

[Wang, Haiyue]

> -----Original Message-----
> From: Qiming Chen <chenqiming_huawei@163.com>
> Sent: Tuesday, August 31, 2021 17:11
> To: dev@dpdk.org
> Cc: Wang, Haiyue <haiyue.wang@intel.com>; Qiming Chen <chenqiming_huawei@163.com>; stable@dpdk.org
> Subject: [PATCH] net/ixgbe: fix rss max rxtx queue num
> 
> Through the local actual test, it is found that the maximum number of
> RSS queues is 16, not 128. Check the chip manual, the register MRQE
> has the following description: RSS only-Single set of RSS 16 queues.
> It also coincides with the actual test value.
> 
> Fixes: abf7275bbaa2 ("ixgbe: move to drivers/net/")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Qiming Chen <chenqiming_huawei@163.com>
> ---
>  drivers/net/ixgbe/base/ixgbe_82599.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ixgbe/base/ixgbe_82599.c b/drivers/net/ixgbe/base/ixgbe_82599.c
> index 69fd4cd3fb..edd63018d1 100644
> --- a/drivers/net/ixgbe/base/ixgbe_82599.c
> +++ b/drivers/net/ixgbe/base/ixgbe_82599.c
> @@ -8,8 +8,8 @@
>  #include "ixgbe_common.h"
>  #include "ixgbe_phy.h"
> 
> -#define IXGBE_82599_MAX_TX_QUEUES 128
> -#define IXGBE_82599_MAX_RX_QUEUES 128
> +#define IXGBE_82599_MAX_TX_QUEUES 16
> +#define IXGBE_82599_MAX_RX_QUEUES 16

Let's keep the value, since there are so many kind of ixgbe,
and sync it with the kernel:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c#n12


>  #define IXGBE_82599_RAR_ENTRIES   128
>  #define IXGBE_82599_MC_TBL_SIZE   128
>  #define IXGBE_82599_VFT_TBL_SIZE  128
> --
> 2.30.1.windows.1