bus: arm-ccn: Use hrtimer_start() again
authorPawel Moll <pawel.moll@arm.com>
Wed, 13 May 2015 15:21:24 +0000 (16:21 +0100)
committerArnd Bergmann <arnd@arndb.de>
Wed, 13 May 2015 16:18:00 +0000 (18:18 +0200)
hrtimer_start() will no longer defer already expired timers to the
softirq in 4.2, and the __hrtimer_start_range_ns() function is
getting removed, causing build errors when both the tip tree and
the arm-ccn changes are merged.

This changes the code back to using hrtimer_start, which will
do the right thing after this branch gets merged with the
timers update from tip.

As pointed out after a discussion on the mailing list, the result will
not be worse than the what was there before you pulled my updates, as
the code was using normal hrtimer_start(). It's just when I realised
that it should be pinned I looked at what x86 uncore pmu is doing and
shamelessly (and probably a bit mindlessly) copied the "do not wakeup"
version from there.

[arnd: update commit message]
Reported-by: Mark Brown <mark.brown@arm.com>
Signed-off-by: Pawel Moll <pawel.moll@arm.com>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
drivers/bus/arm-ccn.c

index df5f307..7d9879e 100644 (file)
@@ -921,9 +921,8 @@ static void arm_ccn_pmu_event_start(struct perf_event *event, int flags)
         * attribute).
         */
        if (!ccn->irq)
-               __hrtimer_start_range_ns(&ccn->dt.hrtimer,
-                               arm_ccn_pmu_timer_period(), 0,
-                               HRTIMER_MODE_REL_PINNED, 0);
+               hrtimer_start(&ccn->dt.hrtimer, arm_ccn_pmu_timer_period(),
+                               HRTIMER_MODE_REL_PINNED);
 
        /* Set the DT bus input, engaging the counter */
        arm_ccn_pmu_xp_dt_config(event, 1);