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_rotation_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_rotation_crc.c')
| -rw-r--r-- | tests/kms_rotation_crc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c index a57692e1..e11d7a98 100644 --- a/tests/kms_rotation_crc.c +++ b/tests/kms_rotation_crc.c @@ -204,7 +204,7 @@ static void test_plane_rotation(data_t *data, enum igt_plane plane_type) igt_display_commit2(display, commit); igt_pipe_crc_collect_crc(data->pipe_crc, &crc_output); - igt_assert(igt_crc_equal(&data->ref_crc, &crc_output)); + igt_assert_crc_equal(&data->ref_crc, &crc_output); /* check the rotation state has been reset when the VT * mode is restored */ @@ -212,7 +212,7 @@ static void test_plane_rotation(data_t *data, enum igt_plane plane_type) kmstest_set_vt_graphics_mode(); prepare_crtc(data, output, pipe, plane); igt_pipe_crc_collect_crc(data->pipe_crc, &crc_output); - igt_assert(igt_crc_equal(&crc_unrotated, &crc_output)); + igt_assert_crc_equal(&crc_unrotated, &crc_output); valid_tests++; |