Re: [Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Chris Wilson
Quoting Tvrtko Ursulin (2020-08-07 15:10:32) > > On 07/08/2020 13:54, Chris Wilson wrote: > > Allow a brief period for continued access to a dead intel_context by > > deferring the release of the struct until after an RCU grace period. > > As we are using a dedicated slab cache for the contexts, w

Re: [Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Tvrtko Ursulin
On 07/08/2020 13:54, Chris Wilson wrote: Allow a brief period for continued access to a dead intel_context by deferring the release of the struct until after an RCU grace period. As we are using a dedicated slab cache for the contexts, we can defer the release of the slab pages via RCU, with th

[Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Chris Wilson
Allow a brief period for continued access to a dead intel_context by deferring the release of the struct until after an RCU grace period. As we are using a dedicated slab cache for the contexts, we can defer the release of the slab pages via RCU, with the caveat that individual structs may be reuse

Re: [Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Chris Wilson
Quoting Tvrtko Ursulin (2020-08-07 12:31:39) > > On 07/08/2020 12:14, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2020-08-07 11:08:59) > >> > >> On 07/08/2020 09:32, Chris Wilson wrote: > >>> +static void __intel_context_ctor(void *arg) > >>> +{ > >>> + struct intel_context *ce = arg; > >>

Re: [Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Tvrtko Ursulin
On 07/08/2020 12:14, Chris Wilson wrote: Quoting Tvrtko Ursulin (2020-08-07 11:08:59) On 07/08/2020 09:32, Chris Wilson wrote: +static void __intel_context_ctor(void *arg) +{ + struct intel_context *ce = arg; + + INIT_LIST_HEAD(&ce->signal_link); + INIT_LIST_HEAD(&ce->signals); +

Re: [Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Chris Wilson
Quoting Tvrtko Ursulin (2020-08-07 11:08:59) > > On 07/08/2020 09:32, Chris Wilson wrote: > > +static void __intel_context_ctor(void *arg) > > +{ > > + struct intel_context *ce = arg; > > + > > + INIT_LIST_HEAD(&ce->signal_link); > > + INIT_LIST_HEAD(&ce->signals); > > + > > + atom

Re: [Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Tvrtko Ursulin
On 07/08/2020 09:32, Chris Wilson wrote: Allow a brief period for continued access to a dead intel_context by deferring the release of the struct until after an RCU grace period. As we are using a dedicated slab cache for the contexts, we can defer the release of the slab pages via RCU, with th

[Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-07 Thread Chris Wilson
Allow a brief period for continued access to a dead intel_context by deferring the release of the struct until after an RCU grace period. As we are using a dedicated slab cache for the contexts, we can defer the release of the slab pages via RCU, with the caveat that individual structs may be reuse

[Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-03 Thread Chris Wilson
Allow a brief period for continued access to a dead intel_context by deferring the release of the struct until after an RCU grace period. As we are using a dedicated slab cache for the contexts, we can defer the release of the slab pages via RCU, with the caveat that individual structs may be reuse

[Intel-gfx] [PATCH 2/7] drm/i915/gt: Protect context lifetime with RCU

2020-08-03 Thread Chris Wilson
Allow a brief period for continued access to a dead intel_context by deferring the release of the struct until after an RCU grace period. As we are using a dedicated slab cache for the contexts, we can defer the release of the slab pages via RCU, with the caveat that individual structs may be reuse