Re: [PATCH 00/10] sched.h modernization -v2, phase #1: "Pre-splitup cleanups"

From: Linus Torvalds
Date: Wed Feb 08 2017 - 15:23:24 EST


On Wed, Feb 8, 2017 at 10:34 AM, Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>
> phase #1: "Pre-splitup cleanups": # 0721da143d99~1..582c7529a1bd, 10 patches
>
> phase #2: "Prepare header dependencies": # 9017f600c534~1..d1eb6f0b580d, 48 patches
> phase #3: "Move definitions between headers": # 3d01f4b6e32d~1..fd88c1773dd6, 53 patches
> phase #4: "Remove header dependencies": # 2d246f83c5d3~1..2205323e0ca1, 35 patches
>
> phase #5: "Post-splitup cleanups": # 55cf8bfb3af4~1..bcc6be9d66b2, 2 patches

So this seemed to be easier to go through, although I basically
skipped all the one-liners so who knows what conflict markers might
have been hiding there ;^)

I would personally have preferred phase 2/3 to be done in the reverse
order (by having the "move definitions" phase just add an include in
<sched.h> to still get them after the move), but it's not a huge issue
and I doubt it's worth re-doing the series over it.

I noted a couple of nits I found, but in general I think you can just
add my ack to this.

Linus