Message ID | 1323702852.13285.25.camel@twins |
---|---|
State | RFC |
Delegated to: | David Miller |
Headers | show |
From: Peter Zijlstra <peterz@infradead.org> Date: Mon, 12 Dec 2011 16:14:12 +0100 > The only thing resting me is asking you to simply test the below patch > and report what happens. Hopefully things will simply work.. if not I've > messed up and need to go look harder :/ I'll give it a spin, and get back to you, thanks! -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: David Miller <davem@davemloft.net> Date: Wed, 21 Dec 2011 16:44:26 -0500 (EST) > From: Peter Zijlstra <peterz@infradead.org> > Date: Mon, 12 Dec 2011 16:14:12 +0100 > >> The only thing resting me is asking you to simply test the below patch >> and report what happens. Hopefully things will simply work.. if not I've >> messed up and need to go look harder :/ > > I'll give it a spin, and get back to you, thanks! Ok, so far so good, I stressed out my Niagara-T3 box for a while doing gcc bootstraps, running the testsuite, etc. and all seems well so far. I'll keep beating on it and let you know if anything funny happens, but for now as far as I'm concerned: Acked-by: David S. Miller <davem@davemloft.net> -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, 2011-12-21 at 18:05 -0500, David Miller wrote: > From: David Miller <davem@davemloft.net> > Date: Wed, 21 Dec 2011 16:44:26 -0500 (EST) > > > From: Peter Zijlstra <peterz@infradead.org> > > Date: Mon, 12 Dec 2011 16:14:12 +0100 > > > >> The only thing resting me is asking you to simply test the below patch > >> and report what happens. Hopefully things will simply work.. if not I've > >> messed up and need to go look harder :/ > > > > I'll give it a spin, and get back to you, thanks! > > Ok, so far so good, I stressed out my Niagara-T3 box for a while doing gcc > bootstraps, running the testsuite, etc. and all seems well so far. > > I'll keep beating on it and let you know if anything funny happens, but for > now as far as I'm concerned: > > Acked-by: David S. Miller <davem@davemloft.net> Thanks David!! I'll probably post again (and propose merger) once Catalin's patches that remove __ARCH_WANT_INTERRUPTS_ON_CTXSW from ARM have landed. ARM being the last user of that feature, and with only SPARC64 and IA64 (needlessly) using __ARCH_WANT_UNLOCKED_CTXSW we could finally remove some of that hairy code from the core scheduler. -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Peter Zijlstra <peterz@infradead.org> Date: Thu, 22 Dec 2011 20:16:39 +0100 > ARM being the last user of that feature, and with only SPARC64 and IA64 > (needlessly) using __ARCH_WANT_UNLOCKED_CTXSW we could finally remove > some of that hairy code from the core scheduler. W00t! :-) -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/sparc/include/asm/system_64.h b/arch/sparc/include/asm/system_64.h index 10bcabc..715fefa 100644 --- a/arch/sparc/include/asm/system_64.h +++ b/arch/sparc/include/asm/system_64.h @@ -123,8 +123,6 @@ extern void __flushw_user(void); #define flush_user_windows flushw_user #define flush_register_windows flushw_all -/* Don't hold the runqueue lock over context switch */ -#define __ARCH_WANT_UNLOCKED_CTXSW #define prepare_arch_switch(next) \ do { \ flushw_all(); \