* John Baldwin <j...@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
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to