[Intel-gfx] [PATCH v2 1/2] drm/i915: Extend intel_wait_for_register_fw() with fast timeout

Chris Wilson chris at chris-wilson.co.uk
Fri Apr 7 17:51:36 UTC 2017


On Fri, Apr 07, 2017 at 06:26:03PM +0100, Chris Wilson wrote:
> Hmm. Thinking about it more, using _wait_for() at all here is pointless.
> 
> You just want to do something like,
> 
> if (fast_timeout_us > 10)
> 	fast_timeout_us = 10;
> 
> So
> 
> -               ret = _wait_for(done, fast_timeout_us, 10);
> -       else
> -               ret = _wait_for_atomic(done, fast_timeout_us, 0);
> +       if (fast_timeout_us > 50000)
> +               fast_timeout_us = 50000;
> +       ret = _wait_for_atomic(done, fast_timeout_us, 0);

After chatting on irc, looking at changing the behaviour here for future
users (those who may want fast timeout > 10, e.g. 100/200) should be
postponed until after they have been converted.

Pushed, thanks for the patches.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list