On 03.04.2024 12:19, Oleksii Kurochko wrote:
> --- a/xen/include/xen/bitops.h
> +++ b/xen/include/xen/bitops.h
> @@ -201,7 +201,7 @@ static always_inline __pure int test_bit(int nr, const
> volatile void *addr)
>
> static always_inline __pure int fls(unsigned int x)
> {
> - if (__builtin_constant_p(x))
> + if ( __builtin_constant_p(x) )
> return generic_fls(x);
>
> #ifndef arch_fls
> @@ -213,7 +213,7 @@ static always_inline __pure int fls(unsigned int x)
>
> static always_inline __pure int flsl(unsigned long x)
> {
> - if (__builtin_constant_p(x))
> + if ( __builtin_constant_p(x) )
> return generic_flsl(x);
>
> #ifndef arch_flsl
As per this something is clearly wrong with the split between the previous
patch and this one.
Jan
