Re: [PATCH 2/2] drm/i915: Use same test for eviction and submitting kernel context

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Joonas Lahtinen (2017-10-25 12:07:59)
> On Tue, 2017-10-24 at 23:08 +0100, Chris Wilson wrote:
> > During evict, we wish to idle the GPU if we see that the GGTT is full.
> > However, our test for idle in i915_gem_evict_something() and in
> > i915_gem_switch_to_kernel_context() do not match leading to
> > disappointment - we never believe that we are idle and keep trying to
> > flush the GGTT ad infinitum.
> > 
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103438
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> 
> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>

Ta, this little fixup now pushed. Afaict, this should only ever affect
the mock selftests, so I considered it to be of low impact and not
require a Fixes:
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux