Re: [Intel-gfx] [PATCH v2] drm/i915: add schedule out notification of preempted but completed request

2018-03-05 Thread Tvrtko Ursulin
On 05/03/2018 11:18, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-03-05 11:06:23) On 24/02/2018 02:59, Weinan Li wrote: There is one corner case missing schedule out notification of the preempted request. The preempted request is just completed when preemption happen, then it will be

Re: [Intel-gfx] [PATCH v2] drm/i915: add schedule out notification of preempted but completed request

2018-03-05 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-03-05 11:06:23) > > On 24/02/2018 02:59, Weinan Li wrote: > > There is one corner case missing schedule out notification of the preempted > > request. The preempted request is just completed when preemption happen, > > then it will be canceled and won't be resubmitted

Re: [Intel-gfx] [PATCH v2] drm/i915: add schedule out notification of preempted but completed request

2018-03-05 Thread Tvrtko Ursulin
On 24/02/2018 02:59, Weinan Li wrote: There is one corner case missing schedule out notification of the preempted request. The preempted request is just completed when preemption happen, then it will be canceled and won't be resubmitted later, GVT-g will lost the schedule out notification.

Re: [Intel-gfx] [PATCH v2] drm/i915: add schedule out notification of preempted but completed request

2018-03-04 Thread Li, Weinan Z
Any comments? This issue impact the preemption enabling in host, without the schedule out notification, vgpu will run into wrong state. Thanks -Weinan > -Original Message- > From: Li, Weinan Z > Sent: Saturday, February 24, 2018 11:00 AM > To: intel-gfx@lists.freedesktop.org > Cc:

[Intel-gfx] [PATCH v2] drm/i915: add schedule out notification of preempted but completed request

2018-02-23 Thread Weinan Li
There is one corner case missing schedule out notification of the preempted request. The preempted request is just completed when preemption happen, then it will be canceled and won't be resubmitted later, GVT-g will lost the schedule out notification. Here add schedule out notification if found