[dpdk-dev] [PATCH] net/ixgbe: fix TC bandwidth setting

[dpdk-dev] [PATCH] net/ixgbe: fix TC bandwidth setting

[Wenzhuo Lu]

4 and 8 TCs are supported on ixgbe. By default there're
8 TCs. So when initializing the device, the bandwidth for
8 TCs is set.
When changing the TC number, it's only considered setting
the bandwidth for 4 TCs. If the user change the number
from 4 to 8, the TCs' bandwidth is not right.

Fixes: 0807f80d35d0 (ixgbe: DCB / flow control)
CC: stable@dpdk.org

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
---
 drivers/net/ixgbe/ixgbe_rxtx.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index 9502432..9294a0d 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -3758,6 +3758,15 @@ void __attribute__((cold))
 			tc->path[IXGBE_DCB_TX_CONFIG].bwg_percent = 0;
 			tc->path[IXGBE_DCB_RX_CONFIG].bwg_percent = 0;
 		}
+	} else {
+		/* Re-configure 8 TCs BW */
+		for (i = 0; i < nb_tcs; i++) {
+			tc = &dcb_config->tc_config[i];
+			tc->path[IXGBE_DCB_TX_CONFIG].bwg_percent =
+				(uint8_t)(100 / nb_tcs + (i & 1));
+			tc->path[IXGBE_DCB_RX_CONFIG].bwg_percent =
+				(uint8_t)(100 / nb_tcs + (i & 1));
+		}
 	}
 
 	switch (hw->mac.type) {
-- 
1.9.3

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/ixgbe: fix TC bandwidth setting

[Ferruh Yigit]

On 3/24/2017 2:51 AM, Wenzhuo Lu wrote:
> 4 and 8 TCs are supported on ixgbe. By default there're
> 8 TCs. So when initializing the device, the bandwidth for
> 8 TCs is set.
> When changing the TC number, it's only considered setting
> the bandwidth for 4 TCs. If the user change the number
> from 4 to 8, the TCs' bandwidth is not right.
> 
> Fixes: 0807f80d35d0 (ixgbe: DCB / flow control)
> CC: stable@dpdk.org
> 
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>

Applied to dpdk-next-net/master, thanks.