Re: [Patch 17/23] mask v2 = [6/7] nodemask_t_ia64_changes

Paul Jackson
2004-04-09T18:55:46Z

> I've reduced this to a pair of bitmap.h operators:
>
> > #define find_next_bit_in_bitmap(src, nbits, n) \
> > ({ int i = (n); while(i < (nbits) && !test_bit(i, (src))) i++; i; })
> >
> > #define find_first_bit_in_bitmap(src, nbits) \
> > find_next_bit_in_bitmap((src), (nbits), 0)
>
> ...
>
> My next mask patchset will have these.

Wrong. My next patch set, released 8 April, did not have these.

A subsequent discussion with Nick Piggin convinced me that the
better path was to take find_next_bit() out of line, which I did
for ia64 (other arch's might also want to do this).