[Intel-gfx] [PATCH] drm/i915: Busy-spin wait_for condition in atomic contexts

Keith Packard keithp at keithp.com
Fri Mar 25 01:38:32 CET 2011


On Thu, 24 Mar 2011 19:55:59 +0000, Chris Wilson <chris at chris-wilson.co.uk> wrote:

> -		if (W && !in_dbg_master()) msleep(W);			\
> +		if (W && !(in_dbg_master() || in_atomic())) msleep(W);  \

If the MSLEEP macro were ever used, would it need the same fix?

wait_for_atomic is never used, so perhaps the _wait_for macro should
just be renamed wait_for and the W hard-coded to 1.

As a simple fix though,

Reviewed-by: Keith Packard <keithp at keithp.com>

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20110324/dc7ac328/attachment.sig>


More information about the Intel-gfx mailing list