On Tue, Mar 17, 2020 at 10:43:03PM -0300, Vitor Massaru Iha wrote: > @@ -1624,6 +1624,7 @@ static inline unsigned int xas_find_chunk(struct xa_state *xas, bool advance, > if (XA_CHUNK_SIZE == BITS_PER_LONG) { > if (offset < XA_CHUNK_SIZE) { > unsigned long data = *addr & (~0UL << offset); > + > if (data) > return __ffs(data); > } Do you seriously think this makes the function in any way more legible?