Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763811AbZFQKAY (ORCPT ); Wed, 17 Jun 2009 06:00:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757741AbZFQKAI (ORCPT ); Wed, 17 Jun 2009 06:00:08 -0400 Received: from cantor.suse.de ([195.135.220.2]:49865 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757788AbZFQKAH (ORCPT ); Wed, 17 Jun 2009 06:00:07 -0400 Date: Wed, 17 Jun 2009 12:00:06 +0200 From: Nick Piggin To: Wu Fengguang Cc: Andi Kleen , Balbir Singh , Andrew Morton , LKML , Ingo Molnar , Mel Gorman , Thomas Gleixner , "H. Peter Anvin" , Peter Zijlstra , Hugh Dickins , "riel@redhat.com" , "chris.mason@oracle.com" , "linux-mm@kvack.org" Subject: Re: [RFC][PATCH] HWPOISON: only early kill processes who installed SIGBUS handler Message-ID: <20090617100006.GC14915@wotan.suse.de> References: <20090615042753.GA20788@localhost> <20090615064447.GA18390@wotan.suse.de> <20090615070914.GC31969@one.firstfloor.org> <20090615071907.GA8665@wotan.suse.de> <20090615121001.GA10944@localhost> <20090615122528.GA13256@wotan.suse.de> <20090615142225.GA11167@localhost> <20090617063702.GA20922@localhost> <20090617080404.GB31192@wotan.suse.de> <20090617095532.GA25001@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090617095532.GA25001@localhost> User-Agent: Mutt/1.5.9i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1976 Lines: 50 On Wed, Jun 17, 2009 at 05:55:32PM +0800, Wu Fengguang wrote: > On Wed, Jun 17, 2009 at 04:04:04PM +0800, Nick Piggin wrote: > > Well then you can still early-kill random apps that did not > > want it, and you may still cause problems if its sigbus > > handler does something nontrivial. > > > > Can you use a prctl or something so it can expclitly > > register interest in this? > > No I don't think prctl would be much better. > > - if an application want early/late kill, it can do so with a proper > written SIGBUS handler: the prctl call is redundant. s/proper written/is switched to new semantics based on the existance of a/ > - if an admin want to control early/late kill for an unmodified app, > prctl is as unhelpful as this patch(*). Clearly you can execute a process with a given prctl. > - prctl does can help legacy apps whose SIGBUS handler has trouble > with the new SIGBUS codes, however such application should be rare > and the application should be fixed(why shall it do something wrong > on newly introduced code at all? Shall we stop introducing new codes > just because some random buggy app cannot handle new codes?) Backwards compatibility? Kind of important. > So I still prefer this patch, until we come up with some solution that > allows both app and admin to change the setting. Not only does it allow that, but it also provides backwards compatibility. Your patch does not allow admin to change anything nor does it guarantee 100% back compat so I can't see how you think it is better. Also it does not allow for an app with a SIGBUS handler to use late kill. If late kill is useful to anyone, why would it not be useful to some app with a SIGBUS handler (that is not KVM)? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/