[Intel-gfx] [PATCH igt 1/6] tests/kms_draw_crc: remove unnecessary mode unset calls
Paulo Zanoni
paulo.r.zanoni at intel.com
Thu Dec 22 20:42:03 UTC 2016
I couldn't think of a reason why we would need to unset the CRTCs
before doing the modesets on this test, so remove all the mode unset
calls.
Before:
$ time -p sudo ./kms_draw_crc
real 44.74
$ time -p for i in $(sudo ./kms_draw_crc --list-subtests); do sudo
./kms_draw_crc --run-subtest $i; done
real 121.61
After:
$ time -p sudo ./kms_draw_crc
real 7.40
$ time -p for i in $(sudo ./kms_draw_crc --list-subtests); do sudo
./kms_draw_crc --run-subtest $i; done
real 14.32
Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
---
tests/kms_draw_crc.c | 7 -------
1 file changed, 7 deletions(-)
diff --git a/tests/kms_draw_crc.c b/tests/kms_draw_crc.c
index cb28052..e163981 100644
--- a/tests/kms_draw_crc.c
+++ b/tests/kms_draw_crc.c
@@ -143,7 +143,6 @@ static void get_method_crc(enum igt_draw_method method, uint32_t drm_format,
igt_pipe_crc_collect_crc(pipe_crc, crc);
- kmstest_unset_all_crtcs(drm_fd, drm_res);
igt_remove_fb(drm_fd, &fb);
}
@@ -152,8 +151,6 @@ static void draw_method_subtest(enum igt_draw_method method,
{
igt_crc_t crc;
- kmstest_unset_all_crtcs(drm_fd, drm_res);
-
/* Use IGT_DRAW_MMAP_GTT on an untiled buffer as the parameter for
* comparison. Cache the value so we don't recompute it for every single
* subtest. */
@@ -184,7 +181,6 @@ static void get_fill_crc(uint64_t tiling, igt_crc_t *crc)
igt_pipe_crc_collect_crc(pipe_crc, crc);
- kmstest_unset_all_crtcs(drm_fd, drm_res);
igt_remove_fb(drm_fd, &fb);
}
@@ -194,8 +190,6 @@ static void fill_fb_subtest(void)
struct igt_fb fb;
igt_crc_t base_crc, crc;
- kmstest_unset_all_crtcs(drm_fd, drm_res);
-
igt_create_fb(drm_fd, ms.mode->hdisplay, ms.mode->vdisplay,
DRM_FORMAT_XRGB8888, LOCAL_DRM_FORMAT_MOD_NONE, &fb);
@@ -214,7 +208,6 @@ static void fill_fb_subtest(void)
get_fill_crc(LOCAL_I915_FORMAT_MOD_X_TILED, &crc);
igt_assert_crc_equal(&crc, &base_crc);
- kmstest_unset_all_crtcs(drm_fd, drm_res);
igt_remove_fb(drm_fd, &fb);
}
--
2.7.4
More information about the Intel-gfx
mailing list