Re: [PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-19 Thread Doug Anderson
Daniel, On Tue, Jun 17, 2014 at 5:13 AM, Daniel Lezcano daniel.lezc...@linaro.org wrote: On 06/04/2014 07:30 PM, Doug Anderson wrote: In (93bfb76 clocksource: exynos_mct: register sched_clock callback) we supported using the MCT as a scheduler clock. We properly marked

Re: [PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-17 Thread Daniel Lezcano
On 06/04/2014 07:30 PM, Doug Anderson wrote: In (93bfb76 clocksource: exynos_mct: register sched_clock callback) we supported using the MCT as a scheduler clock. We properly marked exynos4_read_sched_clock() as notrace. However, we then went and called another function that _wasn't_ notrace.

Re: [PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-16 Thread Daniel Lezcano
On 06/16/2014 06:40 AM, Doug Anderson wrote: Daniel, On Sun, Jun 15, 2014 at 2:18 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: On 06/04/2014 07:30 PM, Doug Anderson wrote: In (93bfb76 clocksource: exynos_mct: register sched_clock callback) we supported using the MCT as a scheduler

Re: [PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-16 Thread Doug Anderson
Daniel, On Mon, Jun 16, 2014 at 1:52 AM, Daniel Lezcano daniel.lezc...@linaro.org wrote: Hi Dough, thanks for the explanation. I still don't get why it is important to keep others users of mct traceable because it is quite slow ? May be it is what you explained here, but I miss the

Re: [PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-15 Thread Daniel Lezcano
On 06/04/2014 07:30 PM, Doug Anderson wrote: In (93bfb76 clocksource: exynos_mct: register sched_clock callback) we supported using the MCT as a scheduler clock. We properly marked exynos4_read_sched_clock() as notrace. However, we then went and called another function that _wasn't_ notrace.

Re: [PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-15 Thread Doug Anderson
Daniel, On Sun, Jun 15, 2014 at 2:18 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: On 06/04/2014 07:30 PM, Doug Anderson wrote: In (93bfb76 clocksource: exynos_mct: register sched_clock callback) we supported using the MCT as a scheduler clock. We properly marked

[PATCH 1/3] clocksource: exynos_mct: Fix ftrace

2014-06-04 Thread Doug Anderson
In (93bfb76 clocksource: exynos_mct: register sched_clock callback) we supported using the MCT as a scheduler clock. We properly marked exynos4_read_sched_clock() as notrace. However, we then went and called another function that _wasn't_ notrace. That means if you do: cd