diff options
| author |   Michael Chan <michael.chan@broadcom.com> | 2018-12-16 18:46:27 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-12-17 23:08:53 -0800 |
| commit | 24654f095ed68eafcdfdbfccbe6324ad1475c1bb (patch) | |
| tree | 8010f47bde35d6db4fc2a8a5ade3aac5492401da /drivers | |
| parent | bnxt_en: Add ethtool -S priority counters. (diff) | |
| download | linux-dev-24654f095ed68eafcdfdbfccbe6324ad1475c1bb.tar.xz linux-dev-24654f095ed68eafcdfdbfccbe6324ad1475c1bb.zip | |
bnxt_en: Don't set ETS on unused TCs.
Currently, the code allows ETS bandwidth weight 0 to be set on unused TCs.
We should not set any DCB parameters on unused TCs at all.
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/broadcom/bnxt/bnxt_dcb.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_dcb.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_dcb.c index a85d2be986af..15c7041e937b 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_dcb.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_dcb.c @@ -471,7 +471,10 @@ static int bnxt_ets_validate(struct bnxt *bp, struct ieee_ets *ets, u8 *tc) if (total_ets_bw > 100) return -EINVAL; - *tc = max_tc + 1; + if (max_tc >= bp->max_tc) + *tc = bp->max_tc; + else + *tc = max_tc + 1; return 0; } |