diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-02-28 15:19:16 +0100 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-03-28 10:08:08 +0100 |
commit | 70a6467b3615f49934a528a1eb6c96d0686e9f17 (patch) | |
tree | 7d55c747f70af8311129c00759ad340187b4b6bf /tests | |
parent | bc30feb7ebbdba2e8abcb53829df364d619189d0 (diff) |
tests/kms_lease: Adjust to kernel errno changes
I dropped a superfluous check for negative object id (the kernel
did a cast to s32, despite that object ids are always unsigned).
Which changes the errno from EINVAL to ENOENT. Allow both.
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/kms_lease.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/tests/kms_lease.c b/tests/kms_lease.c index 6967c489..c2671d57 100644 --- a/tests/kms_lease.c +++ b/tests/kms_lease.c @@ -714,6 +714,10 @@ static void lease_again(data_t *data) terminate_lease(&lease_b); } +#define assert_unleased(ret) \ + igt_assert_f((ret) == -EINVAL || (ret) == -ENOENT, \ + "wrong return code %i, %s\n", ret, \ + strerror(ret)) /* Test leasing an invalid connector */ static void lease_invalid_connector(data_t *data) { @@ -726,7 +730,7 @@ static void lease_invalid_connector(data_t *data) data->connector_id = 0xbaadf00d; ret = make_lease(data, &lease); data->connector_id = save_connector_id; - igt_assert_eq(ret, -EINVAL); + assert_unleased(ret); } /* Test leasing an invalid crtc */ @@ -741,7 +745,7 @@ static void lease_invalid_crtc(data_t *data) data->crtc_id = 0xbaadf00d; ret = make_lease(data, &lease); data->crtc_id = save_crtc_id; - igt_assert_eq(ret, -EINVAL); + assert_unleased(ret); } static void lease_invalid_plane(data_t *data) @@ -755,7 +759,7 @@ static void lease_invalid_plane(data_t *data) data->plane_id = 0xbaadf00d; ret = make_lease(data, &lease); data->plane_id = save_plane_id; - igt_assert_eq(ret, -EINVAL); + assert_unleased(ret); } |