Re: [PATCH 01/10] Makefile: Add option CONFIG_DISABLE_GCC_AUTOMATIC_INLINING

2012-09-13 Thread Ezequiel Garcia
On Thu, Sep 13, 2012 at 4:17 AM, Michal Marek wrote: > Dne 13.9.2012 02:30, Ezequiel Garcia napsal(a): >> Hi, >> >> On Sun, Sep 9, 2012 at 6:25 PM, David Rientjes wrote: >>> On Sat, 8 Sep 2012, Ezequiel Garcia wrote: >>> diff --git a/Makefile b/Makefile index ddf5be9..df6045a 100644 >>>

Re: [PATCH 01/10] Makefile: Add option CONFIG_DISABLE_GCC_AUTOMATIC_INLINING

2012-09-13 Thread Michal Marek
Dne 13.9.2012 02:30, Ezequiel Garcia napsal(a): > Hi, > > On Sun, Sep 9, 2012 at 6:25 PM, David Rientjes wrote: >> On Sat, 8 Sep 2012, Ezequiel Garcia wrote: >> >>> diff --git a/Makefile b/Makefile >>> index ddf5be9..df6045a 100644 >>> --- a/Makefile >>> +++ b/Makefile >>> @@ -561,6 +561,10 @@ el

Re: [PATCH 01/10] Makefile: Add option CONFIG_DISABLE_GCC_AUTOMATIC_INLINING

2012-09-12 Thread Ezequiel Garcia
Hi, On Sun, Sep 9, 2012 at 6:25 PM, David Rientjes wrote: > On Sat, 8 Sep 2012, Ezequiel Garcia wrote: > >> diff --git a/Makefile b/Makefile >> index ddf5be9..df6045a 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -561,6 +561,10 @@ else >> KBUILD_CFLAGS+= -O2 >> endif >> >> +ifdef CON

Re: [PATCH 01/10] Makefile: Add option CONFIG_DISABLE_GCC_AUTOMATIC_INLINING

2012-09-09 Thread David Rientjes
On Sat, 8 Sep 2012, Ezequiel Garcia wrote: > diff --git a/Makefile b/Makefile > index ddf5be9..df6045a 100644 > --- a/Makefile > +++ b/Makefile > @@ -561,6 +561,10 @@ else > KBUILD_CFLAGS+= -O2 > endif > > +ifdef CONFIG_DISABLE_GCC_AUTOMATIC_INLINING > +KBUILD_CFLAGS+= -fno-inl

Re: [PATCH 01/10] Makefile: Add option CONFIG_DISABLE_GCC_AUTOMATIC_INLINING

2012-09-08 Thread Sam Ravnborg
On Sat, Sep 08, 2012 at 05:47:50PM -0300, Ezequiel Garcia wrote: > As its name suggest this option prevents gcc from auto inlining > small functions. This is very important if one wants to obtain > traces with accurate call sites. > > Without this option, gcc will collapse some small functions, >

[PATCH 01/10] Makefile: Add option CONFIG_DISABLE_GCC_AUTOMATIC_INLINING

2012-09-08 Thread Ezequiel Garcia
As its name suggest this option prevents gcc from auto inlining small functions. This is very important if one wants to obtain traces with accurate call sites. Without this option, gcc will collapse some small functions, even when not marked as 'inline' thus making impossible to correlate the trac