Re: [Intel-gfx] [PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-09-03 Thread Tvrtko Ursulin
On 02/09/2021 21:02, Daniel Vetter wrote: On Thu, Sep 2, 2021 at 6:20 PM Tvrtko Ursulin wrote: On 02/09/2021 16:05, Daniel Vetter wrote: On Thu, Sep 2, 2021 at 2:42 PM Tvrtko Ursulin wrote: On 13/08/2021 21:30, Daniel Vetter wrote: The only reason for this really is the i915_gem_engines

Re: [Intel-gfx] [PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-09-02 Thread Daniel Vetter
On Thu, Sep 2, 2021 at 6:20 PM Tvrtko Ursulin wrote: > On 02/09/2021 16:05, Daniel Vetter wrote: > > On Thu, Sep 2, 2021 at 2:42 PM Tvrtko Ursulin > > wrote: > >> > >> > >> On 13/08/2021 21:30, Daniel Vetter wrote: > >>> The only reason for this really is the i915_gem_engines->fence > >>> callbac

Re: [Intel-gfx] [PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-09-02 Thread Tvrtko Ursulin
On 02/09/2021 16:05, Daniel Vetter wrote: On Thu, Sep 2, 2021 at 2:42 PM Tvrtko Ursulin wrote: On 13/08/2021 21:30, Daniel Vetter wrote: The only reason for this really is the i915_gem_engines->fence callback engines_notify(), which exists purely as a fairly funky reference counting scheme

Re: [Intel-gfx] [PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-09-02 Thread Daniel Vetter
On Thu, Sep 2, 2021 at 2:42 PM Tvrtko Ursulin wrote: > > > On 13/08/2021 21:30, Daniel Vetter wrote: > > The only reason for this really is the i915_gem_engines->fence > > callback engines_notify(), which exists purely as a fairly funky > > reference counting scheme for that. Otherwise all other c

[PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-09-02 Thread Daniel Vetter
The only reason for this really is the i915_gem_engines->fence callback engines_notify(), which exists purely as a fairly funky reference counting scheme for that. Otherwise all other callers are from process context, and generally fairly benign locking context. Unfortunately untangling that requi

Re: [Intel-gfx] [PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-09-02 Thread Tvrtko Ursulin
On 13/08/2021 21:30, Daniel Vetter wrote: The only reason for this really is the i915_gem_engines->fence callback engines_notify(), which exists purely as a fairly funky reference counting scheme for that. Otherwise all other callers are from process context, and generally fairly benign locking

[PATCH 01/11] drm/i915: Release i915_gem_context from a worker

2021-08-13 Thread Daniel Vetter
The only reason for this really is the i915_gem_engines->fence callback engines_notify(), which exists purely as a fairly funky reference counting scheme for that. Otherwise all other callers are from process context, and generally fairly benign locking context. Unfortunately untangling that requi