On 01.06.21 18:10, Olaf Hering wrote:
> Introduce new API to test if a fixed number of bits is clear or set,
> and clear or set them all at once.

More precise: the new functions check whether BITS_PER_LONG bits are
set or clear.

> The caller has to make sure the input bitnumber is a multiply of BITS_PER_LONG.

s/multiply/multiple/

> 
> This API avoids the loop over each bit in a known range just to see
> if all of them are either clear or set.
> 
> Signed-off-by: Olaf Hering <olaf@aepfle.de>
> ---
>   tools/libs/ctrl/xc_bitops.h | 25 +++++++++++++++++++++++++
>   1 file changed, 25 insertions(+)
> 
> diff --git a/tools/libs/ctrl/xc_bitops.h b/tools/libs/ctrl/xc_bitops.h
> index f0bac4a071..92f38872fb 100644
> --- a/tools/libs/ctrl/xc_bitops.h
> +++ b/tools/libs/ctrl/xc_bitops.h
> @@ -77,4 +77,29 @@ static inline void bitmap_or(void *_dst, const void *_other,
>           dst[i] |= other[i];
>   }
>   
> +static inline int test_bit_long_set(unsigned long nr_base, const void *_addr)

Make return type bool (here and below)?

> +{
> +    const unsigned long *addr = _addr;
> +    unsigned long val = addr[nr_base / BITS_PER_LONG];

Add a blank line here (same below).

> +    return val == ~0; > +}
> +
> +static inline int test_bit_long_clear(unsigned long nr_base, const void *_addr)
> +{
> +    const unsigned long *addr = _addr;
> +    unsigned long val = addr[nr_base / BITS_PER_LONG];
> +    return val == 0;
> +}
> +
> +static inline void clear_bit_long(unsigned long nr_base, void *_addr)
> +{
> +    unsigned long *addr = _addr;
> +    addr[nr_base / BITS_PER_LONG] = 0;
> +}
> +
> +static inline void set_bit_long(unsigned long nr_base, void *_addr)
> +{
> +    unsigned long *addr = _addr;
> +    addr[nr_base / BITS_PER_LONG] = ~0;
> +}
>   #endif  /* XC_BITOPS_H */
> 


Juergen