Re: svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Benjamin Kaduk
On Mon, Feb 9, 2015 at 6:13 PM, Rui Paulo rpa...@freebsd.org 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)

Re: svn commit: r278474 - head/sys/sys

2015-02-09 Thread Rui Paulo
On Feb 09, 2015, at 03:17 PM, Jung-uk Kim j...@freebsd.org wrote: I am fine with #if defined(__clang__) || !define(_KERNEL) case.   Works for me.  Note, however, that I'm fixing the original problem in the xz repository first.  CPU_COUNT() will only be used by xz after we import 5.2.1.

Re: svn commit: r278474 - head/sys/sys

2015-02-09 Thread John Baldwin
On Monday, February 09, 2015 05:27:45 PM Jung-uk Kim wrote: On 02/09/2015 17:12, John Baldwin wrote: On Monday, February 09, 2015 04:55:52 PM Jung-uk Kim wrote: On 02/09/2015 16:08, John Baldwin wrote: On Monday, February 09, 2015 09:03:24 PM John Baldwin wrote: Author: jhb Date: Mon Feb

svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Rui Paulo
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

Re: svn commit: r278474 - head/sys/sys

2015-02-09 Thread Jung-uk Kim
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 02/09/2015 18:08, John Baldwin wrote: On Monday, February 09, 2015 05:27:45 PM Jung-uk Kim wrote: On 02/09/2015 17:12, John Baldwin wrote: On Monday, February 09, 2015 04:55:52 PM Jung-uk Kim wrote: On 02/09/2015 16:08, John Baldwin wrote:

Re: svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Rui Paulo
On Feb 09, 2015, at 03:16 PM, Benjamin Kaduk bjkf...@gmail.com wrote: On Mon, Feb 9, 2015 at 6:13 PM, Rui Paulo rpa...@freebsd.org 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

Re: svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Benjamin Kaduk
On Mon, Feb 9, 2015 at 6:22 PM, Rui Paulo rpa...@me.com wrote: On Feb 09, 2015, at 03:16 PM, Benjamin Kaduk bjkf...@gmail.com wrote: What advantage does putting this in devd have over a standalone daemon for crash reporting? Is it just the ease of implementation to leverage the existing

Re: svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Konstantin Belousov
On Mon, Feb 09, 2015 at 11:13:51PM +, 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

Re: svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Mateusz Guzik
On Mon, Feb 09, 2015 at 11:13:51PM +, Rui Paulo wrote: +notify 10 { + match system kernel; + match subsystem signal; + match typecoredump; + action logger $comm $core; +}; + */ [..] + if (vn_fullpath_global(td, p-p_textvp, fullpath,

svn commit: r278488 - head/usr.sbin/bsdconfig/usermgmt/share

2015-02-09 Thread Devin Teske
Author: dteske Date: Tue Feb 10 02:53:26 2015 New Revision: 278488 URL: https://svnweb.freebsd.org/changeset/base/278488 Log: Whitespace. MFC after:3 days Modified: head/usr.sbin/bsdconfig/usermgmt/share/user.subr Modified: head/usr.sbin/bsdconfig/usermgmt/share/user.subr

Re: svn commit: r278479 - in head: etc sys/kern

2015-02-09 Thread Rui Paulo
On Feb 9, 2015, at 19:11, Don Lewis truck...@freebsd.org wrote: On 10 Feb, Mateusz Guzik wrote: On Mon, Feb 09, 2015 at 11:13:51PM +, Rui Paulo wrote: +notify 10 { + match system kernel; + match subsystem signal; + match typecoredump; + action logger

Re: svn commit: r278494 - head/sys/kern

2015-02-09 Thread Ian Lepore
On Mon, 2015-02-09 at 20:46 -0800, Rui Paulo wrote: On Feb 9, 2015, at 20:34, Rui Paulo rpa...@freebsd.org wrote: Author: rpaulo Date: Tue Feb 10 04:34:39 2015 New Revision: 278494 URL: https://svnweb.freebsd.org/changeset/base/278494 Log: Sanitise the coredump file names sent

svn commit: r278470 - head/usr.sbin/bsdconfig/includes

2015-02-09 Thread Devin Teske
Author: dteske Date: Mon Feb 9 19:20:59 2015 New Revision: 278470 URL: https://svnweb.freebsd.org/changeset/base/278470 Log: Add new alias bsdconfig api (same as bsdconfig includes) NB: My fingers like typing api a lot more than includes MFC after:3 days Modified:

Re: svn commit: r278467 - head/usr.sbin/bsdconfig/share

2015-02-09 Thread Baptiste Daroussin
On Mon, Feb 09, 2015 at 07:16:19PM +, Devin Teske wrote: Author: dteske Date: Mon Feb 9 19:16:19 2015 New Revision: 278467 URL: https://svnweb.freebsd.org/changeset/base/278467 Log: Replace the only instance of sed(1) in bsdconfig(8) with awk(1). MFC after: 3 days

svn commit: r278494 - head/sys/kern

2015-02-09 Thread Rui Paulo
Author: rpaulo Date: Tue Feb 10 04:34:39 2015 New Revision: 278494 URL: https://svnweb.freebsd.org/changeset/base/278494 Log: Sanitise the coredump file names sent to devd. While there, add a sysctl to turn this feature off as requested by kib@. Modified: head/sys/kern/kern_sig.c

Re: svn commit: r278494 - head/sys/kern

2015-02-09 Thread Rui Paulo
On Feb 9, 2015, at 20:34, Rui Paulo rpa...@freebsd.org wrote: Author: rpaulo Date: Tue Feb 10 04:34:39 2015 New Revision: 278494 URL: https://svnweb.freebsd.org/changeset/base/278494 Log: Sanitise the coredump file names sent to devd. While there, add a sysctl to turn this feature

Re: svn commit: r278494 - head/sys/kern

2015-02-09 Thread Rui Paulo
On Feb 9, 2015, at 21:02, Ian Lepore i...@freebsd.org wrote: Or... we could consider restoring devd to its original relatively benign existance handling device-related events, and move handling of crash dumps into a separate daemon which can shoulder the burden of security for itself. At

<    1   2