| From f2127ce3441e20b66d8c2451dce6d541eeb2073e Mon Sep 17 00:00:00 2001 |
| From: Michael Chan <michael.chan@broadcom.com> |
| Date: Sun, 2 Feb 2020 02:41:38 -0500 |
| Subject: [PATCH] bnxt_en: Fix TC queue mapping. |
| |
| commit 18e4960c18f484ac288f41b43d0e6c4c88e6ea78 upstream. |
| |
| The driver currently only calls netdev_set_tc_queue when the number of |
| TCs is greater than 1. Instead, the comparison should be greater than |
| or equal to 1. Even with 1 TC, we need to set the queue mapping. |
| |
| This bug can cause warnings when the number of TCs is changed back to 1. |
| |
| Fixes: 7809592d3e2e ("bnxt_en: Enable MSIX early in bnxt_init_one().") |
| Signed-off-by: Michael Chan <michael.chan@broadcom.com> |
| Signed-off-by: Jakub Kicinski <kuba@kernel.org> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethernet/broadcom/bnxt/bnxt.c |
| index ee1fe9d89c94..2fa84597803d 100644 |
| --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c |
| +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c |
| @@ -7391,7 +7391,7 @@ static void bnxt_setup_msix(struct bnxt *bp) |
| int tcs, i; |
| |
| tcs = netdev_get_num_tc(dev); |
| - if (tcs > 1) { |
| + if (tcs) { |
| int i, off, count; |
| |
| for (i = 0; i < tcs; i++) { |
| -- |
| 2.7.4 |
| |
