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

Nate Lawson 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 
>>> 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

-- 
Nate


More information about the cvs-src mailing list