Re: [PATCH v1 1/6] Add oops notification chain.

From: Mike Waychison
Date: Wed Jan 26 2011 - 16:50:35 EST


On Tue, Jan 25, 2011 at 6:48 PM, Greg KH <greg@xxxxxxxxx> wrote:
> On Tue, Jan 25, 2011 at 02:21:03PM -0800, Aaron Durbin wrote:
>> As Mike talked about it might be more desirable to actually pass a
>> reason field that indicates the appropriate kernel event. However, the
>> naming of some of the variables becomes less clear. Maybe name it
>> panic_oops_notifier_list? That way there would only be a single
>> notifier chain.
>
> Yes, I think that's the better thing to do here instead of adding more
> notifier chains.  It would just start to get even messier than it
> currently is :)

Hmm. I'm going to try and just re-use the notify_die() bits here
instead which seems to do what I want. The overloading of "die" is a
bit of a mess, which is probably why I missed it the first time
around.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/