[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1331100381.2259.2.camel@sbsiddha-mobl2>
Date: Tue, 06 Mar 2012 22:06:21 -0800
From: Suresh Siddha <suresh.b.siddha@...el.com>
To: Venkatesh Pallipadi <venki@...gle.com>
Cc: Ingo Molnar <mingo@...e.hu>, Peter Zijlstra <peterz@...radead.org>,
Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...hat.com>,
"H. Peter Anvin" <hpa@...or.com>,
Aaron Durbin <adurbin@...gle.com>,
Paul Turner <pjt@...gle.com>,
Yong Zhang <yong.zhang0@...il.com>,
linux-kernel@...r.kernel.org, Tony Luck <tony.luck@...el.com>,
Fenghua Yu <fenghua.yu@...el.com>,
Ralf Baechle <ralf@...ux-mips.org>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Paul Mackerras <paulus@...ba.org>,
Martin Schwidefsky <schwidefsky@...ibm.com>,
Heiko Carstens <heiko.carstens@...ibm.com>
Subject: Re: fork_idle from wq cleanup
On Tue, 2012-03-06 at 13:41 -0800, Venkatesh Pallipadi wrote:
> I looked at avoiding the wq stuff. But, there is no easy/clean way to do
> if as we have number of routines within copy_process assume current
> being the parent.
>
> But, we can move most of the complication out of arch code into generic code
> and share it across multiple archs. Here is the patch for that.
>
> We can easily add x86 percpu idle task on top of this cleanup (except for
> mini-circus to switch idle_task of CPU 0).
>
> Sorry. The patches for ia64, mips, powerpc and s390 are untested and are in
> "should work" category.
>
> Overall diffstat looks like
> ---
> arch/ia64/kernel/smpboot.c | 49 ++---------------------------
> arch/mips/kernel/smp.c | 47 +---------------------------
> arch/powerpc/kernel/smp.c | 63 ++++----------------------------------
> arch/s390/kernel/smp.c | 38 +++--------------------
> arch/x86/kernel/smpboot.c | 74 ++++++---------------------------------------
> include/linux/sched.h | 1
> kernel/fork.c | 48 +++++++++++++++++++++++++++++
> 7 files changed, 79 insertions(+), 241 deletions(-)
>
Looks good to me.
Reviewed-by: Suresh Siddha <suresh.b.siddha@...el.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists