cvs commit: src/sys/conf kern.post.mk

Kris Kennaway kris at obsecurity.org
Sat Sep 10 19:29:08 PDT 2005


On Sat, Sep 10, 2005 at 10:01:15PM -0400, Garance A Drosihn wrote:
> At 6:05 PM -0700 9/10/05, Nate Lawson wrote:
> >David E. O'Brien wrote:
> >>obrien      2005-09-11 00:22:21 UTC
> >>
> >>  FreeBSD src repository
> >>
> >>  Modified files:
> >>    sys/conf             kern.post.mk   Log:
> >>  For HEAD, install a kernel with debug information if DEBUG is a kernel
> >>  config option.  It is too easy to loose the build directory and not have
> >>  symbols for kgdb to read.
> >>    Revision  Changes    Path
> >>  1.84      +4 -17     src/sys/conf/kern.post.mk
> >
> >I disagree with this change.  We do not need to waste the space
> >in /. If I'm running a debug kernel, it is based on the latest
> >version of kernel.debug in my kernel compile dir and I know to
> >find it there.
> 
> Fwiw, I've been burned by building a debug kernel, only to have
> removed the original compile-directory for that kernel by the time I
> actually *needed* the debug symbols.  It's one thing if you're building
> a debug kernel because you know you're going to spend the next hour
> debugging some change.  It's another if you're building a debug kernel
> because your machine might panic sometime in the next two or three
> weeks.

Likewise, I also find this change very useful.  When I'm juggling a
few dozen panics on a few dozen machines with a few dozen different
customized source trees, it's hard to keep track of all the
kernel.debugs.  Now I don't have to.

Also, now we won't have to throw away panic reports from users who
didn't think to save their kernel.debug, and the quality of bug
reports can increase.

Kris
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20050910/75c5eaeb/attachment.bin


More information about the cvs-all mailing list