From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from ns.suse.de ([195.135.220.2] helo=mx1.suse.de) by canuck.infradead.org with esmtps (Exim 4.63 #1 (Red Hat Linux)) id 1IKrtr-0001cb-Dc for kexec@lists.infradead.org; Tue, 14 Aug 2007 04:37:20 -0400 Date: Tue, 14 Aug 2007 10:37:10 +0200 From: Bernhard Walle Subject: Re: [patch] add kdump_after_notifier Message-ID: <20070814083710.GA14538@suse.de> References: <20070802112852.GA7054@in.ibm.com> <31687.1186113947@kao2.melbourne.sgi.com> <20070805110746.GA12540@in.ibm.com> <46C1691E.7090708@ah.jp.nec.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <46C1691E.7090708@ah.jp.nec.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: kexec-bounces@lists.infradead.org Errors-To: kexec-bounces+dwmw2=infradead.org+dwmw2=infradead.org@lists.infradead.org To: Takenori Nagano Cc: Keith Owens , k-miyoshi@cb.jp.nec.com, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, vgoyal@in.ibm.com, "Eric W. Biederman" , Andrew Morton KiBUYWtlbm9yaSBOYWdhbm8gPHQtbmFnYW5vQGFoLmpwLm5lYy5jb20+IFsyMDA3LTA4LTE0IDEw OjM0XToKPiBWaXZlayBHb3lhbCB3cm90ZToKPiA+IE9uIEZyaSwgQXVnIDAzLCAyMDA3IGF0IDAy OjA1OjQ3UE0gKzEwMDAsIEtlaXRoIE93ZW5zIHdyb3RlOgo+ID4gCj4gPiAKPiA+IFRvIHN1bSB1 cCwgY291cGxlIG9mIG9wdGlvbnMgY29tZSB0byBtaW5kLgo+ID4gCj4gPiAtIFJlZ2lzdGVyIGFs bCB0aGUgUkFTIHRvb2xzIG9uIGRpZSBub3RpZmllciBhbmQgcGFuaWMKPiA+ICAgbm90aWZpZXIg bGlzdHMgd2l0aCBmYWlybHkgaGlnaCBwcmlvcml0eS4gRXhwb3J0IGxpc3QKPiA+ICAgb2YgUkFT IHRvb2xzIHRvIHVzZXIgc3BhY2UgYW5kIGFsbG93IHVzZXJzIHRvIGRlY2lkZSB0aGUKPiA+ICAg b3JkZXIgb2YgZXhlY3V0aW9uIGFuZCBwcmlvcml0eSBvZiBSQVMgdG9vbHMuCj4gPiAKPiA+IC0g Q3JlYXRlIGEgc2VwYXJhdGUgUkFTIHRvb2wgbm90aWZpZXIgbGlzdCAocmFzX3Rvb2xfbm90aWZl cl9saXN0KS4KPiA+ICAgQWxsIHRoZSBSQVMgdG9vbHMgcmVnaXN0ZXIgb24gdGhpcyBsaXN0LiBU aGlzIGxpc3QgZ2V0cyBwcmlvcml0eQo+ID4gICBvdmVyIGRpZSBvciBwYW5pYyBub3RpZmllciBs aXN0LiBVc2VyIGRlY2lkZXMgdGhlIG9kZXIgb2YgZXhlY3V0aW9uCj4gPiAgIG9mIFJBUyB0b29s cy4gCj4gPiAKPiA+ICAgSGVyZSBhc3N1bXB0aW9uIGlzIHRoYXQgYWJvdmUgbGlzdCB3aWxsIG5v dCBiZSBleHBvcnRlZCB0byBtb2R1bGVzLgo+ID4gICBBbGwgdGhlIFJBUyB0b29scyB3aWxsIGJl IGluIGtlcm5lbCBhbmQgdGhleSBhbHdheXMgZ2V0IGEgcHJpb3JpdHkKPiA+ICAgdG8gaW5zcGVj dCBhbiBldmVudC4KPiA+IAo+ID4gV2hhdCBkbyBvdGhlcnMgdGhpbms/Cj4gCj4gVmVyeSBnb29k IGlkZWEuIEJ1dCB0aGVyZSBpcyBhIHByb2JsZW0gaG93IHRvIGdpdmUgZGVmYXVsdCBwcmlvcml0 eSB0byBSQVMgdG9vbHMuCj4gCj4gSG93IGFib3V0IHByaW9yaXR5IGNoYW5nZWFibGUgbm90aWZp ZXJfbGlzdD8gVXNlciBjYW4gY2hhbmdlIGxpc3Qgb3JkZXIKPiBkeW5hbWljYWxseSBpZiB0aGV5 IHdhbnQuIE9mIGNvdXJzZSwgd2UgaGF2ZSB0byBnaXZlIGhpZ2hlc3QgcHJpb3JpdHkgdG8ga2R1 bXAKPiBieSBkZWZhdWx0LiBJdCBpcyB2ZXJ5IHVzZWZ1bCBmb3IgdXNlcnMgd2hvIHdhbnQgdG8g dXNlIHNvbWUgUkFTIHRvb2xzLgoKSSB0aGluayB0aGF0IHdhcyB0aGUgaWRlYSBvZiB0aGUgZmly c3Qg4oCcLeKAnSAo4oCcZXhwb3J0IGxpc3Qgb2YgUkFTIHRvb2xzCnRvIHVzZXIgc3BhY2XigJ0p LgoKClRoYW5rcywKICAgQmVybmhhcmQKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCmtleGVjIG1haWxpbmcgbGlzdAprZXhlY0BsaXN0cy5pbmZyYWRlYWQu b3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8va2V4ZWMK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754396AbXHNIhi (ORCPT ); Tue, 14 Aug 2007 04:37:38 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752656AbXHNIhP (ORCPT ); Tue, 14 Aug 2007 04:37:15 -0400 Received: from mx1.suse.de ([195.135.220.2]:34262 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751451AbXHNIhN (ORCPT ); Tue, 14 Aug 2007 04:37:13 -0400 Date: Tue, 14 Aug 2007 10:37:10 +0200 From: Bernhard Walle To: Takenori Nagano Cc: vgoyal@in.ibm.com, Keith Owens , "Eric W. Biederman" , k-miyoshi@cb.jp.nec.com, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, Andrew Morton Subject: Re: [patch] add kdump_after_notifier Message-ID: <20070814083710.GA14538@suse.de> Mail-Followup-To: Takenori Nagano , vgoyal@in.ibm.com, Keith Owens , "Eric W. Biederman" , k-miyoshi@cb.jp.nec.com, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, Andrew Morton References: <20070802112852.GA7054@in.ibm.com> <31687.1186113947@kao2.melbourne.sgi.com> <20070805110746.GA12540@in.ibm.com> <46C1691E.7090708@ah.jp.nec.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <46C1691E.7090708@ah.jp.nec.com> Organization: SUSE LINUX Products GmbH User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org * Takenori Nagano [2007-08-14 10:34]: > Vivek Goyal wrote: > > On Fri, Aug 03, 2007 at 02:05:47PM +1000, Keith Owens wrote: > > > > > > To sum up, couple of options come to mind. > > > > - Register all the RAS tools on die notifier and panic > > notifier lists with fairly high priority. Export list > > of RAS tools to user space and allow users to decide the > > order of execution and priority of RAS tools. > > > > - Create a separate RAS tool notifier list (ras_tool_notifer_list). > > All the RAS tools register on this list. This list gets priority > > over die or panic notifier list. User decides the oder of execution > > of RAS tools. > > > > Here assumption is that above list will not be exported to modules. > > All the RAS tools will be in kernel and they always get a priority > > to inspect an event. > > > > What do others think? > > Very good idea. But there is a problem how to give default priority to RAS tools. > > How about priority changeable notifier_list? User can change list order > dynamically if they want. Of course, we have to give highest priority to kdump > by default. It is very useful for users who want to use some RAS tools. I think that was the idea of the first “-” (“export list of RAS tools to user space”). Thanks, Bernhard