From: Paolo Valente <paolo.vale...@unimore.it> Date: Wed, 10 Jul 2013 15:46:07 +0200
> Hi, > this patchset tries to address the two issues raised by Dave in > http://marc.info/?l=linux-kernel&m=136254542019113&w=2 > > The first of these issues has been also raised by > David Laight: > http://marc.info/?l=linux-kernel&m=136256446624557&w=2 > >> Although two topics for possibly resolving later: >> >> 1) That 1ULL bit mask fix is quite expensive on 32-bit, it would >> probably be cheaper to test for that case using a helper function >> that nops out on 64-bit. Although this is not so important. >> > I have tried to find a simple solution, based on the properties of the mask. > It is in the first patch. > >> 2) That static inline forward declaration is ugly, better to remove >> the inline tag (let the compiler handle it) or move the function >> above all the call sites. >> > Done by the second patch. Looks good, thanks for doing this, both applied. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/