Commit 38b7fb0b authored by Chris Wilson's avatar Chris Wilson

drm/i915/selftests: Release the struct_mutex to free the objects

live_gtt is a very slow test to run, simply because it tries to allocate
and use as much as the 48b address space as possibly can and in the
process will try to own all of the system memory. This leads to resource
exhaustion and CPU starvation; the latter impacts us when the NMI
watchdog declares a task hung due to a mutex contention with ourselves.
This we can prevent by releasing the struct_mutex and forcing our
i915/rcu workers to run, and in particular flushing the freed object
worker that is the cause for concern.

References: https://bugs.freedesktop.org/show_bug.cgi?id=107094Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180703101829.7360-1-chris@chris-wilson.co.uk
parent a6089879
......@@ -32,6 +32,20 @@
#include "mock_drm.h"
#include "mock_gem_device.h"
static void cleanup_freed_objects(struct drm_i915_private *i915)
{
/*
* As we may hold onto the struct_mutex for inordinate lengths of
* time, the NMI khungtaskd detector may fire for the free objects
* worker.
*/
mutex_unlock(&i915->drm.struct_mutex);
i915_gem_drain_freed_objects(i915);
mutex_lock(&i915->drm.struct_mutex);
}
static void fake_free_pages(struct drm_i915_gem_object *obj,
struct sg_table *pages)
{
......@@ -291,6 +305,8 @@ static int lowlevel_hole(struct drm_i915_private *i915,
i915_gem_object_put(obj);
kfree(order);
cleanup_freed_objects(i915);
}
return 0;
......@@ -519,6 +535,7 @@ static int fill_hole(struct drm_i915_private *i915,
}
close_object_list(&objects, vm);
cleanup_freed_objects(i915);
}
return 0;
......@@ -605,6 +622,8 @@ static int walk_hole(struct drm_i915_private *i915,
i915_gem_object_put(obj);
if (err)
return err;
cleanup_freed_objects(i915);
}
return 0;
......@@ -789,6 +808,8 @@ static int drunk_hole(struct drm_i915_private *i915,
kfree(order);
if (err)
return err;
cleanup_freed_objects(i915);
}
return 0;
......@@ -857,6 +878,7 @@ static int __shrink_hole(struct drm_i915_private *i915,
}
close_object_list(&objects, vm);
cleanup_freed_objects(i915);
return err;
}
......@@ -949,6 +971,7 @@ static int shrink_boom(struct drm_i915_private *i915,
i915_gem_object_put(explode);
memset(&vm->fault_attr, 0, sizeof(vm->fault_attr));
cleanup_freed_objects(i915);
}
return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment