diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-02-27 20:37:29 +0100 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-03-23 11:02:28 +0100 |
commit | e588f6dfa6f022120d5a55ef96994dbbc33d0dc9 (patch) | |
tree | 38e1075d458cf02f959c4b2da4488f1ba07d6558 /tests/kms_cursor_crc.c | |
parent | 4fec18e5e0cbbf384b8a49febd9dc650f2712bad (diff) |
lib/debugfs: Add igt_assert_crc_equal
Because of hash collisions tests should only ever compare crc
checksums for equality. Checking for inequality can result in random
failures.
To ensure this only expose and igt_assert function and use that.
Follow-up patches will rework the code for tests which don't follow
this requirement and try to compare for CRC inequality.
v2: Rebase on top of Matt's kms_plane changes.
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests/kms_cursor_crc.c')
-rw-r--r-- | tests/kms_cursor_crc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index d5b681a0..37c13bfe 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -134,7 +134,7 @@ static void do_single_test(data_t *data, int x, int y) igt_wait_for_vblank(data->drm_fd, data->pipe); igt_pipe_crc_collect_crc(pipe_crc, &ref_crc); /* Clear screen afterwards */ - igt_assert(igt_crc_equal(&crc, &ref_crc)); + igt_assert_crc_equal(&crc, &ref_crc); igt_paint_color(cr, 0, 0, data->screenw, data->screenh, 0.0, 0.0, 0.0); @@ -453,7 +453,7 @@ static void test_cursor_size(data_t *data) /* Clear screen afterwards */ igt_paint_color(cr, 0, 0, data->screenw, data->screenh, 0.0, 0.0, 0.0); - igt_assert(igt_crc_equal(&crc[i], &ref_crc)); + igt_assert_crc_equal(&crc[i], &ref_crc); } } |