On Wed, Jun 29, 2016 at 12:27:22PM +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <[email protected]> > > usleep_range is not recommended for waits shorten than 10us. > > Make the wait_for_us use the atomic variant for such waits. > > To do so we need to reimplement the _wait_for_atomic macro to > be safe with regards to preemption and interrupts. > > v2: Reimplement _wait_for_atomic to be irq and preemption safe. > (Chris Wilson and Imre Deak) > > v3: Fixed in_atomic check due rebase error. > v4: Build bug on non-constant timeouts. > v5: Compile away cpu migration code in atomic paths. > > Signed-off-by: Tvrtko Ursulin <[email protected]> > Cc: Chris Wilson <[email protected]> > Cc: Imre Deak <[email protected]> > Cc: Mika Kuoppala <[email protected]>
I like the polish. Reviewed-by: Chris Wilson <[email protected]> Using wait_for_hybrid() is really tempting, just need to kick Mika to finish intel_wait_for_register()... -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
