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

2007-10-24 Thread Takenori Nagano
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

Re: sysfs sys/kernel/ namespace (was Re: [PATCH 0/2] add new notifier function ,take2)

2007-10-24 Thread Nick Piggin
On Thursday 25 October 2007 15:45, Greg KH wrote: > On Thu, Oct 25, 2007 at 12:31:06PM +1000, Nick Piggin wrote: > > On Wednesday 24 October 2007 21:12, Kay Sievers wrote: > > > On 10/24/07, Nick Piggin <[EMAIL PROTECTED]> wrote: > > > It was intended to be something like /proc/sys/kernel/ only. >

Re: sysfs sys/kernel/ namespace (was Re: [PATCH 0/2] add new notifier function ,take2)

2007-10-24 Thread Greg KH
On Thu, Oct 25, 2007 at 12:31:06PM +1000, Nick Piggin wrote: > On Wednesday 24 October 2007 21:12, Kay Sievers wrote: > > On 10/24/07, Nick Piggin <[EMAIL PROTECTED]> wrote: > > > On Tuesday 23 October 2007 10:55, Takenori Nagano wrote: > > > > Nick Piggin wrote: > > > > > One thing I'd suggest is

Re: sysfs sys/kernel/ namespace (was Re: [PATCH 0/2] add new notifier function ,take2)

2007-10-24 Thread Nick Piggin
On Wednesday 24 October 2007 21:12, Kay Sievers wrote: > On 10/24/07, Nick Piggin <[EMAIL PROTECTED]> wrote: > > On Tuesday 23 October 2007 10:55, Takenori Nagano wrote: > > > Nick Piggin wrote: > > > > One thing I'd suggest is not to use debugfs, if it is going to > > > > be a useful end-user feat

Re: sysfs sys/kernel/ namespace (was Re: [PATCH 0/2] add new notifier function ,take2)

2007-10-24 Thread Kay Sievers
On 10/24/07, Nick Piggin <[EMAIL PROTECTED]> wrote: > On Tuesday 23 October 2007 10:55, Takenori Nagano wrote: > > Nick Piggin wrote: > > > > One thing I'd suggest is not to use debugfs, if it is going to > > > be a useful end-user feature. > > > > Is /sys/kernel/notifier_name/ an appropriate place

sysfs sys/kernel/ namespace (was Re: [PATCH 0/2] add new notifier function ,take2)

2007-10-23 Thread Nick Piggin
On Tuesday 23 October 2007 10:55, Takenori Nagano wrote: > Nick Piggin wrote: > > One thing I'd suggest is not to use debugfs, if it is going to > > be a useful end-user feature. > > Is /sys/kernel/notifier_name/ an appropriate place? Hi list, I'm curious about the /sys/kernel/ namespace. I had

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

2007-10-21 Thread Nick Piggin
On Thursday 18 October 2007 18:52, Takenori Nagano wrote: > Vivek Goyal wrote: > > > My stance is that _all_ the RAS tools (kdb, kgdb, nlkd, netdump, lkcd, > > > crash, kdump etc.) should be using a common interface that safely puts > > > the entire system in a stopped state and saves the state of

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

2007-10-18 Thread Takenori Nagano
Vivek Goyal wrote: > On Thu, Oct 18, 2007 at 12:06:51AM -0700, Andrew Morton wrote: >> On Thu, 18 Oct 2007 15:45:08 +0900 Takenori Nagano <[EMAIL PROTECTED]> wrote: >> I can sort-of see what this is doing. Runtime-definable management of >> which notifier functions will be called on a panic? Or m

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

2007-10-18 Thread Vivek Goyal
On Thu, Oct 18, 2007 at 12:06:51AM -0700, Andrew Morton wrote: > On Thu, 18 Oct 2007 15:45:08 +0900 Takenori Nagano <[EMAIL PROTECTED]> wrote: > > > Hi, > > > > A big thanks to everybody who read and replied to first version. I have > > tried to > > incorporate reviewer's comments and suggestion

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

2007-10-18 Thread Andrew Morton
On Thu, 18 Oct 2007 15:45:08 +0900 Takenori Nagano <[EMAIL PROTECTED]> wrote: > Hi, > > A big thanks to everybody who read and replied to first version. I have tried > to > incorporate reviewer's comments and suggestions. > > changelog take1 -> take2 > > - Rebased 2.6.23 > - comment updated >

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

2007-10-17 Thread Takenori Nagano
Hi, A big thanks to everybody who read and replied to first version. I have tried to incorporate reviewer's comments and suggestions. changelog take1 -> take2 - Rebased 2.6.23 - comment updated - renamed the notifiner name "tunable_notifier" to "tunable_atomic_notifier" - fixed typo These patc

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

2007-10-09 Thread Takenori Nagano
Eric W. Biederman wrote: > Takenori Nagano <[EMAIL PROTECTED]> writes: > >> Hi, >> >> These patches add new notifier function and implement it to >> panic_notifier_list. >> We used the hardcoded notifier chain so far, but it was not flexible. New >> notifier is very flexible, because user can cha

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

2007-10-05 Thread Eric W. Biederman
Takenori Nagano <[EMAIL PROTECTED]> writes: > Hi, > > These patches add new notifier function and implement it to > panic_notifier_list. > We used the hardcoded notifier chain so far, but it was not flexible. New > notifier is very flexible, because user can change a list of order by debugfs. Ho

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

2007-10-04 Thread Takenori Nagano
Vivek Goyal wrote: > On Thu, Oct 04, 2007 at 08:38:05PM +0900, Takenori Nagano wrote: > > In summary, right now co-existence of kdb with kdump seems to be your pain > point. I would prefer that kdb just puts a break point on panic() and we move > on. If there are more candidates down the line and t

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

2007-10-04 Thread Vivek Goyal
On Thu, Oct 04, 2007 at 08:38:05PM +0900, Takenori Nagano wrote: > Hi, > > These patches add new notifier function and implement it to > panic_notifier_list. > We used the hardcoded notifier chain so far, but it was not flexible. New > notifier is very flexible, because user can change a list of

[PATCH 0/2] add new notifier function

2007-10-04 Thread Takenori Nagano
Hi, These patches add new notifier function and implement it to panic_notifier_list. We used the hardcoded notifier chain so far, but it was not flexible. New notifier is very flexible, because user can change a list of order by debugfs. Please review, and give some comments. Thanks, Example)