[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-23 Thread Simon Kågström
On 2015-06-23 09:47, Thomas Monjalon wrote: > 2015-06-23 08:39, Gonzalez Monroy, Sergio: >> I guess you could argue that, to always build with debug info then strip >> it down. >> You would need another flag to strip debug info for production, or leave >> it for debugging. >> >> In my opinion is

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-23 Thread Thomas Monjalon
2015-06-23 08:39, Gonzalez Monroy, Sergio: > On 22/06/2015 08:56, Simon K?gstr?m 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 conf

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-23 Thread Gonzalez Monroy, Sergio
On 22/06/2015 08:56, Simon K?gstr?m 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. >>> >>>

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-22 Thread Simon Kågström
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) >> +TOOLCHA

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-22 Thread Cyril Chemparathy
On Mon, 22 Jun 2015 08:44:41 +0100 "Gonzalez Monroy, Sergio" wrote: > 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=x86_64-nat

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-22 Thread Gonzalez Monroy, Sergio
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. > > Note: This patch does not include corresponding changes for ICC. The > author pleads abject ignorance in this re

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-19 Thread Cyril Chemparathy
From: Cyril Chemparathy It is often useful to build with debug enabled, we add a config (CONFIG_RTE_TOOLCHAIN_DEBUG) to do so. Note: This patch does not include corresponding changes for ICC. The author pleads abject ignorance in this regard, and welcomes recommendations. :-) Change-Id: I499e5

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-04 Thread Olivier MATZ
Hi Marc, On 03/03/2015 02:27 PM, Marc Sune wrote: > > On 03/03/15 14:03, Bruce Richardson wrote: >> On Tue, Mar 03, 2015 at 01:56:19PM +0100, Marc Sune wrote: >> [...] >> I believe that the global option of overriding the CFLAGS is already >> sufficiently >> covered - including being documented in

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Ananyev, Konstantin
> -Original Message- > From: Marc Sune [mailto:marc.sune at bisdn.de] > Sent: Tuesday, March 03, 2015 2:39 PM > To: Ananyev, Konstantin; Richardson, Bruce > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] mk: add support for gdb debug info generation > &

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Marc Sune
On 03/03/15 14:31, Ananyev, Konstantin wrote: > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson >> Sent: Tuesday, March 03, 2015 1:03 PM >> To: Marc Sune >> Cc: dev at dpdk.org >> Subject: Re: [dp

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Panu Matilainen
On 03/03/2015 02:19 PM, Marc Sune wrote: > > On 03/03/15 10:33, Bruce Richardson wrote: >> On Mon, Mar 02, 2015 at 06:32:13PM +0100, Marc Sune wrote: >>> On 22/02/15 12:51, Marc Sune wrote: I don't like the proposed patch, but I am recovering this old thread because I agree on the problem

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Thomas Monjalon
2015-03-03 13:03, Bruce Richardson: > On Tue, Mar 03, 2015 at 01:56:19PM +0100, Marc Sune wrote: > > On 03/03/15 13:40, Panu Matilainen wrote: > > >My 5c is that if anything, DPDK needs *less* places that muck around with > > >compiler flags, not more. If you something like this for all the librari

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Marc Sune
On 03/03/15 14:03, Bruce Richardson wrote: > On Tue, Mar 03, 2015 at 01:56:19PM +0100, Marc Sune wrote: >> On 03/03/15 13:40, Panu Matilainen wrote: >>> On 03/03/2015 02:19 PM, Marc Sune wrote: On 03/03/15 10:33, Bruce Richardson wrote: > On Mon, Mar 02, 2015 at 06:32:13PM +0100, Marc Sun

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Marc Sune
On 03/03/15 13:40, Panu Matilainen wrote: > On 03/03/2015 02:19 PM, Marc Sune wrote: >> >> On 03/03/15 10:33, Bruce Richardson wrote: >>> On Mon, Mar 02, 2015 at 06:32:13PM +0100, Marc Sune wrote: On 22/02/15 12:51, Marc Sune wrote: > I don't like the proposed patch, but I am recovering t

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > Sent: Tuesday, March 03, 2015 1:03 PM > To: Marc Sune > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] mk: add support for gdb debug info generation > > On Tu

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Marc Sune
, to have a global "DEBUG" knob, which would enable to compile the entire DPDK library code with -O0 -g and possibly also with -fno-inline. This would also help debugging the inline functions. Marc > > /Bruce > >>> Marc >>> >>>> Thanks >>>

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Bruce Richardson
On Tue, Mar 03, 2015 at 01:56:19PM +0100, Marc Sune wrote: > > On 03/03/15 13:40, Panu Matilainen wrote: > >On 03/03/2015 02:19 PM, Marc Sune wrote: > >> > >>On 03/03/15 10:33, Bruce Richardson wrote: > >>>On Mon, Mar 02, 2015 at 06:32:13PM +0100, Marc Sune wrote: > On 22/02/15 12:51, Marc Sun

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-03 Thread Bruce Richardson
the library. /Bruce > > > >Marc > > > >>Thanks > >>Konstantin > >> > >>-Original Message- > >>From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cyril Chemparathy > >>Sent: Thursday, April 03, 2014 6:31 PM &g

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-03-02 Thread Marc Sune
ssage----- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cyril Chemparathy >> Sent: Thursday, April 03, 2014 6:31 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH] mk: add support for gdb debug info >> generation >> >> It is often useful

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-02-22 Thread Marc Sune
m: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cyril Chemparathy > Sent: Thursday, April 03, 2014 6:31 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] mk: add support for gdb debug info generation > > It is often useful to build with debug enabled, we add a config > (CONFI

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2014-04-04 Thread Ananyev, Konstantin
Of Cyril Chemparathy Sent: Thursday, April 03, 2014 6:31 PM To: dev at dpdk.org Subject: [dpdk-dev] [PATCH] mk: add support for gdb debug info generation It is often useful to build with debug enabled, we add a config (CONFIG_RTE_TOOLCHAIN_DEBUG) to do so. Note: This patch does not include correspon

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2014-04-03 Thread Cyril Chemparathy
It is often useful to build with debug enabled, we add a config (CONFIG_RTE_TOOLCHAIN_DEBUG) to do so. Note: This patch does not include corresponding changes for ICC. The author pleads abject ignorance in this regard, and welcomes recommendations. :-) Signed-off-by: Cyril Chemparathy --- conf