cvs commit: src/sys/conf kern.post.mk
nate at root.org
Sun Sep 11 14:49:27 PDT 2005
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
>>> 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
> 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
More information about the cvs-all