On Tue, Dec 23, 2014 at 05:16:04PM +0000, Michel Thierry wrote: > From: Ben Widawsky <benjamin.widawsky@xxxxxxxxx> > > v2: Use WARN_ONs (Daniel) > > Cc: Daniel Vetter <daniel@xxxxxxxx> > Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx> > Signed-off-by: Michel Thierry <michel.thierry@xxxxxxxxx> Nitpick: Please put in some indication about what you've changed since this isn't precisly Ben's patch. Usually we go by putting a (v1) or similar behind the relavant sob line. Just for next time around. -Daniel > --- > drivers/gpu/drm/i915/i915_gem_evict.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_gem_evict.c b/drivers/gpu/drm/i915/i915_gem_evict.c > index 886ff2e..3dc7b37 100644 > --- a/drivers/gpu/drm/i915/i915_gem_evict.c > +++ b/drivers/gpu/drm/i915/i915_gem_evict.c > @@ -214,6 +214,7 @@ int i915_gem_evict_vm(struct i915_address_space *vm, bool do_idle) > struct i915_vma *vma, *next; > int ret; > > + WARN_ON(!mutex_is_locked(&vm->dev->struct_mutex)); > trace_i915_gem_evict_vm(vm); > > if (do_idle) { > @@ -222,6 +223,8 @@ int i915_gem_evict_vm(struct i915_address_space *vm, bool do_idle) > return ret; > > i915_gem_retire_requests(vm->dev); > + > + WARN_ON(!list_empty(&vm->active_list)); > } > > list_for_each_entry_safe(vma, next, &vm->inactive_list, mm_list) > -- > 2.1.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx