kdb
[Top] [All Lists]

Re: [PATCH 0/2] add new notifier function ,take2

To: Nick Piggin <nickpiggin@xxxxxxxxxxxx>
Subject: Re: [PATCH 0/2] add new notifier function ,take2
From: Takenori Nagano <t-nagano@xxxxxxxxxxxxx>
Date: Thu, 25 Oct 2007 15:48:01 +0900
Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, vgoyal@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>, k-miyoshi@xxxxxxxxxxxxx, kexec@xxxxxxxxxxxxxxxxxxx, Bernhard Walle <bwalle@xxxxxxx>, Keith Owens <kaos@xxxxxxxxxx>, kdb@xxxxxxxxxxx
In-reply-to: <200710212200.04361.nickpiggin@xxxxxxxxxxxx>
References: <471700F4.1080200@xxxxxxxxxxxxx> <20071018080621.GB8779@xxxxxxxxxx> <47171ED9.7010907@xxxxxxxxxxxxx> <200710212200.04361.nickpiggin@xxxxxxxxxxxx>
Sender: kdb-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.6 (Windows/20070728)
Nick Piggin wrote:
>>> Is it possible to use a single bit of common code and a single
>>> notifier for these things? Or is it too difficult?
>> >
>> > I'm sorry, I can't understand your image well. I'd like to know details of
>> > your image.
> 
> Rather than have each of "RAS tools" have their own notifier, and have
> the user specify the priority of the notifiers, introduce some layer
> which _knows_ that, for example, only one of these subsystems will be
> called (it could arbitrate, perhaps distinguish between destructive and
> non-destructive ones). It would need only a single notifier, but would
> then have a specific way of calling into the ras modules.
> 
> Does this make sense? I guess it is a lot more work to do, so maybe your
> solution is the best one for now.

Hi Nick,

Thank you for your explanation. I understand. :-)

This is crash_stop (the common infrastructure for debug tools) by Keith Owens.
http://www.mail-archive.com/linux-arch@xxxxxxxxxxxxxxx/msg01929.html

Is it same as your idea? I think it is very nice solution for debug tools
conflict problem.

By the way, on old notify_chain, if admin wants to change the list order, admin
have to recompile the kernel. My patches add new *generic* notify_chain which
admin can modify the list order. My patches are not only for RAS tools problem.

I'm happy if both patches are merged into mainline. :-)

Thanks,
  Takenori
---------------------------
Use http://oss.sgi.com/ecartis to modify your settings or to unsubscribe.
<Prev in Thread] Current Thread [Next in Thread>