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

Chris Wilson chris at chris-wilson.co.uk
Wed Oct 25 15:35:15 UTC 2017


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 at chris-wilson.co.uk>
> > Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> > Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> > Cc: Mika Kuoppala <mika.kuoppala at intel.com>
> 
> Reviewed-by: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>

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


More information about the Intel-gfx mailing list