Re: [PATCH] x86: bitops: fix build regression

From: Nick Desaulniers
Date: Thu May 07 2020 - 15:19:44 EST


On Thu, May 7, 2020 at 7:00 AM Brian Gerst <brgerst@xxxxxxxxx> wrote:
>
> This change will make sparse happy and allow these cleanups:
> #define CONST_MASK(nr) ((u8)1 << ((nr) & 7))

yep, this is more elegant, IMO. Will send a v3 later with this
change. Looking at the uses of CONST_MASK, I noticed
arch_change_bit() currently has the (u8) cast from commit
838e8bb71dc0c ("x86: Implement change_bit with immediate operand as
"lock xorb""), so that instance can get cleaned up with the above
suggestion.

--
Thanks,
~Nick Desaulniers