[Intel-gfx] [PATCH] drm/i915: Only expand COND once in wait_for()

Chris Wilson chris at chris-wilson.co.uk
Tue Sep 13 19:40:19 UTC 2016


I was looking at some wait_for() timeouts on a slow system, with lots of
debug enabled (KASAN, lockdep, mmio_debug). Thinking that we were
mishandling the timeout, I tried to ensure that we loop at least once
after first testing COND. However, the double test of COND either side
of the timeout check makes that unlikely. But we can do an equivalent
loop, that keeps the COND check after testing for timeout (required so
that we are not preempted between testing COND and then testing for a
timeout) without expanding COND twice.

The advantage of only expanding COND once is a dramatic reduction in
code size:

   text	   data	    bss	    dec	    hex
1308733	   5184	   1152	1315069	 1410fd	before
1305341	   5184	   1152	1311677	 1403bd	after

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
---
 drivers/gpu/drm/i915/intel_drv.h | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index cb99a2540863..597899d71df9 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -52,13 +52,16 @@
  */
 #define _wait_for(COND, US, W) ({ \
 	unsigned long timeout__ = jiffies + usecs_to_jiffies(US) + 1;	\
-	int ret__ = 0;							\
-	while (!(COND)) {						\
-		if (time_after(jiffies, timeout__)) {			\
-			if (!(COND))					\
-				ret__ = -ETIMEDOUT;			\
+	int ret__;							\
+	for (;;) {							\
+		if (time_after(jiffies, timeout__))			\
+			ret__ = -ETIMEDOUT;				\
+		if (COND) {						\
+			ret__ = 0;					\
 			break;						\
 		}							\
+		if (ret__)						\
+			break;						\
 		if ((W) && drm_can_sleep()) {				\
 			usleep_range((W), (W)*2);			\
 		} else {						\
-- 
2.9.3



More information about the Intel-gfx mailing list