Date: Tue, 2 Mar 2010 09:50:40 -0800 From: Alfred Perlstein <alfred@freebsd.org> To: John Baldwin <jhb@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r204552 - in head/sys: conf kern net sys Message-ID: <20100302175040.GS55658@elvis.mu.org> In-Reply-To: <201003020756.48609.jhb@freebsd.org> References: <201003020658.o226wwB2051156@svn.freebsd.org> <201003020756.48609.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
* John Baldwin <jhb@freebsd.org> [100302 05:30] wrote: > On Tuesday 02 March 2010 1:58:58 am Alfred Perlstein wrote: > > Author: alfred > > Date: Tue Mar 2 06:58:58 2010 > > New Revision: 204552 > > URL: http://svn.freebsd.org/changeset/base/204552 > > > > Log: > > Merge projects/enhanced_coredumps (r204346) into HEAD: > > > > Enhanced process coredump routines. > > > > This brings in the following features: > > 1) Limit number of cores per process via the %I coredump formatter. > > Example: > > if corefilename is set to %N.%I.core AND num_cores = 3, then > > if a process "rpd" cores, then the corefile will be named > > "rpd.0.core", however if it cores again, then the kernel will > > generate "rpd.1.core" until we hit the limit of "num_cores". > > > > this is useful to get several corefiles, but also prevent filling > > the machine with corefiles. > > > > 2) Encode machine hostname in core dump name via %H. > > > > 3) Compress coredumps, useful for embedded platforms with limited space. > > A sysctl kern.compress_user_cores is made available if turned on. > > > > To enable compressed coredumps, the following config options need to be set: > > options COMPRESS_USER_CORES > > device zlib # brings in the zlib requirements. > > device gzio # brings in the kernel vnode gzip output module. > > > > 4) Eventhandlers are fired to indicate coredumps in progress. > > > > 5) The imgact sv_coredump routine has grown a flag to pass in more > > state, currently this is used only for passing a flag down to compress > > the coredump or not. > > > > Note that the gzio facility can be used for generic output of gzip'd > > streams via vnodes. > > > > Obtained from: Juniper Networks > > Reviewed by: kan > > > > Added: > > head/sys/kern/kern_gzio.c (contents, props changed) > > head/sys/net/zutil.h (contents, props changed) > > Modified: > > head/sys/conf/files > > head/sys/conf/options > > head/sys/kern/imgact_elf.c > > head/sys/kern/kern_sig.c > > head/sys/net/zlib.h > > head/sys/sys/eventhandler.h > > head/sys/sys/imgact.h > > head/sys/sys/imgact_aout.h > > head/sys/sys/imgact_elf.h > > head/sys/sys/sysent.h > > > > Modified: head/sys/conf/files > > ============================================================================== > > --- head/sys/conf/files Tue Mar 2 06:54:15 2010 (r204551) > > +++ head/sys/conf/files Tue Mar 2 06:58:58 2010 (r204552) > > @@ -2053,6 +2053,7 @@ kern/kern_exec.c standard > > kern/kern_exit.c standard > > kern/kern_fail.c standard > > kern/kern_fork.c standard > > +kern/kern_gzio.c optional gzio > > kern/kern_idle.c standard > > kern/kern_intr.c standard > > kern/kern_jail.c standard > > @@ -2344,7 +2345,7 @@ net/slcompress.c optional netgraph_vjc > > net/vnet.c optional vimage > > net/zlib.c optional crypto | geom_uzip | ipsec | \ > > mxge | netgraph_deflate | \ > > - ddb_ctf > > + ddb_ctf | zlib > > net80211/ieee80211.c optional wlan > > net80211/ieee80211_acl.c optional wlan wlan_acl > > net80211/ieee80211_action.c optional wlan > > Did you consider just making net/zlib.c be included if 'gzio' was defined? > Then you would just need to add 'device gzio'. This would appear to be more > consistent with the rest of the zlib-using kernel components as they all do > that rather than requiring a separate 'device zlib' line. > > -- > John Baldwin testing this now. -- - Alfred Perlstein .- AMA, VMOA #5191, 03 vmax, 92 gs500, 85 ch250 .- FreeBSD committer
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100302175040.GS55658>