[Intel-gfx] [PATCH 27/70] drm/i915: Remove vestigal DRI1 ring quiescing code

Daniel Vetter daniel at ffwll.ch
Thu Apr 9 08:02:36 PDT 2015


On Tue, Apr 07, 2015 at 04:20:51PM +0100, Chris Wilson wrote:
> @@ -640,7 +641,7 @@ static int logical_ring_wait_request(struct intel_ringbuffer *ringbuf,
>  			break;
>  	}
>  
> -	if (&request->list == &ring->request_list)
> +	if (WARN_ON(&request->list == &ring->request_list))
>  		return -ENOSPC;

Checking for new_space < n (and initializing new_space to 0) would be a
clearer check imo. But that's just a bikeshed. Same for the legacy one
below.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the Intel-gfx mailing list