@@ -152,8 +152,10 @@ static int __run_selftests(const char *name,
continue;
cond_resched();
- if (signal_pending(current))
- return -EINTR;
+ if (signal_pending(current)) {
+ err = -EINTR;
+ goto out;
+ }
pr_info(DRIVER_NAME ": Running %s\n", st->name);
if (data)
@@ -171,6 +173,10 @@ static int __run_selftests(const char *name,
st->name, err))
err = -1;
+out:
+ rcu_barrier(); /* flush deferred fput() */
+ yield();
+ rcu_barrier(); /* and once more for anything freed from the files */
return err;
}
Use an rcu_barrier() to flush any mock files used by the selftests as the deferred cleanup may be holding resources that we need to cleanup. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> --- drivers/gpu/drm/i915/selftests/i915_selftest.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)