diff options
author | Mark Yacoub <markyacoub@google.com> | 2021-09-24 14:55:10 -0400 |
---|---|---|
committer | Mark Yacoub <markyacoub@google.com> | 2021-09-27 16:07:51 +0000 |
commit | 357f5887f0b42d780ec4352600670d0c733c4cf7 (patch) | |
tree | 5f7452e8e10c8c546509fcf646caca62d153ca80 /tests/kms_addfb_basic.c | |
parent | 4b5644c9751b489c73c9bb174644c08b31533cc8 (diff) |
tests/kms_addfb_basic: Remove handling for Unknown Vendor.
[Why]
Looking at multiple drivers such as i915, amdgpu, and msm, EINVAL is the
standard errno when failing some fb errors.
Unless it's an exception like nouveau, every driver should act in a
standard manner and return EINVAL just like every other driver. Hence,
no special handling for unknown vendor is needed.
[How]
Remove asserts for specific drivers except for a known exception.
Tested on ChromeOS Trogdor(msm).
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tests/kms_addfb_basic.c')
-rw-r--r-- | tests/kms_addfb_basic.c | 26 |
1 files changed, 7 insertions, 19 deletions
diff --git a/tests/kms_addfb_basic.c b/tests/kms_addfb_basic.c index eed7038c..55f4852d 100644 --- a/tests/kms_addfb_basic.c +++ b/tests/kms_addfb_basic.c @@ -308,17 +308,10 @@ static void pitch_tests(int fd) igt_subtest_f("bad-pitch-%i", bad_pitches[i]) { f.pitches[0] = bad_pitches[i]; igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_MODE_ADDFB2, &f), -1); - igt_assert(errno != 0); - if (is_i915_device(fd) || is_amdgpu_device(fd) || is_msm_device(fd)) { + if (is_nouveau_device(fd) && bad_pitches[i] > 4 * 1024) + igt_assert_eq(errno, ERANGE); + else igt_assert_eq(errno, EINVAL); - } else if (is_nouveau_device(fd)) { - if (bad_pitches[i] > 4 * 1024) - igt_assert_eq(errno, ERANGE); - else - igt_assert_eq(errno, EINVAL); - } else { - igt_info("Unknown vendor; errno unchecked (returned %i)", errno); - } errno = 0; } } @@ -484,13 +477,10 @@ static void size_tests(int fd) for (i = 0; i < ARRAY_SIZE(framebuffers); i++) { igt_debug("Checking framebuffer %i\n", i); igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_MODE_ADDFB2, framebuffers[i]), -1); - igt_assert(errno != 0); - if (is_i915_device(fd)) - igt_assert_eq(errno, EINVAL); - else if (is_nouveau_device(fd)) + if (is_nouveau_device(fd)) igt_assert_eq(errno, ERANGE); else - igt_info("Unknown vendor; errno unchecked (returned %i)", errno); + igt_assert_eq(errno, EINVAL); errno = 0; } } @@ -500,12 +490,10 @@ static void size_tests(int fd) igt_subtest("bo-too-small") { igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_MODE_ADDFB2, &f), -1); igt_assert(errno != 0); - if (is_i915_device(fd)) - igt_assert_eq(errno, EINVAL); - else if (is_nouveau_device(fd)) + if (is_nouveau_device(fd)) igt_assert_eq(errno, ERANGE); else - igt_info("Unknown vendor; errno unchecked (returned %i)", errno); + igt_assert_eq(errno, EINVAL); errno = 0; } |