From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH 28/28] Separate runtime debug output from debug symbols Date: Thu, 2 Oct 2014 17:52:25 +0100 Message-ID: <542D82C9.5010501@citrix.com> References: <1412181690-19609-1-git-send-email-olaf@aepfle.de> <1412181690-19609-29-git-send-email-olaf@aepfle.de> <542D1A5F020000780003BCCF@mail.emea.novell.com> <20141002080023.GA30907@aepfle.de> <542D23C1020000780003BCFF@mail.emea.novell.com> <20141002095229.GA43394@deinos.phlegethon.org> <542D44A0020000780003BDDD@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <542D44A0020000780003BDDD@mail.emea.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich , Tim Deegan Cc: Olaf Hering , Wei Liu , Ian Campbell , Stefano Stabellini , Ian Jackson , xen-devel@lists.xen.org, Samuel Thibault , Keir Fraser , David Scott List-Id: xen-devel@lists.xenproject.org On 02/10/14 11:27, Jan Beulich wrote: >>>> On 02.10.14 at 11:52, wrote: >> I think Andrew is talking about the symbol table rather than the debug >> info - xen's backtrace walker doesn't speak DWARF. AFAICT the symbol >> tables generated without '-g' are just fine for xen's own use printing >> backtraces &c, > Right. > >> though possibly post-mortem tools for crash analysis >> might need more? > Perhaps. In which case people interested in this can enable > debug info generation (and iirc Olaf's patch also doesn't change > any defaults here). > > Jan > After some investigation of my own, I have confirmed that neither System.map nor xen.gz are altered. I do find it somewhat silly to generate a xen-syms without debugging symbols, but as that doesn't affect the eventual xen.gz, I am less worried about it. However, you may have noticed that there are actually two -g's on the Xen build command line, the first of which is directly inherited directly from the root Config.mk. Therefore, I think you can just unilaterally drop the -g from xen/Rules.mk ~Andrew