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
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to