On 27/05/2024 2:44 pm, Jan Beulich wrote: >> --- a/xen/include/xen/bitops.h >> +++ b/xen/include/xen/bitops.h >> @@ -60,6 +60,14 @@ static always_inline __pure unsigned int ffsl(unsigned >> long x) >> #endif >> } >> >> +static always_inline __pure unsigned int ffs64(uint64_t x) >> +{ >> + if ( BITS_PER_LONG == 64 ) > In principle >= 64 would be okay here, and hence I'd prefer if we used that > less strict form. Yet I'm not going to insist.
Sorry - I'd meant to include this, but I've just found it still local to my dev branch. ~Andrew