Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 10 Feb 2015 07:06:03 -0800
From:      Adrian Chadd <adrian@freebsd.org>
To:        John Baldwin <jhb@freebsd.org>
Cc:        "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, Rui Paulo <rpaulo@freebsd.org>
Subject:   Re: svn commit: r278479 - in head: etc sys/kern
Message-ID:  <CAJ-Vmomm=vZtaYRXjo-Oyw0cAjYaa1dQHLPPGc4KT6txTuQzRw@mail.gmail.com>
In-Reply-To: <1516483.e0EXgdk9ur@ralph.baldwin.cx>
References:  <201502092313.t19NDpoS083043@svn.freebsd.org> <1516483.e0EXgdk9ur@ralph.baldwin.cx>

next in thread | previous in thread | raw e-mail | index | archive | help
On 10 February 2015 at 06:16, John Baldwin <jhb@freebsd.org> wrote:
> On Monday, February 09, 2015 11:13:51 PM Rui Paulo wrote:
>> Author: rpaulo
>> Date: Mon Feb  9 23:13:50 2015
>> New Revision: 278479
>> URL: https://svnweb.freebsd.org/changeset/base/278479
>>
>> Log:
>>   Notify devd(8) when a process crashed.
>>
>>   This change implements a notification (via devctl) to userland when
>>   the kernel produces coredumps after a process has crashed.
>>   devd can then run a specific command to produce a human readable crash
>>   report.  The command is most usually a helper that runs gdb/lldb
>>   commands on the file/coredump pair.  It's possible to use this
>>   functionality for implementing automatic generation of crash reports.
>>
>>   devd(8) will be notified of the full path of the binary that crashed and
>>   the full path of the coredump file.
>
> I think this is a very useful feature and I think this is fine to be in the
> tree as-is for now.  My only note is that this is a bit of feature creep for
> devd (this isn't a device notification, this is a system event notification).
> As such, I think it might be worth thinking if we (collectively) want to think
> about having a separate framework at all for system event notification.  You
> could possibly publish other interesting events this way.  For example, Isilon
> currently has a patch to log(9) Witness LORs.  I personally think it's a bit
> hackish and potentially unreliable.  A much nicer interface if you want to
> capture such things would be to publish an event for each logged LOR instead.
> Machine checks are another example of something that might be nice to publish
> (though you could possibly make the case that those would not be inappropriate
> to publish via devd since actual hardware is involved).  Disk and PCI errors
> are another class of thing that it would be nice to publish in an easier to
> programmaticaly parse manner.

Cool, so someone's going to add multi-subscriber support to /dev/devctl ?

I think devd grows these things because it's easier than teaching the
devctl interface to support multiple listeners.




-adrian



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-Vmomm=vZtaYRXjo-Oyw0cAjYaa1dQHLPPGc4KT6txTuQzRw>