Re: [Intel-gfx] [PATCH] drm/i915: Release the atomic kmap relocation cache around snb GTT w/a

2018-02-13 Thread Tvrtko Ursulin
On 13/02/2018 13:46, Chris Wilson wrote: Quoting Chris Wilson (2018-02-13 13:45:33) Quoting Tvrtko Ursulin (2018-02-13 13:42:03) On 12/02/2018 21:11, Chris Wilson wrote: When we need to rebind the vma into the global GTT for snb, we need to drop the current reloc_cache as it will be holding

Re: [Intel-gfx] [PATCH] drm/i915: Release the atomic kmap relocation cache around snb GTT w/a

2018-02-13 Thread Chris Wilson
Quoting Chris Wilson (2018-02-13 13:45:33) > Quoting Tvrtko Ursulin (2018-02-13 13:42:03) > > > > On 12/02/2018 21:11, Chris Wilson wrote: > > > When we need to rebind the vma into the global GTT for snb, we need to > > > drop the current reloc_cache as it will be holding a kmap_atomic() and > > >

Re: [Intel-gfx] [PATCH] drm/i915: Release the atomic kmap relocation cache around snb GTT w/a

2018-02-13 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-02-13 13:42:03) > > On 12/02/2018 21:11, Chris Wilson wrote: > > When we need to rebind the vma into the global GTT for snb, we need to > > drop the current reloc_cache as it will be holding a kmap_atomic() and > > we may need to sleep for i915_vma_bind(). In practice,

Re: [Intel-gfx] [PATCH] drm/i915: Release the atomic kmap relocation cache around snb GTT w/a

2018-02-13 Thread Tvrtko Ursulin
On 12/02/2018 21:11, Chris Wilson wrote: When we need to rebind the vma into the global GTT for snb, we need to drop the current reloc_cache as it will be holding a kmap_atomic() and we may need to sleep for i915_vma_bind(). In practice, this is not an issue as we already hold an rpm reference f

[Intel-gfx] [PATCH] drm/i915: Release the atomic kmap relocation cache around snb GTT w/a

2018-02-12 Thread Chris Wilson
When we need to rebind the vma into the global GTT for snb, we need to drop the current reloc_cache as it will be holding a kmap_atomic() and we may need to sleep for i915_vma_bind(). In practice, this is not an issue as we already hold an rpm reference for the execbuffer, but with tighter error ch