From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ben Widawsky <ben@bwidawsk.net>
Subject: Re: [PATCH 2/3] drm/i915: Check whether the ring is
 initialised prior to dispatch
Date: Fri, 11 May 2012 12:53:23 -0700
Message-ID: <20120511125323.4b7469df@bwidawsk.net>
References: <1336742972-12713-1-git-send-email-chris@chris-wilson.co.uk>
	<1336742972-12713-2-git-send-email-chris@chris-wilson.co.uk>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from cloud01.chad-versace.us (184-106-247-128.static.cloud-ips.com
	[184.106.247.128])
	by gabe.freedesktop.org (Postfix) with ESMTP id 4C6A2A0C2B
	for <intel-gfx@lists.freedesktop.org>;
	Fri, 11 May 2012 12:53:31 -0700 (PDT)
In-Reply-To: <1336742972-12713-2-git-send-email-chris@chris-wilson.co.uk>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
To: Chris Wilson <chris@chris-wilson.co.uk>
Cc: intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Fri, 11 May 2012 14:29:31 +0100
Chris Wilson <chris@chris-wilson.co.uk> wrote:

> Rather than use the magic feature tests HAS_BLT/HAS_BSD just check
> whether the ring we are about to dispatch the execbuffer on is
> initialised.
> 
> v2: Use intel_ring_initialized()
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Ben Widawsky <ben@bwidawsk.net>
> ---
>  drivers/gpu/drm/i915/i915_gem_execbuffer.c |   13 +++++--------
>  1 file changed, 5 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> index 21fc11d..974a9f1 100644
> --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> @@ -1064,17 +1064,9 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
>  		ring = &dev_priv->ring[RCS];
>  		break;
>  	case I915_EXEC_BSD:
> -		if (!HAS_BSD(dev)) {
> -			DRM_DEBUG("execbuf with invalid ring (BSD)\n");
> -			return -EINVAL;
> -		}
>  		ring = &dev_priv->ring[VCS];
>  		break;
>  	case I915_EXEC_BLT:
> -		if (!HAS_BLT(dev)) {
> -			DRM_DEBUG("execbuf with invalid ring (BLT)\n");
> -			return -EINVAL;
> -		}
>  		ring = &dev_priv->ring[BCS];
>  		break;
>  	default:
> @@ -1082,6 +1074,11 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
>  			  (int)(args->flags & I915_EXEC_RING_MASK));
>  		return -EINVAL;
>  	}
> +	if (!intel_ring_initialized(ring)) {
> +		DRM_DEBUG("execbuf with invalid ring: %d\n",
> +			  (int)(args->flags & I915_EXEC_RING_MASK));
> +		return -EINVAL;
> +	}
>  
>  	mode = args->flags & I915_EXEC_CONSTANTS_MASK;
>  	mask = I915_EXEC_CONSTANTS_MASK;



-- 
Ben Widawsky, Intel Open Source Technology Center