Date: Sun, 11 Sep 2005 20:58:22 +1000 From: Peter Jeremy <peterjeremy@optushome.com.au> To: Nate Lawson <nate@root.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org, "David E. O'Brien" <obrien@freebsd.org> Subject: Re: cvs commit: src/sys/conf kern.post.mk Message-ID: <20050911105821.GA91588@server.vk2pj.dyndns.org> In-Reply-To: <432382BC.5080105@root.org> References: <20050911002229.51F4916A471@hub.freebsd.org> <432382BC.5080105@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Sep 10, 2005 at 06:05:00PM -0700, Nate Lawson wrote: >David E. O'Brien wrote: >> 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. > >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. I'd agree with Nate here. I agree that the debug kernel is worth keeping but it's not needed in /. If you want to install it, how about installing it in /var/crash. A number of people have mentioned losing the debug kernel when they do a rebuild. The normal reasons for doing a rebuild are either that you have updated your source or you've changed your config. In the former case, the old debug kernel loses some of its validity because it no longer matches the source code it was built from. In the latter case, I would suggest that the only reason to change the kernel config is if you are planning on installing a new kernel, so again you don't need the old debug kernel. This just leaves the relatively small window whilst you are building and installing the new kernel - though David has found that this window can unfortunately be hit. In this change intended for -current only or will it remain in 7-RELEASE? -- Peter Jeremy
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050911105821.GA91588>