diff options
| author | Katarzyna Dec <katarzyna.dec@intel.com> | 2019-01-23 14:25:26 +0100 |
|---|---|---|
| committer | Katarzyna Dec <katarzyna.dec@intel.com> | 2019-01-24 10:49:50 +0100 |
| commit | 3f77380fabd4083f9857daa6cd454d0937077901 (patch) | |
| tree | 1cef640cfe57c82b1ec280349e28d3dd091e4262 /lib/igt_gt.c | |
| parent | 2a02ff059e040e98a84a6896d7b51fe89bc26d10 (diff) | |
lib/igt_gt: Move assert with setting 'reset' parameter after IGT env check
We are trying to set reset parameter earlier than checking IGT env.
Let's move it after checking IGT env to avoid setting reset parameter
when hang injection is disabled.
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Diffstat (limited to 'lib/igt_gt.c')
| -rw-r--r-- | lib/igt_gt.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/igt_gt.c b/lib/igt_gt.c index dd28d821..64669672 100644 --- a/lib/igt_gt.c +++ b/lib/igt_gt.c @@ -170,15 +170,15 @@ igt_hang_t igt_allow_hang(int fd, unsigned ctx, unsigned flags) */ igt_require_gem(fd); - igt_assert(igt_sysfs_set_parameter - (fd, "reset", "%d", INT_MAX /* any reset method */)); - if (!igt_check_boolean_env_var("IGT_HANG", true)) igt_skip("hang injection disabled by user"); gem_context_require_bannable(fd); if (!igt_check_boolean_env_var("IGT_HANG_WITHOUT_RESET", false)) igt_require(has_gpu_reset(fd)); + igt_assert(igt_sysfs_set_parameter + (fd, "reset", "%d", INT_MAX /* any reset method */)); + if ((flags & HANG_ALLOW_CAPTURE) == 0) { param.param = I915_CONTEXT_PARAM_NO_ERROR_CAPTURE; param.value = 1; |