diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2020-02-28 17:14:13 +0300 |
|---|---|---|
| committer |   Jani Nikula <jani.nikula@intel.com> | 2020-03-02 12:11:07 +0200 |
| commit | f4aaa44e8b20f7e0d4ea68d3bca4968b6ae5aaff (patch) | |
| tree | 9dce5c6e5064c7cbeeafcd7bbb182202d51d196c /drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | |
| parent | drm/i915: Protect i915_request_await_start from early waits (diff) | |
| download | linux-dev-f4aaa44e8b20f7e0d4ea68d3bca4968b6ae5aaff.tar.xz linux-dev-f4aaa44e8b20f7e0d4ea68d3bca4968b6ae5aaff.zip | |
drm/i915/selftests: Fix return in assert_mmap_offset()
The assert_mmap_offset() returns type bool so if we return an error
pointer that is "return true;" or success. If we have an error, then
we should return false.
Fixes: 3d81d589d6e3 ("drm/i915: Test exhaustion of the mmap space")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20200228141413.qfjf4abr323drlo4@kili.mountain
(cherry picked from commit efbf928824820f2738f41271934f6ec2c6ebd587)
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c index ef7c74cff28a..43912e9b683d 100644 --- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c +++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c @@ -570,7 +570,7 @@ static bool assert_mmap_offset(struct drm_i915_private *i915, obj = i915_gem_object_create_internal(i915, size); if (IS_ERR(obj)) - return PTR_ERR(obj); + return false; mmo = mmap_offset_attach(obj, I915_MMAP_OFFSET_GTT, NULL); i915_gem_object_put(obj); |