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

[Kris Kennaway]

On Sun, Sep 11, 2005 at 02:49:25PM -0700, Nate Lawson wrote:
> David O'Brien wrote:
> >On Sat, Sep 10, 2005 at 06:05:00PM -0700, 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.
> >
> >
> >We do need this.  The panic I posted yesterday happened in the 'make'
> >stage of the below sequence:
> >
> >    cd /sys/<arch>/conf
> >    config FOO
> >    cd ../compile/FOO
> >    make clean
> >    make
> >
> >Note that the kernel.debug matching the running kernel was blown away at
> >this point.
> >
> >I was going to just install kernel.sym instead of a kernel with symbols,
> >but kgdb's usage doesn't claim to support '-s' as the previous kgdb did.
> >
> 
> kernel.sym would be better since it wouldn't have the duplication that 
> kernel and kernel.debug have.  Alternately, we can make loader(8) skip 
> symbol sections when booting and just install kernel.debug

It does skip symbol sections.

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-src/attachments/20050911/5a9dd82f/attachment-0001.bin