[Intel-gfx] [PATCH 2/2] drm/i915: Report all objects with allocated pages to the shrinker
Chris Wilson
chris at chris-wilson.co.uk
Fri May 31 20:18:53 UTC 2019
Quoting Matthew Auld (2019-05-31 21:06:46)
> On Thu, 30 May 2019 at 21:35, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > index 9080a736663a..8b3a23bff7f6 100644
> > --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > @@ -690,7 +690,8 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *dev_priv
> > mutex_unlock(&ggtt->vm.mutex);
> >
> > spin_lock(&dev_priv->mm.obj_lock);
> > - list_move_tail(&obj->mm.link, &dev_priv->mm.bound_list);
> > + if (i915_gem_object_is_shrinkable(obj))
> > + list_move_tail(&obj->mm.link, &dev_priv->mm.bound_list);
>
> Always false, no, or maybe just future thinking?
Always false, I was just thinking of being consistent (i.e. I grepped
for bound_list). Probably better to mark it with a GEM_BUG_ON indeed.
-Chris
More information about the Intel-gfx
mailing list