On ke, 2016-04-20 at 08:43 +0100, Chris Wilson wrote: > When iterating over the bound list, we expect all objects there to have > their pages pinned (by the bound VMA). So only add those objects with > additional pin count on their pages as "pinned". These should be those > objects used for display and hardware access. > > Reported-by: Akash Goel <akash.goel@xxxxxxxxx> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Akash Goel <akash.goel@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_gem_shrinker.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem_shrinker.c b/drivers/gpu/drm/i915/i915_gem_shrinker.c > index d46388f25e04..4e4fcfa76b4c 100644 > --- a/drivers/gpu/drm/i915/i915_gem_shrinker.c > +++ b/drivers/gpu/drm/i915/i915_gem_shrinker.c > @@ -361,7 +361,7 @@ i915_gem_shrinker_oom(struct notifier_block *nb, unsigned long event, void *ptr) > if (!obj->base.filp) > continue; > > - if (obj->pages_pin_count) > + if (obj->pages_pin_count > num_vma_bound(obj)) > pinned += obj->base.size; > else > bound += obj->base.size; -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
