Re: [Intel-gfx] [PATCH] drm/i915: Don't adjust priority on an already signaled fence

2018-01-08 Thread Chris Wilson
Quoting Mika Kuoppala (2018-01-08 09:27:38) > Chris Wilson writes: > > > When we retire a signaled fence, we free the dependency tree. However, > > we skip clearing the list so that if we then try to adjust the priority > > of the signaled fence, we may walk the list of freed dependencies. > > >

Re: [Intel-gfx] [PATCH] drm/i915: Don't adjust priority on an already signaled fence

2018-01-08 Thread Mika Kuoppala
Chris Wilson writes: > When we retire a signaled fence, we free the dependency tree. However, > we skip clearing the list so that if we then try to adjust the priority > of the signaled fence, we may walk the list of freed dependencies. > > [ 3083.156757] > ==

Re: [Intel-gfx] [PATCH] drm/i915: Don't adjust priority on an already signaled fence

2018-01-08 Thread MichaƂ Winiarski
On Sat, Jan 06, 2018 at 10:56:18AM +, Chris Wilson wrote: > When we retire a signaled fence, we free the dependency tree. However, > we skip clearing the list so that if we then try to adjust the priority > of the signaled fence, we may walk the list of freed dependencies. > > [ 3083.156757]

Re: [Intel-gfx] [PATCH] drm/i915: Don't adjust priority on an already signaled fence

2018-01-06 Thread Chris Wilson
Quoting Chris Wilson (2018-01-06 10:56:18) > When we retire a signaled fence, we free the dependency tree. However, > we skip clearing the list so that if we then try to adjust the priority > of the signaled fence, we may walk the list of freed dependencies. > > [ 3083.156757] > =

[Intel-gfx] [PATCH] drm/i915: Don't adjust priority on an already signaled fence

2018-01-06 Thread Chris Wilson
When we retire a signaled fence, we free the dependency tree. However, we skip clearing the list so that if we then try to adjust the priority of the signaled fence, we may walk the list of freed dependencies. [ 3083.156757] == [ 308