Re: [PATCH] recordmcount: arm: Implement make_nop

2016-02-02 Thread Steven Rostedt
On Tue, 2 Feb 2016 18:31:41 +0100 Rabin Vincent wrote: > On Mon, Feb 01, 2016 at 11:44:32AM -0800, Stephen Boyd wrote: > > On 01/30, Rabin Vincent wrote: > > > For __gnu_mcount_nc, wouldn't it be better to replace both the push {lr} > > > and the bl with nop instructions, instead of keeping a (

Re: [PATCH] recordmcount: arm: Implement make_nop

2016-02-02 Thread Rabin Vincent
On Mon, Feb 01, 2016 at 11:44:32AM -0800, Stephen Boyd wrote: > On 01/30, Rabin Vincent wrote: > > For __gnu_mcount_nc, wouldn't it be better to replace both the push {lr} > > and the bl with nop instructions, instead of keeping a (useless) push + > > pop sequence? > > Agreed. I was trying to do a

Re: [PATCH] recordmcount: arm: Implement make_nop

2016-02-01 Thread Stephen Boyd
On 01/30, Rabin Vincent wrote: > On Fri, Jan 29, 2016 at 05:18:06PM -0800, Stephen Boyd wrote: > > diff --git a/scripts/recordmcount.c b/scripts/recordmcount.c > > index e167592793a7..0b16d14c54fb 100644 > > --- a/scripts/recordmcount.c > > +++ b/scripts/recordmcount.c > > @@ -206,6 +206,52 @@ stat

Re: [PATCH] recordmcount: arm: Implement make_nop

2016-01-30 Thread Rabin Vincent
On Fri, Jan 29, 2016 at 05:18:06PM -0800, Stephen Boyd wrote: > diff --git a/scripts/recordmcount.c b/scripts/recordmcount.c > index e167592793a7..0b16d14c54fb 100644 > --- a/scripts/recordmcount.c > +++ b/scripts/recordmcount.c > @@ -206,6 +206,52 @@ static int make_nop_x86(void *map, size_t const

[PATCH] recordmcount: arm: Implement make_nop

2016-01-29 Thread Stephen Boyd
In similar spirit to x86 and arm64 support, add a make_nop_arm() to replace calls to mcount with a "nop" in sections that aren't traced. Cc: Russell King Cc: Rabin Vincent Signed-off-by: Stephen Boyd --- scripts/recordmcount.c | 49 + scripts/rec