Re: [PATCH] sched: remove sched_find_first_bit()

From: Arnd Bergmann
Date: Mon May 15 2017 - 10:18:56 EST


On Sat, May 13, 2017 at 3:01 AM, Yury Norov <ynorov@xxxxxxxxxxxxxxxxxx> wrote:

> include/asm-generic/bitops/sched.h as it declares only this
> function. Alpha has it's own implementation, very similar to
> generic one, so it is also removed.

Nice consolidation, I see that every architecture once had a copy of
this function, and they were all removed at one point long ago,
except for Alpha, and there is no reason to leave that one in the tree.

I'll leave it to you and Ingo to figure out what the two callers should
do, but for removing function from asm-generic:

Acked-by: Arnd Bergmann <arnd@xxxxxxxx>

Maybe you could split the patch into two changesets, and make the
first of them move the function to kernel/sched/rt.c, and the second
one (if we still want it then) can change the implementation to whatever
you conclude is ideal.

Arnd