Messages in this thread | | | From | Nick Desaulniers <> | Date | Thu, 7 May 2020 12:19:30 -0700 | Subject | Re: [PATCH] x86: bitops: fix build regression |
| |
On Thu, May 7, 2020 at 7:00 AM Brian Gerst <brgerst@gmail.com> 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
| |