diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-11-24 12:26:56 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-11-24 14:08:01 +0000 |
| commit | 50361845e873bb2c7820d3eace1add980c420067 (patch) | |
| tree | a57ec81dae503cd244bcef73e3e0e6985d26ce4e /lib | |
| parent | 54cefc989e1f8cf6eb6e01ef61c3524b79c52b68 (diff) | |
Cast negative debugfs values to u64
Since
commit 488dac0c9237647e9b8f788b6a342595bfa40bda
Author: Yicong Yang <yangyicong@hisilicon.com>
Date: Sat Nov 21 22:17:19 2020 -0800
libfs: fix error cast of negative value in simple_attr_write()
the kernel now rejects any negative values written to debugfs, rather
than casting them to u64. Since we are accustomed to having the -1 mean
U64_MAX, perform that conversion ourselves.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/i915/gem.c | 2 | ||||
| -rw-r--r-- | lib/igt_gt.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/i915/gem.c b/lib/i915/gem.c index 45db8a0f..93ef4073 100644 --- a/lib/i915/gem.c +++ b/lib/i915/gem.c @@ -123,7 +123,7 @@ static void reset_device(int i915) if (ioctl(i915, DRM_IOCTL_I915_GEM_THROTTLE)) { igt_info("Found wedged device, trying to reset and continue\n"); - igt_sysfs_set(dir, "i915_wedged", "-1"); + igt_sysfs_printf(dir, "i915_wedged", "%llu", -1ull); } igt_sysfs_set(dir, "i915_next_seqno", "1"); diff --git a/lib/igt_gt.c b/lib/igt_gt.c index 8213526f..453446da 100644 --- a/lib/igt_gt.c +++ b/lib/igt_gt.c @@ -369,7 +369,7 @@ void igt_force_gpu_reset(int drm_fd) dir = igt_debugfs_dir(drm_fd); wedged = 0; - igt_sysfs_set(dir, "i915_wedged", "-1"); + igt_sysfs_printf(dir, "i915_wedged", "%llu", -1ull); igt_sysfs_scanf(dir, "i915_wedged", "%d", &wedged); close(dir); |