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

2016-10-19 Thread Zhenyu Wang
On 2016.10.19 10:14:39 +0100, Chris Wilson wrote: > For whatever reason, the gvt scheduler runs synchronously. At the very > least, lets run synchronously without holding the struct_mutex. > > v2: cut'n'paste mutex_lock instead of unlock. > Replace long hold of struct_mutex with a mutex to

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

2016-10-19 Thread Chris Wilson
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); > > +

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

2016-10-19 Thread Chris Wilson
For whatever reason, the gvt scheduler runs synchronously. At the very least, lets run synchronously without holding the struct_mutex. v2: cut'n'paste mutex_lock instead of unlock. Replace long hold of struct_mutex with a mutex to serialise the worker threads. Signed-off-by: Chris Wilson