On 10/10/24 17:45, Colin Ian King wrote:
The left shift int 32 bit integer constants 1 is evaluated using 32 bit arithmetic and then assigned to a 64 bit unsigned integer. In the case where the shift is 32 or more this can lead to an overflow. Avoid this by shifting using the BIT_ULL macro instead. Fixes: 019aba04f08c ("octeontx2-af: Modify SMQ flush sequence to drop packets") Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> --- V2: Fix both (1 << i) shifts, thanks to Dan Carpenter for spotting the second shift that I overlooked in the first patch.
The blamed commit is in the 'net' tree already, I'm applying the patch there.
Cheers, Paolo