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_universal_plane.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_universal_plane.c')
| -rw-r--r-- | tests/kms_universal_plane.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c index 39dd0aa3..e6927171 100644 --- a/tests/kms_universal_plane.c +++ b/tests/kms_universal_plane.c @@ -261,37 +261,37 @@ functional_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output) igt_display_commit2(display, COMMIT_LEGACY); /* Blue bg + red sprite should be same under both types of API's */ - igt_assert(igt_crc_equal(&test.crc_2, &test.crc_4)); + igt_assert_crc_equal(&test.crc_2, &test.crc_4); /* Disabling primary plane should be same as black primary */ - igt_assert(igt_crc_equal(&test.crc_1, &test.crc_5)); + igt_assert_crc_equal(&test.crc_1, &test.crc_5); /* Re-enabling primary should return to blue properly */ - igt_assert(igt_crc_equal(&test.crc_2, &test.crc_6)); + igt_assert_crc_equal(&test.crc_2, &test.crc_6); /* * We should be able to setup plane FB's while CRTC is disabled and * then have them pop up correctly when the CRTC is re-enabled. */ - igt_assert(igt_crc_equal(&test.crc_2, &test.crc_7)); + igt_assert_crc_equal(&test.crc_2, &test.crc_7); /* * We should be able to modeset with the primary plane off * successfully */ - igt_assert(igt_crc_equal(&test.crc_3, &test.crc_8)); + igt_assert_crc_equal(&test.crc_3, &test.crc_8); /* * We should be able to move the primary plane completely offscreen * and have it disable successfully. */ - igt_assert(igt_crc_equal(&test.crc_5, &test.crc_9)); + igt_assert_crc_equal(&test.crc_5, &test.crc_9); /* * We should be able to explicitly disable an already * implicitly-disabled primary plane */ - igt_assert(igt_crc_equal(&test.crc_5, &test.crc_10)); + igt_assert_crc_equal(&test.crc_5, &test.crc_10); igt_plane_set_fb(primary, NULL); igt_plane_set_fb(sprite, NULL); |