Quoting Michal Wajdeczko (2019-07-19 16:33:22) > When __gen8_ppgtt_alloc fails without allocating anything > we should not try to call __gen8_ppgtt_clear as there is > nothing to clear and underlying code will complain with: > > [ 157.861645] gen8_pd_range:881 GEM_BUG_ON(start >= end) > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c > index 220aba5a94d2..5e753e075047 100644 > --- a/drivers/gpu/drm/i915/i915_gem_gtt.c > +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c > @@ -1115,7 +1115,7 @@ static int gen8_ppgtt_alloc(struct i915_address_space *vm, > > err = __gen8_ppgtt_alloc(vm, i915_vm_to_ppgtt(vm)->pd, > &start, start + length, vm->top); > - if (unlikely(err)) > + if (unlikely(err && from != start)) > __gen8_ppgtt_clear(vm, i915_vm_to_ppgtt(vm)->pd, > from, start, vm->top); Ugh. Everyone's favourite error today. Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
