Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1251420 > unrolled thread

[PATCH 0/7] clocksource: add missing notrace attribute

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/7] clocksource: add missing notrace attribute Jisheng Zhang <jszhang@marvell.com> - 2015-10-20 10:10 +0200
    [PATCH 3/7] clocksource: pistachio: fix ftrace Jisheng Zhang <jszhang@marvell.com> - 2015-10-20 10:20 +0200
    [PATCH 5/7] clocksource: vf_pit_timer: don't trace pit_read_sched_clock() Jisheng Zhang <jszhang@marvell.com> - 2015-10-20 10:20 +0200
    [PATCH 2/7] clocksource: samsung_pwm_timer: fix ftrace Jisheng Zhang <jszhang@marvell.com> - 2015-10-20 10:20 +0200
    Re: [PATCH 0/7] clocksource: add missing notrace attribute Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-10-26 19:00 +0100
      Re: [PATCH 0/7] clocksource: add missing notrace attribute Jisheng Zhang <jszhang@marvell.com> - 2015-10-27 03:30 +0100

#1251420 — [PATCH 0/7] clocksource: add missing notrace attribute

Some clocksource drivers sched_clock implementation either miss notrace
attribute or call wasn't notrace function. This series try to fix them.

Jisheng Zhang (7):
  clocksource: arm_global_timer: fix ftrace
  clocksource: samsung_pwm_timer: fix ftrace
  clocksource: pistachio: fix ftrace
  clocksource: prima2: fix ftrace
  clocksource: vf_pit_timer: don't trace pit_read_sched_clock()
  clocksource: digicolor: don't trace digicolor_timer_sched_read()
  clocksource: fsl_ftm_timer: don't trace ftm_read_sched_clock

 drivers/clocksource/arm_global_timer.c  | 9 +++++++--
 drivers/clocksource/fsl_ftm_timer.c     | 2 +-
 drivers/clocksource/samsung_pwm_timer.c | 2 +-
 drivers/clocksource/time-pistachio.c    | 3 ++-
 drivers/clocksource/timer-digicolor.c   | 2 +-
 drivers/clocksource/timer-prima2.c      | 2 +-
 drivers/clocksource/vf_pit_timer.c      | 2 +-
 7 files changed, 14 insertions(+), 8 deletions(-)

-- 
2.6.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1251421 — [PATCH 3/7] clocksource: pistachio: fix ftrace

Currently pistachio can be used as a scheduler clock, we properly marked
pistachio_read_sched_clock() as notrace. But we then call another function
pistachio_clocksource_read_cycles() that _wasn't_ notrace.

This patch fix this by adding notrace attribute to the
pistachio_clocksource_read_cycles() function.

Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
 drivers/clocksource/time-pistachio.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clocksource/time-pistachio.c b/drivers/clocksource/time-pistachio.c
index 18d4266..bba6799 100644
--- a/drivers/clocksource/time-pistachio.c
+++ b/drivers/clocksource/time-pistachio.c
@@ -67,7 +67,8 @@ static inline void gpt_writel(void __iomem *base, u32 value, u32 offset,
 	writel(value, base + 0x20 * gpt_id + offset);
 }
 
-static cycle_t pistachio_clocksource_read_cycles(struct clocksource *cs)
+static cycle_t notrace
+pistachio_clocksource_read_cycles(struct clocksource *cs)
 {
 	struct pistachio_clocksource *pcs = to_pistachio_clocksource(cs);
 	u32 counter, overflw;
-- 
2.6.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1251424 — [PATCH 5/7] clocksource: vf_pit_timer: don't trace pit_read_sched_clock()

We should not trace pit_read_sched_clock() function, add notrace
attribute to this function.

Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
 drivers/clocksource/vf_pit_timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/vf_pit_timer.c b/drivers/clocksource/vf_pit_timer.c
index f07ba99..a0e6c68 100644
--- a/drivers/clocksource/vf_pit_timer.c
+++ b/drivers/clocksource/vf_pit_timer.c
@@ -52,7 +52,7 @@ static inline void pit_irq_acknowledge(void)
 	__raw_writel(PITTFLG_TIF, clkevt_base + PITTFLG);
 }
 
-static u64 pit_read_sched_clock(void)
+static u64 notrace pit_read_sched_clock(void)
 {
 	return ~__raw_readl(clksrc_base + PITCVAL);
 }
-- 
2.6.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1251427 — [PATCH 2/7] clocksource: samsung_pwm_timer: fix ftrace

Currently samsung_pwm_timer can be used as a scheduler clock, we properly
marked samsung_read_sched_clock() as notrace. But we then call another
function samsung_clocksource_read() that _wasn't_ notrace.

This patch fix this by adding notrace attribute to the
samsung_clocksource_read() function.

Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
 drivers/clocksource/samsung_pwm_timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/samsung_pwm_timer.c b/drivers/clocksource/samsung_pwm_timer.c
index bc90e13..9502bc4 100644
--- a/drivers/clocksource/samsung_pwm_timer.c
+++ b/drivers/clocksource/samsung_pwm_timer.c
@@ -307,7 +307,7 @@ static void samsung_clocksource_resume(struct clocksource *cs)
 	samsung_time_start(pwm.source_id, true);
 }
 
-static cycle_t samsung_clocksource_read(struct clocksource *c)
+static cycle_t notrace samsung_clocksource_read(struct clocksource *c)
 {
 	return ~readl_relaxed(pwm.source_reg);
 }
-- 
2.6.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1256195

On 10/20/2015 10:02 AM, Jisheng Zhang wrote:
> Some clocksource drivers sched_clock implementation either miss notrace
> attribute or call wasn't notrace function. This series try to fix them.
>
> Jisheng Zhang (7):
>    clocksource: arm_global_timer: fix ftrace
>    clocksource: samsung_pwm_timer: fix ftrace
>    clocksource: pistachio: fix ftrace
>    clocksource: prima2: fix ftrace
>    clocksource: vf_pit_timer: don't trace pit_read_sched_clock()
>    clocksource: digicolor: don't trace digicolor_timer_sched_read()
>    clocksource: fsl_ftm_timer: don't trace ftm_read_sched_clock
>
>   drivers/clocksource/arm_global_timer.c  | 9 +++++++--
>   drivers/clocksource/fsl_ftm_timer.c     | 2 +-
>   drivers/clocksource/samsung_pwm_timer.c | 2 +-
>   drivers/clocksource/time-pistachio.c    | 3 ++-
>   drivers/clocksource/timer-digicolor.c   | 2 +-
>   drivers/clocksource/timer-prima2.c      | 2 +-
>   drivers/clocksource/vf_pit_timer.c      | 2 +-
>   7 files changed, 14 insertions(+), 8 deletions(-)

Hi Jisheng,

I will apply this series as 4.3 fixes.

In the future, could you fix the subject by:

clocksource/drivers/<driver>: [first letter uppercase]...;

Thanks

   -- Daniel


-- 
  <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1256418

Dear Daniel,

On Mon, 26 Oct 2015 18:50:10 +0100
Daniel Lezcano <daniel.lezcano@linaro.org> wrote:

> On 10/20/2015 10:02 AM, Jisheng Zhang wrote:
> > Some clocksource drivers sched_clock implementation either miss notrace
> > attribute or call wasn't notrace function. This series try to fix them.
> >
> > Jisheng Zhang (7):
> >    clocksource: arm_global_timer: fix ftrace
> >    clocksource: samsung_pwm_timer: fix ftrace
> >    clocksource: pistachio: fix ftrace
> >    clocksource: prima2: fix ftrace
> >    clocksource: vf_pit_timer: don't trace pit_read_sched_clock()
> >    clocksource: digicolor: don't trace digicolor_timer_sched_read()
> >    clocksource: fsl_ftm_timer: don't trace ftm_read_sched_clock
> >
> >   drivers/clocksource/arm_global_timer.c  | 9 +++++++--
> >   drivers/clocksource/fsl_ftm_timer.c     | 2 +-
> >   drivers/clocksource/samsung_pwm_timer.c | 2 +-
> >   drivers/clocksource/time-pistachio.c    | 3 ++-
> >   drivers/clocksource/timer-digicolor.c   | 2 +-
> >   drivers/clocksource/timer-prima2.c      | 2 +-
> >   drivers/clocksource/vf_pit_timer.c      | 2 +-
> >   7 files changed, 14 insertions(+), 8 deletions(-)  
> 
> Hi Jisheng,
> 
> I will apply this series as 4.3 fixes.
> 
> In the future, could you fix the subject by:
> 
> clocksource/drivers/<driver>: [first letter uppercase]...;

Got it. Will take care in the future.

Thanks a lot for kind reminding,
Jisheng
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web