From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754605Ab2D2WvL (ORCPT ); Sun, 29 Apr 2012 18:51:11 -0400 Received: from zeniv.linux.org.uk ([195.92.253.2]:41780 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754479Ab2D2WvI (ORCPT ); Sun, 29 Apr 2012 18:51:08 -0400 Date: Sun, 29 Apr 2012 23:51:00 +0100 From: Al Viro To: Geert Uytterhoeven Cc: Oleg Nesterov , Linus Torvalds , linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, Roland McGrath , Greg Ungerer Subject: Re: [RFC] TIF_NOTIFY_RESUME, arch/*/*/*signal*.c and all such Message-ID: <20120429225100.GB6871@ZenIV.linux.org.uk> References: <20120420180748.GI6871@ZenIV.linux.org.uk> <20120423180150.GA6871@ZenIV.linux.org.uk> <20120424072617.GB6871@ZenIV.linux.org.uk> <20120426183742.GA324@redhat.com> <20120426231942.GJ6871@ZenIV.linux.org.uk> <20120427172444.GA30267@redhat.com> <20120427184528.GL6871@ZenIV.linux.org.uk> <20120427193413.GM6871@ZenIV.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120427193413.GM6871@ZenIV.linux.org.uk> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 27, 2012 at 08:34:13PM +0100, Al Viro wrote: > On Fri, Apr 27, 2012 at 09:14:53PM +0200, Geert Uytterhoeven wrote: > > On Fri, Apr 27, 2012 at 20:45, Al Viro wrote: > > >> The only comment I have, > > >> 38671a3e831ed7327affb24d715f98bb99c80e56 m68k: add TIF_NOTIFY_RESUME and handle it > > >> forgets to unexport do_signal(). > > > > > > Meh... ??The thing is, there are _two_ of them. ??signal_mm.c and signal_no.c > > > badly need merging, with common stuff moved to signal.c. ??I really hate > > > > Greg recently posted a patch to merge them: > > http://www.spinics.net/lists/linux-m68k/msg04995.html > > Nice... BTW, could you comment on > m68k: don't open-code block_sigmask() > m68k: use set_current_blocked in sigreturn/rt_sigreturn > m68k-nommu: do_signal() is only called when returning to user mode > m68k: add TIF_NOTIFY_RESUME and handle it. > in signal.git tree? The last one is really interesting... > > Just pick the current tree - one that was there yesterday had a dumb typo > in thread_info.h part, so it wouldn't even compile (TIF_NOTIFE_RESUME > defined instead of TIF_NOTIFY_RESUME ;-/) BTW, Greg's patch put into the queue in the very beginning, everything rebased on top of it.