From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wg0-f42.google.com (mail-wg0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id 28F22C790 for ; Tue, 23 Jun 2015 09:48:19 +0200 (CEST) Received: by wgbhy7 with SMTP id hy7so1884091wgb.2 for ; Tue, 23 Jun 2015 00:48:19 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=pPkqcpjrxzd/TDv52o0HONkF0sjF/tYlvpVh6AbxLFk=; b=dj6dRzwgx1d6tQqua8QR0wqy/TtcTcSts+oLlolGVISYStsMJx8nHcHCrmDa8b9rZM mGvdMr+zlaojLi9VkbAl2f9oOF7oRXLNRloEmafsSlWeESvlW9UokCrHNFQ/kUm8eL24 ZweXQcHvGE+w1wB/v79NeOjM1e15OBT27X0MlHPKwohWdA5e7ysC+fKI0YggLVnUdJDY IlaKxgfb0m+xDLP82+QVtr+12xLZo1b22rUmu4UZXDVFYvkd+wDDK7bPkhvWyMxcL1vz eXGUooSxaAayBTOGOJF22nhfa9gzz/dXic/KhmJVa8nz/bVrQ+eBwKUrQeWLZ+iojRmu o3Ig== X-Gm-Message-State: ALoCoQkL1FmAZnWpMT0LCVvCBF/PNiqtiVlRItxI+C9OAjgGlF1LOLJSKh5HZB4A4Z42AoVuestU X-Received: by 10.180.83.40 with SMTP id n8mr866172wiy.57.1435045699040; Tue, 23 Jun 2015 00:48:19 -0700 (PDT) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id pd7sm34276367wjb.27.2015.06.23.00.48.17 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 23 Jun 2015 00:48:17 -0700 (PDT) From: Thomas Monjalon To: "Gonzalez Monroy, Sergio" Date: Tue, 23 Jun 2015 09:47:15 +0200 Message-ID: <28151366.mYixNO6eb8@xps13> Organization: 6WIND User-Agent: KMail/4.14.8 (Linux/4.0.4-2-ARCH; KDE/4.14.8; x86_64; ; ) In-Reply-To: <55890D3A.109@intel.com> References: <1434749378-8578-1-git-send-email-cchemparathy@ezchip.com> <5587BFA7.2000206@netinsight.net> <55890D3A.109@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] mk: add support for gdb debug info generation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 23 Jun 2015 07:48:19 -0000 2015-06-23 08:39, Gonzalez Monroy, Sergio: > On 22/06/2015 08:56, Simon K=E5gstr=F6m wrote: > > On 2015-06-22 09:44, Gonzalez Monroy, Sergio wrote: > >> On 19/06/2015 22:29, Cyril Chemparathy wrote: > >>> From: Cyril Chemparathy > >>> > >>> It is often useful to build with debug enabled, we add a config > >>> (CONFIG_RTE_TOOLCHAIN_DEBUG) to do so. > >>> > >>> +ifeq ($(CONFIG_RTE_TOOLCHAIN_DEBUG),y) > >>> +TOOLCHAIN_CFLAGS +=3D -g -ggdb > >>> +TOOLCHAIN_LDFLAGS +=3D -g -ggdb > >>> +endif > >> I don't think you need to modify the makefiles and introduce a new= > >> compile time option for this. > >> The same result can be easily achieved by setting EXTRA_CFLAGS in = the > >> command line. ie: > >> $ make install T=3Dx86_64-native-linuxapp-gcc EXTRA_CFLAGS=3D= '-g -ggdb' > > Why isn't -g standard though? The binaries should/will anyhow be > > stripped when used for production - but debugging information shoul= d be > > useful when analysing crashes. >=20 > I guess you could argue that, to always build with debug info then st= rip=20 > it down. > You would need another flag to strip debug info for production, or le= ave=20 > it for debugging. >=20 > In my opinion is not worth it, but it you feel strongly about it you = can=20 > submit patches and > let the community decide. I think stripping is a packaging responsibility. It would be a good idea to always provide debugging symbols.