Re: [PATCH] ARM: sched_clock: Add more notrace to prevent recursion

2013-04-18 Thread Russell King - ARM Linux
On Wed, Apr 17, 2013 at 05:34:45PM -0700, Stephen Boyd wrote: > On 03/26/13 10:35, Stephen Boyd wrote: > > On 03/21/13 10:49, Stephen Boyd wrote: > >> On 03/14/13 17:08, Stephen Boyd wrote: > >>> cyc_to_sched_clock() is called by sched_clock() and cyc_to_ns() > >>> is called by cyc_to_sched_clock()

Re: [PATCH] ARM: sched_clock: Add more notrace to prevent recursion

2013-04-17 Thread Stephen Boyd
On 03/26/13 10:35, Stephen Boyd wrote: > On 03/21/13 10:49, Stephen Boyd wrote: >> On 03/14/13 17:08, Stephen Boyd wrote: >>> cyc_to_sched_clock() is called by sched_clock() and cyc_to_ns() >>> is called by cyc_to_sched_clock(). I suspect that some compilers >>> inline both of these functions into

Re: [PATCH] ARM: sched_clock: Add more notrace to prevent recursion

2013-03-26 Thread Stephen Boyd
On 03/21/13 10:49, Stephen Boyd wrote: > On 03/14/13 17:08, Stephen Boyd wrote: >> cyc_to_sched_clock() is called by sched_clock() and cyc_to_ns() >> is called by cyc_to_sched_clock(). I suspect that some compilers >> inline both of these functions into sched_clock() and so we've >> been getting aw

Re: [PATCH] ARM: sched_clock: Add more notrace to prevent recursion

2013-03-21 Thread Stephen Boyd
On 03/14/13 17:08, Stephen Boyd wrote: > cyc_to_sched_clock() is called by sched_clock() and cyc_to_ns() > is called by cyc_to_sched_clock(). I suspect that some compilers > inline both of these functions into sched_clock() and so we've > been getting away without having a notrace marking. It seems

[PATCH] ARM: sched_clock: Add more notrace to prevent recursion

2013-03-14 Thread Stephen Boyd
cyc_to_sched_clock() is called by sched_clock() and cyc_to_ns() is called by cyc_to_sched_clock(). I suspect that some compilers inline both of these functions into sched_clock() and so we've been getting away without having a notrace marking. It seems that my compiler isn't inlining cyc_to_sched_c