Re: [Intel-gfx] [PATCH] drm/i915: avoid false fence signaling when enabling preemption

2018-03-12 Thread He, Min
-- > From: Chris Wilson [mailto:ch...@chris-wilson.co.uk] > Sent: Monday, March 12, 2018 6:47 PM > To: He, Min <min...@intel.com>; intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915: avoid false fence signaling when > enabling preemption > >

Re: [Intel-gfx] [PATCH] drm/i915: avoid false fence signaling when enabling preemption

2018-03-12 Thread Chris Wilson
Quoting Chris Wilson (2018-03-12 10:42:56) > Quoting Min He (2018-03-12 06:17:10) > > In i915_request_wait(), it adds a local intel_wait variable into rb tree > > of waiters. However, when the corresponding request is preempted, the > > seqno of this wait will not be updated, which will lead to a

Re: [Intel-gfx] [PATCH] drm/i915: avoid false fence signaling when enabling preemption

2018-03-12 Thread Chris Wilson
Quoting Chris Wilson (2018-03-12 10:42:56) > Quoting Min He (2018-03-12 06:17:10) > > In i915_request_wait(), it adds a local intel_wait variable into rb tree > > of waiters. However, when the corresponding request is preempted, the > > seqno of this wait will not be updated, which will lead to a

Re: [Intel-gfx] [PATCH] drm/i915: avoid false fence signaling when enabling preemption

2018-03-12 Thread Chris Wilson
Quoting Min He (2018-03-12 06:17:10) > In i915_request_wait(), it adds a local intel_wait variable into rb tree > of waiters. However, when the corresponding request is preempted, the > seqno of this wait will not be updated, which will lead to a false > signaling to the request and cause the