diff options
| author | Aleksandr Loktionov | 2023-05-16 10:41:46 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2023-05-24 17:32:47 +0100 |
| commit | e02d2b987c0297dd8a2b069a03b6b30616a88b23 (patch) | |
| tree | 371be4a6dce76da9647c5d836badfe0ef3768bc1 /net/ncsi | |
| parent | 516114d7fb584af33af26a859f8f51810f961900 (diff) | |
igb: fix bit_shift to be in [1..8] range
[ Upstream commit 60d758659f1fb49e0d5b6ac2691ede8c0958795b ]
In igb_hash_mc_addr() the expression:
"mc_addr[4] >> 8 - bit_shift", right shifting "mc_addr[4]"
shift by more than 7 bits always yields zero, so hash becomes not so different.
Add initialization with bit_shift = 1 and add a loop condition to ensure
bit_shift will be always in [1..8] range.
Fixes: 9d5c824399de ("igb: PCI-Express 82575 Gigabit Ethernet driver")
Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'net/ncsi')
0 files changed, 0 insertions, 0 deletions