[Intel-gfx] [PATCH v2] drm/i915/gvt: Stop waiting whilst holding struct_mutex

Chris Wilson chris at chris-wilson.co.uk
Wed Oct 19 09:22:53 UTC 2016


On Wed, Oct 19, 2016 at 10:14:39AM +0100, Chris Wilson wrote:
> @@ -451,15 +452,18 @@ static int workload_thread(void *priv)
>  		gvt_dbg_sched("will complete workload %p\n, status: %d\n",
>  				workload, workload->status);
>  
> +		mutex_lock(&gvt->dev_priv->drm.struct_mutex);
>  		complete_current_workload(gvt, ring_id);
> +		mutex_unlock(&gvt->dev_priv->drm.struct_mutex);

I can't see any reason why complete_current_workload() would require
struct_mutex.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list