drm/i915: Remove i915_gem_evict_inactive()

This was only used by one external caller who would just be as happy
with evict-everything, so perform the replacement and make the function
private.

In the process we note that unbinding the inactive list should not fail,
and make it a warning instead.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

1 files changed, 6 insertions, 15 deletions

diff --git a/drivers/gpu/drm/i915/i915_gem_evict.c b/drivers/gpu/drm/i915/i915_gem_evict.c
index 21a82710f4b2..399a3a8946b3 100644
--- a/drivers/gpu/drm/i915/i915_gem_evict.c
+++ b/drivers/gpu/drm/i915/i915_gem_evict.c
@@ -166,8 +166,9 @@ int
 i915_gem_evict_everything(struct drm_device *dev, bool purgeable_only)
 {
         drm_i915_private_t *dev_priv = dev->dev_private;
-        int ret;
+        struct drm_i915_gem_object *obj, *next;
         bool lists_empty;
+        int ret;
 
         lists_empty = (list_empty(&dev_priv->mm.inactive_list) &&
                        list_empty(&dev_priv->mm.flushing_list) &&
@@ -184,24 +185,14 @@ i915_gem_evict_everything(struct drm_device *dev, bool purgeable_only)
 
         BUG_ON(!list_empty(&dev_priv->mm.flushing_list));
 
-        return i915_gem_evict_inactive(dev, purgeable_only);
-}
-
-/** Unbinds all inactive objects. */
-int
-i915_gem_evict_inactive(struct drm_device *dev, bool purgeable_only)
-{
-        drm_i915_private_t *dev_priv = dev->dev_private;
-        struct drm_i915_gem_object *obj, *next;
-
+        /* Having flushed everything, unbind() should never raise an error */
         list_for_each_entry_safe(obj, next,
                                  &dev_priv->mm.inactive_list, mm_list) {
                 if (!purgeable_only || obj->madv != I915_MADV_WILLNEED) {
-                        int ret = i915_gem_object_unbind(obj);
-                        if (ret)
-                                return ret;
+                        if (obj->pin_count == 0)
+                                WARN_ON(i915_gem_object_unbind(obj));
                 }
         }
 
-        return 0;
+        return ret;
 }