diff options
| author | Dominik Grzegorzek <dominik.grzegorzek@intel.com> | 2020-05-19 16:41:57 +0200 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-05-19 19:23:11 +0100 |
| commit | a772a7c7a761c6125bc0af5284ad603478107737 (patch) | |
| tree | 92b49782f86482a189aebea72ff9f64e0f9416b3 | |
| parent | e1a6d252495d71c1bd86fdc5f293cc10d6c28e6c (diff) | |
i915/gem_busy: Remove unnecessary non-preemptible workload
Non-preemptible spinners should be avoided as often as possible.
gem_busy@basic has no reason to use non-preemptible workload, only
hung subtests need it.
Signed-off-by: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
| -rw-r--r-- | tests/i915/gem_busy.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c index eb2b6d5c..9e28e2a9 100644 --- a/tests/i915/gem_busy.c +++ b/tests/i915/gem_busy.c @@ -347,8 +347,8 @@ static void basic(int fd, const struct intel_execution_engine2 *e, unsigned flag igt_spin_t *spin = igt_spin_new(fd, .engine = e->flags, - .flags = IGT_SPIN_NO_PREEMPTION | - (flags & HANG ? IGT_SPIN_INVALID_CS : 0)); + .flags = flags & HANG ? + IGT_SPIN_NO_PREEMPTION | IGT_SPIN_INVALID_CS : 0); struct timespec tv; int timeout; |