From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: [PATCH 2/2] x86 TS_RESTORE_SIGMASK Date: Mon, 31 Mar 2008 15:01:06 +0200 Message-ID: <20080331130106.GA15198@elte.hu> References: <20080329001230.D013726FA1D@magilla.localdomain> <20080329001341.7F93826FA1D@magilla.localdomain> <20080329022408.0DD4726FA1D@magilla.localdomain> <20080329031413.EA38926FA1D@magilla.localdomain> <20080329031451.D2E1226FA1D@magilla.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20080329031451.D2E1226FA1D@magilla.localdomain> Sender: linux-ia64-owner@vger.kernel.org To: Roland McGrath Cc: Linus Torvalds , Andrew Morton , Martin Schwidefsky , linux-s390@vger.kernel.org, tony.luck@intel.com, linux-ia64@vger.kernel.org, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-arch.vger.kernel.org * Roland McGrath wrote: > Replace TIF_RESTORE_SIGMASK with TS_RESTORE_SIGMASK and define our own > set_restore_sigmask() function. This saves the costly SMP-safe > set_bit operation, which we do not need for the sigmask flag since > TIF_SIGPENDING always has to be set too. > > Signed-off-by: Roland McGrath > --- > arch/x86/ia32/ia32_signal.c | 2 +- > arch/x86/kernel/signal_32.c | 19 ++++++++++--------- > arch/x86/kernel/signal_64.c | 16 +++++++++------- > include/asm-x86/thread_info_32.h | 13 +++++++++++-- > include/asm-x86/thread_info_64.h | 13 +++++++++++-- > 5 files changed, 42 insertions(+), 21 deletions(-) the x86 bits look nice - but i guess this all wants to go into 2.6.26 via -mm (or linux-next), due to its cross-arch nature? Ingo From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.mail.elte.hu ([157.181.151.9]:34616 "EHLO mx2.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754829AbYCaNBW (ORCPT ); Mon, 31 Mar 2008 09:01:22 -0400 Date: Mon, 31 Mar 2008 15:01:06 +0200 From: Ingo Molnar Subject: Re: [PATCH 2/2] x86 TS_RESTORE_SIGMASK Message-ID: <20080331130106.GA15198@elte.hu> References: <20080329001230.D013726FA1D@magilla.localdomain> <20080329001341.7F93826FA1D@magilla.localdomain> <20080329022408.0DD4726FA1D@magilla.localdomain> <20080329031413.EA38926FA1D@magilla.localdomain> <20080329031451.D2E1226FA1D@magilla.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080329031451.D2E1226FA1D@magilla.localdomain> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Roland McGrath Cc: Linus Torvalds , Andrew Morton , Martin Schwidefsky , linux-s390@vger.kernel.org, tony.luck@intel.com, linux-ia64@vger.kernel.org, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org Message-ID: <20080331130106.wtfpqxe4eeKf0BcpKZnNdjHYi-eMchzxJeGoIVJwUZU@z> * Roland McGrath wrote: > Replace TIF_RESTORE_SIGMASK with TS_RESTORE_SIGMASK and define our own > set_restore_sigmask() function. This saves the costly SMP-safe > set_bit operation, which we do not need for the sigmask flag since > TIF_SIGPENDING always has to be set too. > > Signed-off-by: Roland McGrath > --- > arch/x86/ia32/ia32_signal.c | 2 +- > arch/x86/kernel/signal_32.c | 19 ++++++++++--------- > arch/x86/kernel/signal_64.c | 16 +++++++++------- > include/asm-x86/thread_info_32.h | 13 +++++++++++-- > include/asm-x86/thread_info_64.h | 13 +++++++++++-- > 5 files changed, 42 insertions(+), 21 deletions(-) the x86 bits look nice - but i guess this all wants to go into 2.6.26 via -mm (or linux-next), due to its cross-arch nature? Ingo