On Thu, 2017-11-09 at 10:59 +0000, Chris Wilson wrote: > Whenever we want to unbind a vma, we must wait on all GPU activity to > complete first. (This is what gives us the ability to do fine grained > eviction and purging by only having to wait on the VMA that we need to > unbind to proceed; though if pushed we can make it a rule that we are > only allowed to unbind already idle VMA and move the burden of the work > and organising the sleep onto the caller.) Currently, we might only > sleep if the vma is still active on the GPU, but in principle > i915_vma_unbind() always implies a sleep, so mark it up with a > might_sleep(). > > Suggested-by: Tvrtko Ursulin <[email protected]> > Signed-off-by: Chris Wilson <[email protected]> > Cc: Tvrtko Ursulin <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Cc: Matthew Auld <[email protected]> > Cc: Daniel Vetter <[email protected]>
<SNIP> > +++ b/drivers/gpu/drm/i915/i915_vma.c > @@ -743,6 +743,7 @@ int i915_vma_unbind(struct i915_vma *vma) > /* First wait upon any activity as retiring the request may > * have side-effects such as unpinning or even unbinding this vma. > */ > + might_sleep(); Please lift this up, even before lockdep_assert_held, for more clarity. might_sleep(); lockdep_assert_held(...); Reviewed-by: Joonas Lahtinen <[email protected]> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
