From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759431AbcG1DNU (ORCPT ); Wed, 27 Jul 2016 23:13:20 -0400 Received: from mail.efficios.com ([78.47.125.74]:45473 "EHLO mail.efficios.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759352AbcG1DNK (ORCPT ); Wed, 27 Jul 2016 23:13:10 -0400 Date: Thu, 28 Jul 2016 03:13:07 +0000 (UTC) From: Mathieu Desnoyers To: Boqun Feng Cc: linux-kernel , linux-api , Andrew Morton , Russell King , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Paul Turner , Andrew Hunter , Peter Zijlstra , Andy Lutomirski , Andi Kleen , Dave Watson , Chris Lameter , Ben Maurer , rostedt , "Paul E. McKenney" , Josh Triplett , Linus Torvalds , Michael Ellerman , Benjamin Herrenschmidt , Paul Mackerras Message-ID: <2132356373.430.1469675587375.JavaMail.zimbra@efficios.com> In-Reply-To: <20160727150517.26983-2-boqun.feng@gmail.com> References: <20160727150352.GA32142@tardis.cn.ibm.com> <20160727150517.26983-1-boqun.feng@gmail.com> <20160727150517.26983-2-boqun.feng@gmail.com> Subject: Re: [RFC 2/4] Restartable sequences: powerpc architecture support MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [78.47.125.74] X-Mailer: Zimbra 8.7.0_GA_1659 (ZimbraWebClient - FF45 (Linux)/8.7.0_GA_1659) Thread-Topic: Restartable sequences: powerpc architecture support Thread-Index: 3Z+cOVnJavdmdaq6+UL54edVX/F/AA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ----- On Jul 27, 2016, at 11:05 AM, Boqun Feng boqun.feng@gmail.com wrote: > Call the rseq_handle_notify_resume() function on return to userspace if > TIF_NOTIFY_RESUME thread flag is set. > > Increment the event counter and perform fixup on the pre-signal when a > signal is delivered on top of a restartable sequence critical section. Picked into my patchset. I'm keeping a volatile branch of my current work at https://github.com/compudj/linux-percpu-dev/tree/rseq-fallback if you are interested in the progress. Thanks! Mathieu > > Signed-off-by: Boqun Feng > --- > arch/powerpc/Kconfig | 1 + > arch/powerpc/kernel/signal.c | 3 +++ > 2 files changed, 4 insertions(+) > > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig > index 0a9d439bcda6..4e93629c6b84 100644 > --- a/arch/powerpc/Kconfig > +++ b/arch/powerpc/Kconfig > @@ -120,6 +120,7 @@ config PPC > select HAVE_PERF_USER_STACK_DUMP > select HAVE_REGS_AND_STACK_ACCESS_API > select HAVE_HW_BREAKPOINT if PERF_EVENTS && PPC_BOOK3S_64 > + select HAVE_RSEQ > select ARCH_WANT_IPC_PARSE_VERSION > select SPARSE_IRQ > select IRQ_DOMAIN > diff --git a/arch/powerpc/kernel/signal.c b/arch/powerpc/kernel/signal.c > index cb64d6feb45a..339d0ebe2906 100644 > --- a/arch/powerpc/kernel/signal.c > +++ b/arch/powerpc/kernel/signal.c > @@ -131,6 +131,8 @@ static void do_signal(struct pt_regs *regs) > /* Re-enable the breakpoints for the signal stack */ > thread_change_pc(current, regs); > > + rseq_signal_deliver(regs); > + > if (is32) { > if (ksig.ka.sa.sa_flags & SA_SIGINFO) > ret = handle_rt_signal32(&ksig, oldset, regs); > @@ -157,6 +159,7 @@ void do_notify_resume(struct pt_regs *regs, unsigned long > thread_info_flags) > if (thread_info_flags & _TIF_NOTIFY_RESUME) { > clear_thread_flag(TIF_NOTIFY_RESUME); > tracehook_notify_resume(regs); > + rseq_handle_notify_resume(regs); > } > > user_enter(); > -- > 2.9.0 -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com