2021-09-23 16:10:48

by Thomas Gleixner

Subject: [patch 07/11] drm/i915/pmu: Use hrtimer_forward_now()

hrtimer_forward() is about to be removed from the public
interfaces. Replace it with hrtimer_forward_now() which provides the same
functionality.

Signed-off-by: Thomas Gleixner <[email protected]>
Cc: David Airlie <[email protected]>
Cc: [email protected]
Cc: Joonas Lahtinen <[email protected]>
Cc: Jani Nikula <[email protected]>
Cc: [email protected]
Cc: Daniel Vetter <[email protected]>
Cc: Rodrigo Vivi <[email protected]>
---
drivers/gpu/drm/i915/i915_pmu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/gpu/drm/i915/i915_pmu.c
+++ b/drivers/gpu/drm/i915/i915_pmu.c
@@ -439,7 +439,7 @@ static enum hrtimer_restart i915_sample(
engines_sample(gt, period_ns);
frequency_sample(gt, period_ns);

- hrtimer_forward(hrtimer, now, ns_to_ktime(PERIOD));
+ hrtimer_forward_now(hrtimer, ns_to_ktime(PERIOD));

return HRTIMER_RESTART;
}

2021-09-24 10:58:27

by Tvrtko Ursulin

Subject: Re: [Intel-gfx] [patch 07/11] drm/i915/pmu: Use hrtimer_forward_now()

On 23/09/2021 17:04, Thomas Gleixner wrote:
> hrtimer_forward() is about to be removed from the public
> interfaces. Replace it with hrtimer_forward_now() which provides the same
> functionality.
>
> Signed-off-by: Thomas Gleixner <[email protected]>
> Cc: David Airlie <[email protected]>
> Cc: [email protected]
> Cc: Joonas Lahtinen <[email protected]>
> Cc: Jani Nikula <[email protected]>
> Cc: [email protected]
> Cc: Daniel Vetter <[email protected]>
> Cc: Rodrigo Vivi <[email protected]>
> ---
> drivers/gpu/drm/i915/i915_pmu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/gpu/drm/i915/i915_pmu.c
> +++ b/drivers/gpu/drm/i915/i915_pmu.c
> @@ -439,7 +439,7 @@ static enum hrtimer_restart i915_sample(
> engines_sample(gt, period_ns);
> frequency_sample(gt, period_ns);
>
> - hrtimer_forward(hrtimer, now, ns_to_ktime(PERIOD));
> + hrtimer_forward_now(hrtimer, ns_to_ktime(PERIOD));
>
> return HRTIMER_RESTART;
> }
>

Reviewed-by: Tvrtko Ursulin <[email protected]>

Regards,

Tvrtko