Re: kern/53590: DTrace fbt_getargdesc() fails with "Unknown function"

[Andreas Gustafsson <gson%gson.org@localhost>]

The following reply was made to PR kern/53590; it has been noted by GNATS.

From: Andreas Gustafsson <gson%gson.org@localhost>
To: martin%NetBSD.org@localhost, gnats-bugs%NetBSD.org@localhost
Cc: christos%NetBSD.org@localhost, prlw1%cam.ac.uk@localhost, chuq%chuq.com@localhost
Subject: Re: kern/53590: DTrace fbt_getargdesc() fails with "Unknown function"
Date: Wed, 31 Jul 2019 13:32:48 +0300

 Last year, I wrote:
 >  Looks like something in the release build is overriding the 
 >  makeoptions DEBUG="-g" in GENERIC.
 
 I found that "something".  /etc/Makefile contains "-U DEBUG", which
 was added in the following commit by martin:
 
   revision 1.413
   date: 2014-05-05 22:10:00 +0300;  author: martin;  state: Exp;  lines: +2 -2;  commitid: TIGgqdYim0L6Nnzx;
   Use the new -U option for config to avoid generating kernel debug info
   during release builds. This should save quite some space in the evbarm
   build, where makeoptions DEBUG is heavily used.
 
 Martin, could you please revert this?  I see three reasons to do so:
 
  - To fix this PR.  This is a necessary step towards making tracing of
    function arguments and return types work in DTrace, but sadly not
    sufficient; there's still the CTF type table overflow issue
    (https://mail-index.netbsd.org/tech-kern/2017/10/13/msg022436.html).
 
  - It is really confusing that the GENERIC kernels produced by
    "build.sh release" and "build.sh kernel=GENERIC" are not equivalent
    (to the point that when I raised this question in the PR, no one
    could explain why)
 
  - It is a necessary step towards get kernel debug symbols included in
    releases as discussed in PR 50128.
 
 -- 
 Andreas Gustafsson, gson%gson.org@localhost