On Mon, Jun 17, 2019 at 05:12:49PM +0100, Colin King wrote: > From: Colin Ian King <[email protected]> > > Left shifting the signed int value 1 by 31 bits has undefined behaviour > and the shift amount oq_no can be as much as 63. Fix this by using > BIT_ULL(oq_no) instead. > > Addresses-Coverity: ("Bad shift operation") > Fixes: f21fb3ed364b ("Add support of Cavium Liquidio ethernet adapters") > Signed-off-by: Colin Ian King <[email protected]> > ---
Looks good. Thanks! Reviewed-by: Dan Carpenter <[email protected]> regards, dan carpenter
