From patchwork Wed Apr 22 11:14:05 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: sonika.jindal@intel.com X-Patchwork-Id: 6255651 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 7784E9F1C4 for ; Wed, 22 Apr 2015 11:22:55 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 35B7820295 for ; Wed, 22 Apr 2015 11:22:54 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id BA6F120253 for ; Wed, 22 Apr 2015 11:22:52 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id DA5156E158; Wed, 22 Apr 2015 04:22:50 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id A66E56E158 for ; Wed, 22 Apr 2015 04:22:49 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga103.fm.intel.com with ESMTP; 22 Apr 2015 04:22:49 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,623,1422950400"; d="scan'208";a="683895178" Received: from sonikaji-desktop.iind.intel.com ([10.223.25.81]) by orsmga001.jf.intel.com with ESMTP; 22 Apr 2015 04:22:48 -0700 From: Sonika Jindal To: intel-gfx@lists.freedesktop.org Date: Wed, 22 Apr 2015 16:44:05 +0530 Message-Id: <1429701245-23185-1-git-send-email-sonika.jindal@intel.com> X-Mailer: git-send-email 1.7.10.4 In-Reply-To: <55376B0C.2060602@linux.intel.com> References: <55376B0C.2060602@linux.intel.com> Subject: [Intel-gfx] [PATCH] kms_rotation_crc: Adding test for 90/270 rotation X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Adding 90/270 rotation testcase for primary and sprite planes. v2: Added position test for sprite. Checking for gen > 9 for 90/270. Some cleanup and rebase. v3: Added test for unsupported tiling and unsupported pixel format for 90/270 v4: Added the legacy commit to initiate modeset in the negative test(Tvrtko) Signed-off-by: Sonika Jindal Reviewed-by: Tvrtko Ursulin --- tests/kms_rotation_crc.c | 250 ++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 217 insertions(+), 33 deletions(-) diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c index e11d7a9..eda66de 100644 --- a/tests/kms_rotation_crc.c +++ b/tests/kms_rotation_crc.c @@ -28,14 +28,21 @@ #include "igt_debugfs.h" #include "igt_kms.h" #include "igt_core.h" +#include "intel_chipset.h" typedef struct { int gfx_fd; igt_display_t display; struct igt_fb fb; struct igt_fb fb_cursor; + struct igt_fb fb_full; + struct igt_fb fb_565; + struct igt_fb fb_tiling; igt_crc_t ref_crc; igt_pipe_crc_t *pipe_crc; + igt_rotation_t rotation; + int pos_x; + int pos_y; } data_t; static void @@ -63,18 +70,46 @@ paint_squares(data_t *data, struct igt_fb *fb, drmModeModeInfo *mode, w = mode->hdisplay; h = mode->vdisplay; - cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb); + cr = igt_get_cairo_ctx(data->gfx_fd, fb); if (rotation == IGT_ROTATION_180) { cairo_translate(cr, w, h); cairo_rotate(cr, M_PI); } - /* Paint with 4 squares of Red, Green, White, Blue Clockwise */ - igt_paint_color(cr, 0, 0, w / 2, h / 2, 1.0, 0.0, 0.0); - igt_paint_color(cr, w / 2, 0, w / 2, h / 2, 0.0, 1.0, 0.0); - igt_paint_color(cr, 0, h / 2, w / 2, h / 2, 0.0, 0.0, 1.0); - igt_paint_color(cr, w / 2, h / 2, w / 2, h / 2, 1.0, 1.0, 1.0); + /* + * "rotation" is used for creating ref rotated fb and + * "data->rotation" is used to determine the required size + * while creating unrotated fb. + */ + if (rotation == IGT_ROTATION_90) { + /* Paint 4 squares with width == height in Blue, Red, + Green, White Clockwise order to look like 90 degree rotated*/ + w = h = mode->vdisplay; + igt_paint_color(cr, 0, 0, w / 2, h / 2, 0.0, 0.0, 1.0); + igt_paint_color(cr, w / 2, 0, w / 2, h / 2, 1.0, 0.0, 0.0); + igt_paint_color(cr, 0, h / 2, w / 2, h / 2, 1.0, 1.0, 1.0); + igt_paint_color(cr, w / 2, h / 2, w / 2, h / 2, 0.0, 1.0, 0.0); + + } else if (rotation == IGT_ROTATION_270) { + /* Paint 4 squares with width == height in Green, White, + Blue, Red Clockwise order to look like 270 degree rotated*/ + w = h = mode->vdisplay; + igt_paint_color(cr, 0, 0, w / 2, h / 2, 0.0, 1.0, 0.0); + igt_paint_color(cr, w / 2, 0, w / 2, h / 2, 1.0, 1.0, 1.0); + igt_paint_color(cr, 0, h / 2, w / 2, h / 2, 1.0, 0.0, 0.0); + igt_paint_color(cr, w / 2, h / 2, w / 2, h / 2, 0.0, 0.0, 1.0); + + } else { + if (data->rotation == IGT_ROTATION_90 || + data->rotation == IGT_ROTATION_270) + w = h = mode->vdisplay; + /* Paint with 4 squares of Red, Green, White, Blue Clockwise */ + igt_paint_color(cr, 0, 0, w / 2, h / 2, 1.0, 0.0, 0.0); + igt_paint_color(cr, w / 2, 0, w / 2, h / 2, 0.0, 1.0, 0.0); + igt_paint_color(cr, 0, h / 2, w / 2, h / 2, 0.0, 0.0, 1.0); + igt_paint_color(cr, w / 2, h / 2, w / 2, h / 2, 1.0, 1.0, 1.0); + } } cairo_destroy(cr); } @@ -84,7 +119,12 @@ static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, { drmModeModeInfo *mode; igt_display_t *display = &data->display; - int fb_id, fb_cursor_id; + int fb_id, fb_cursor_id, fb_full_id; + int w, h; + uint64_t tiling = LOCAL_DRM_FORMAT_MOD_NONE; + enum igt_commit_style commit = COMMIT_LEGACY; + int old_rotation; + igt_plane_t *primary; igt_output_set_pipe(output, pipe); @@ -94,10 +134,45 @@ static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, mode = igt_output_get_mode(output); + w = mode->hdisplay; + h = mode->vdisplay; + + fb_full_id = igt_create_fb(data->gfx_fd, + w, h, + DRM_FORMAT_XRGB8888, + tiling, + &data->fb_full); + igt_assert(fb_full_id); + + /* + * With igt_display_commit2 and COMMIT_UNIVERSAL, we call just the + * setplane without a modeset. So, to be able to call + * igt_display_commit and ultimately setcrtc to do the first modeset, + * we create an fb covering the crtc and call commit + */ + + old_rotation = data->rotation; + data->rotation = IGT_ROTATION_0; + primary = igt_output_get_plane(output, IGT_PLANE_PRIMARY); + paint_squares(data, &data->fb_full, mode, IGT_ROTATION_0, primary); + igt_plane_set_fb(primary, &data->fb_full); + igt_display_commit(display); + data->rotation = old_rotation; + + /* + * For 90/270, we will use create smaller fb so that the rotated + * frame can fit in + */ + if (data->rotation == IGT_ROTATION_90 || + data->rotation == IGT_ROTATION_270) { + tiling = LOCAL_I915_FORMAT_MOD_Y_TILED; + w = h = mode->vdisplay; + } + fb_id = igt_create_fb(data->gfx_fd, - mode->hdisplay, mode->vdisplay, + w, h, DRM_FORMAT_XRGB8888, - LOCAL_DRM_FORMAT_MOD_NONE, + tiling, &data->fb); igt_assert(fb_id); @@ -110,27 +185,20 @@ static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, /* Step 1: create a reference CRC for a software-rotated fb */ - /* - * XXX: We always set the primary plane to actually enable the pipe as - * there's no way (that works) to light up a pipe with only a sprite - * plane enabled at the moment. - */ - if (!plane->is_primary) { - igt_plane_t *primary; - - primary = igt_output_get_plane(output, IGT_PLANE_PRIMARY); - paint_squares(data, &data->fb, mode, IGT_ROTATION_180, primary); - igt_plane_set_fb(primary, &data->fb); - } - if (plane->is_cursor) { - paint_squares(data, &data->fb_cursor, mode, IGT_ROTATION_180, plane); + paint_squares(data, &data->fb_cursor, mode, data->rotation, plane); igt_plane_set_fb(plane, &data->fb_cursor); } else { - paint_squares(data, &data->fb, mode, IGT_ROTATION_180, plane); + paint_squares(data, &data->fb, mode, data->rotation, plane); igt_plane_set_fb(plane, &data->fb); + igt_plane_set_position(plane, data->pos_x, data->pos_y); } - igt_display_commit(display); + if (plane->is_primary || plane->is_cursor) { + igt_require(data->display.has_universal_planes); + commit = COMMIT_UNIVERSAL; + } + + igt_display_commit2(display, commit); igt_pipe_crc_collect_crc(data->pipe_crc, &data->ref_crc); /* @@ -155,6 +223,7 @@ static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane) igt_remove_fb(data->gfx_fd, &data->fb); igt_remove_fb(data->gfx_fd, &data->fb_cursor); + igt_remove_fb(data->gfx_fd, &data->fb_full); /* XXX: see the note in prepare_crtc() */ if (!plane->is_primary) { @@ -170,6 +239,71 @@ static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane) igt_display_commit(display); } +static void test_unsupported_tiling_pixel_format(data_t *data, enum igt_plane plane_type) +{ + drmModeModeInfo *mode; + igt_display_t *display = &data->display; + igt_output_t *output; + enum pipe pipe; + int valid_tests = 0; + int fb_tiling_id, fb_565_id; + + for_each_connected_output(display, output) { + for_each_pipe(display, pipe) { + igt_plane_t *plane; + + igt_output_set_pipe(output, pipe); + + plane = igt_output_get_plane(output, plane_type); + igt_require(igt_plane_supports_rotation(plane)); + mode = igt_output_get_mode(output); + + fb_tiling_id = igt_create_fb(data->gfx_fd, + mode->hdisplay, mode->vdisplay, + DRM_FORMAT_XRGB8888, + LOCAL_DRM_FORMAT_MOD_NONE, + &data->fb_tiling); + igt_assert(fb_tiling_id); + paint_squares(data, &data->fb_tiling, mode, IGT_ROTATION_0, plane); + igt_plane_set_fb(plane, &data->fb_tiling); + /* For the first modeset with legacy commit */ + igt_display_commit(display); + igt_plane_set_rotation(plane, data->rotation); + /* Shud fail because 90/270 is only supported with Y/Yf */ + igt_assert(igt_display_try_commit2(display, COMMIT_UNIVERSAL) == -EINVAL); + + fb_565_id = igt_create_fb(data->gfx_fd, + mode->hdisplay, mode->vdisplay, + DRM_FORMAT_RGB565, + LOCAL_I915_FORMAT_MOD_Y_TILED, + &data->fb_565); + igt_assert(fb_565_id); + paint_squares(data, &data->fb_565, mode, IGT_ROTATION_0, plane); + igt_plane_set_fb(plane, &data->fb_565); + igt_plane_set_rotation(plane, data->rotation); + /* Shud fail because 90/270 is not supported with RGB565 */ + igt_assert(igt_display_try_commit2(display, COMMIT_UNIVERSAL) == -EINVAL); + + /* + * check the rotation state has been reset when the VT + * mode is restored + */ + kmstest_restore_vt_mode(); + kmstest_set_vt_graphics_mode(); + prepare_crtc(data, output, pipe, plane); + + valid_tests++; + + igt_remove_fb(data->gfx_fd, &data->fb_tiling); + igt_remove_fb(data->gfx_fd, &data->fb_565); + cleanup_crtc(data, output, plane); + + igt_display_commit(display); + + } + } +} + static void test_plane_rotation(data_t *data, enum igt_plane plane_type) { igt_display_t *display = &data->display; @@ -200,21 +334,24 @@ static void test_plane_rotation(data_t *data, enum igt_plane plane_type) /* collect unrotated CRC */ igt_pipe_crc_collect_crc(data->pipe_crc, &crc_unrotated); - igt_plane_set_rotation(plane, IGT_ROTATION_180); + igt_plane_set_rotation(plane, data->rotation); igt_display_commit2(display, commit); igt_pipe_crc_collect_crc(data->pipe_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 */ + /* + * check the rotation state has been reset when the VT + * mode is restored + */ kmstest_restore_vt_mode(); kmstest_set_vt_graphics_mode(); prepare_crtc(data, output, pipe, plane); + igt_pipe_crc_collect_crc(data->pipe_crc, &crc_output); igt_assert_crc_equal(&crc_unrotated, &crc_output); - valid_tests++; cleanup_crtc(data, output, plane); } @@ -225,11 +362,13 @@ static void test_plane_rotation(data_t *data, enum igt_plane plane_type) igt_main { data_t data = {}; + int gen = 0; igt_skip_on_simulation(); igt_fixture { data.gfx_fd = drm_open_any_master(); + gen = intel_gen(intel_get_drm_devid(data.gfx_fd)); kmstest_set_vt_graphics_mode(); @@ -237,15 +376,60 @@ igt_main igt_display_init(&data.display, data.gfx_fd); } - - igt_subtest_f("primary-rotation") + igt_subtest_f("primary-rotation-180") { + data.rotation = IGT_ROTATION_180; test_plane_rotation(&data, IGT_PLANE_PRIMARY); + } - igt_subtest_f("sprite-rotation") + igt_subtest_f("sprite-rotation-180") { + data.rotation = IGT_ROTATION_180; test_plane_rotation(&data, IGT_PLANE_2); + } - igt_subtest_f("cursor-rotation") + igt_subtest_f("cursor-rotation-180") { + data.rotation = IGT_ROTATION_180; test_plane_rotation(&data, IGT_PLANE_CURSOR); + } + + igt_subtest_f("primary-rotation-90") { + igt_require(gen >= 9); + data.rotation = IGT_ROTATION_90; + test_plane_rotation(&data, IGT_PLANE_PRIMARY); + } + + igt_subtest_f("primary-rotation-270") { + igt_require(gen >= 9); + data.rotation = IGT_ROTATION_270; + test_plane_rotation(&data, IGT_PLANE_PRIMARY); + } + + igt_subtest_f("sprite-rotation-90") { + igt_require(gen >= 9); + data.rotation = IGT_ROTATION_90; + test_plane_rotation(&data, IGT_PLANE_2); + } + + igt_subtest_f("sprite-rotation-270") { + igt_require(gen >= 9); + data.rotation = IGT_ROTATION_270; + test_plane_rotation(&data, IGT_PLANE_2); + } + + igt_subtest_f("sprite-rotation-90-pos-100-0") { + igt_require(gen >= 9); + data.rotation = IGT_ROTATION_90; + data.pos_x = 100, + data.pos_y = 0; + test_plane_rotation(&data, IGT_PLANE_2); + } + + igt_subtest_f("90-rotation-unsupported-tiling-pixel-format") { + igt_require(gen >= 9); + data.rotation = IGT_ROTATION_90; + data.pos_x = 0, + data.pos_y = 0; + test_unsupported_tiling_pixel_format(&data, IGT_PLANE_PRIMARY); + } igt_fixture { igt_display_fini(&data.display);