Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-19 Thread Michael Ellerman
Balbir Singh writes: > On Wed, 2017-04-19 at 21:13 +1000, Anton Blanchard wrote: >> Hi Balbir, >> >> > > FTRACE is quite CPU consumming, shouldn't it really be on by >> > > default ? >> > >> > It does some work at boot to NOP out function entry points at _mcount >> >

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-19 Thread Balbir Singh
On Wed, 2017-04-19 at 21:13 +1000, Anton Blanchard wrote: > Hi Balbir, > > > > FTRACE is quite CPU consumming, shouldn't it really be on by > > > default ? > > > > It does some work at boot to NOP out function entry points at _mcount > > locations. Is that what you are referring to? Or the

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-19 Thread Anton Blanchard
Hi Balbir, > > FTRACE is quite CPU consumming, shouldn't it really be on by > > default ? > > It does some work at boot to NOP out function entry points at _mcount > locations. Is that what you are referring to? Or the overhead of the > code in terms of size? Most distro kernels have tracing

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-13 Thread Naveen N. Rao
Excerpts from Balbir Singh's message of April 13, 2017 12:33: We expect to have these configs on by default, most distros turn them off, its always good to have them on so that we can use them and hopefully not break them Thanks! I find myself constantly enabling these options.

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-13 Thread Balbir Singh
On Thu, Apr 13, 2017 at 6:41 PM, Christophe LEROY wrote: > > > Le 13/04/2017 à 10:19, Balbir Singh a écrit : >> >> On Thu, 2017-04-13 at 09:24 +0200, Christophe LEROY wrote: >>> >>> Hi Baldir >> >> >> Hi Christophe, It's Balbir > > > Sorry > No worries! >> >>> >>> Le

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-13 Thread Christophe LEROY
Le 13/04/2017 à 10:19, Balbir Singh a écrit : On Thu, 2017-04-13 at 09:24 +0200, Christophe LEROY wrote: Hi Baldir Hi Christophe, It's Balbir Sorry Le 13/04/2017 à 09:03, Balbir Singh a écrit : We expect to have these configs on by default, most distros turn them off, its always

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-13 Thread Balbir Singh
On Thu, 2017-04-13 at 09:24 +0200, Christophe LEROY wrote: > Hi Baldir Hi Christophe, It's Balbir > > Le 13/04/2017 à 09:03, Balbir Singh a écrit : > > We expect to have these configs on by default, most > > distros turn them off, its always good to have them on > > so that we can use them and

Re: [PATCH] powerpc/configs: Enable function trace by default

2017-04-13 Thread Christophe LEROY
Hi Baldir Le 13/04/2017 à 09:03, Balbir Singh a écrit : We expect to have these configs on by default, most distros turn them off, its always good to have them on so that we can use them and hopefully not break them Isn't it the purpose of the target allyesconfig to allow such tests ? FTRACE

[PATCH] powerpc/configs: Enable function trace by default

2017-04-13 Thread Balbir Singh
We expect to have these configs on by default, most distros turn them off, its always good to have them on so that we can use them and hopefully not break them Signed-off-by: Balbir Singh --- arch/powerpc/configs/powernv_defconfig | 4