Quoting Chris Wilson (2017-08-12 11:26:08)
> Quoting Joonas Lahtinen (2017-08-10 17:22:49)
> > On ke, 2017-07-26 at 14:26 +0100, Chris Wilson wrote:
> > > +++ b/drivers/gpu/drm/i915/i915_gem_shrinker.c
> > > @@ -92,9 +92,6 @@ static bool swap_available(void)
> > >
> > > static bool can_release_pages(struct drm_i915_gem_object *obj)
> > > {
> > > - if (!i915_gem_object_has_pages(obj))
> > > - return false;
> >
> > So you think the inaccuracies we get for being lockless don't matter
> > compared to better forward progress?
>
> Actually, looking at this again, it is all improvement.
Bah. Forgot i915_gem_object_is_active() depends on the explicit
retirement and not the resv object lookup.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
