Re: [RFC 05/11] drm/i915: Extending i915_gem_check_wedge to check engine reset in progress

From: Chris Wilson <chris@chris-wilson.co.uk>
To: Tomas Elf <tomas.elf@intel.com>
Cc: Intel-GFX@Lists.FreeDesktop.Org, Ian Lister <ian.lister@intel.com>
Subject: Re: [RFC 05/11] drm/i915: Extending i915_gem_check_wedge to check engine reset in progress
Date: Mon, 8 Jun 2015 18:24:27 +0100	[thread overview]
Message-ID: <20150608172427.GE11457@nuc-i3427.alporthouse.com> (raw)
In-Reply-To: <1433783009-17251-6-git-send-email-tomas.elf@intel.com>

On Mon, Jun 08, 2015 at 06:03:23PM +0100, Tomas Elf wrote:
> @@ -4089,11 +4130,7 @@ i915_gem_ring_throttle(struct drm_device *dev, struct drm_file *file)
>  	unsigned reset_counter;
>  	int ret;
>  
> -	ret = i915_gem_wait_for_error(&dev_priv->gpu_error);
> -	if (ret)
> -		return ret;
> -
> -	ret = i915_gem_check_wedge(&dev_priv->gpu_error, false);
> +	ret = i915_gem_wait_for_error(dev_priv);
>  	if (ret)
>  		return ret;
>  
> @@ -4112,9 +4149,17 @@ i915_gem_ring_throttle(struct drm_device *dev, struct drm_file *file)
>  	if (target == NULL)
>  		return 0;
>  
> -	ret = __i915_wait_request(target, reset_counter, true, NULL, NULL);
> -	if (ret == 0)
> -		queue_delayed_work(dev_priv->wq, &dev_priv->mm.retire_work, 0);
> +	if (target->ring) {
> +		if (i915_gem_check_wedge(dev_priv, NULL, false))
> +			return -EIO;
> +
> +		ret = __i915_wait_request(target, reset_counter, true, NULL,
> +			NULL);
> +
> +		if (ret == 0)
> +			queue_delayed_work(dev_priv->wq,
> +				&dev_priv->mm.retire_work, 0);
> +	}

This breaks an important bit of ABI. throttle() is used to detect when
the GPU is hung, even when the client is idle (i.e. when the client
starts up or is switched to).
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx