From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id C96CE10E0A3 for ; Thu, 25 May 2023 11:48:04 +0000 (UTC) From: Swati Sharma To: igt-dev@lists.freedesktop.org Date: Thu, 25 May 2023 17:17:29 +0530 Message-Id: <20230525114729.298274-1-swati2.sharma@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t] tests/kms_rotation_crc: enable and optimize test for sim List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Added 2 checks. First, if rotation is supported by plane or not before creating fb. This is generic change to optimize test. Second check, is to reduce execution time in simulation by dropping RGB565 and PO10 formats (coverage is not compromised, we do have NV12 and RGB888 formats being validated). Signed-off-by: Swati Sharma --- tests/kms_rotation_crc.c | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c index 28b4fd27..07040e31 100644 --- a/tests/kms_rotation_crc.c +++ b/tests/kms_rotation_crc.c @@ -847,9 +847,17 @@ static void test_multi_plane_rotation(data_t *data, enum pipe pipe) p[0].fbinfo = &planeconfigs[i]; pointlocation(data, p, mode, 0); + if (!igt_plane_has_rotation(p[0].plane, planeconfigs[i].rotation)) + continue; + for (k = 0; k < ARRAY_SIZE(multiplaneformatlist); k++) { p[0].formatindex = k; + if (igt_run_in_simulation() && + (multiplaneformatlist[k] == DRM_FORMAT_P010 || + multiplaneformatlist[k] == DRM_FORMAT_RGB565)) + continue; + for (j = 0; j < ARRAY_SIZE(planeconfigs); j++) { p[1].fbinfo = &planeconfigs[j]; pointlocation(data, p, mode, 1); @@ -857,6 +865,14 @@ static void test_multi_plane_rotation(data_t *data, enum pipe pipe) for (l = 0; l < ARRAY_SIZE(multiplaneformatlist); l++) { p[1].formatindex = l; + if (!igt_plane_has_rotation(p[1].plane, planeconfigs[k].rotation)) + continue; + + if (igt_run_in_simulation() && + (multiplaneformatlist[k] == DRM_FORMAT_P010 || + multiplaneformatlist[k] == DRM_FORMAT_RGB565)) + continue; + if (!multiplaneskiproundcheck(data, p)) continue; @@ -872,7 +888,8 @@ static void test_multi_plane_rotation(data_t *data, enum pipe pipe) retcrc_sw = crclog[logindex]; have_crc = true; - } else if(reusecrcfromlastround(p, lastroundjformat, + } else if(!igt_run_in_simulation() && + reusecrcfromlastround(p, lastroundjformat, lastroundirotation, lastroundjrotation)) { /* -- 2.25.1