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 8B6A010E123 for ; Thu, 2 Mar 2023 11:40:13 +0000 (UTC) From: Karolina Stolarek To: igt-dev@lists.freedesktop.org Date: Thu, 2 Mar 2023 12:39:46 +0100 Message-Id: <20230302113946.1155746-1-karolina.stolarek@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t] tests/gem_ccs: Remove inplace subtests List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: block-copy-inplace and block-multicopy-inplace attempt to decompress and detile in place, that is the destination surface is the mid surface. When working in full resolve mode of operation, the tiling format has to be the same for source and destination surfaces. We can't use different tiling formats, which defeats the purpose of these tests. Remove the tests where we try to reuse the mid surface. Remove references to igt_hang_t, as without inplace subtests we expect to see no GPU hangs. Signed-off-by: Karolina Stolarek --- tests/i915/gem_ccs.c | 38 +------------------------------------- 1 file changed, 1 insertion(+), 37 deletions(-) diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c index b7e8bda7..34408db9 100644 --- a/tests/i915/gem_ccs.c +++ b/tests/i915/gem_ccs.c @@ -38,7 +38,6 @@ static struct param { struct test_config { bool compression; - bool inplace; bool surfcopy; bool new_ctx; bool suspend_resume; @@ -423,11 +422,6 @@ static void block_copy(int i915, blt_set_copy_object(&blt.dst, dst); blt_set_object_ext(&ext.src, mid_compression_format, width, height, SURFACE_TYPE_2D); blt_set_object_ext(&ext.dst, 0, width, height, SURFACE_TYPE_2D); - if (config->inplace) { - blt_set_object(&blt.dst, mid->handle, dst->size, mid->region, 0, - T_LINEAR, COMPRESSION_DISABLED, comp_type); - blt.dst.ptr = mid->ptr; - } blt_set_batch(&blt.bb, bb, bb_size, region1); blt_block_copy(i915, ctx, e, ahnd, &blt, pext); @@ -496,13 +490,6 @@ static void block_multicopy(int i915, blt_set_copy_object(&blt3.dst, dst); blt_set_copy_object(&blt3.final, final); - if (config->inplace) { - blt_set_object(&blt3.dst, mid->handle, dst->size, mid->region, - mid->mocs, mid_tiling, COMPRESSION_DISABLED, - comp_type); - blt3.dst.ptr = mid->ptr; - } - blt_set_object_ext(&ext3.src, 0, width, height, SURFACE_TYPE_2D); blt_set_object_ext(&ext3.mid, mid_compression_format, width, height, SURFACE_TYPE_2D); blt_set_object_ext(&ext3.dst, 0, width, height, SURFACE_TYPE_2D); @@ -513,8 +500,7 @@ static void block_multicopy(int i915, gem_sync(i915, blt3.final.handle); WRITE_PNG(i915, run_id, "src", &blt3.src, width, height); - if (!config->inplace) - WRITE_PNG(i915, run_id, "mid", &blt3.mid, width, height); + WRITE_PNG(i915, run_id, "mid", &blt3.mid, width, height); WRITE_PNG(i915, run_id, "dst", &blt3.dst, width, height); WRITE_PNG(i915, run_id, "final", &blt3.final, width, height); @@ -559,9 +545,6 @@ static void block_copy_test(int i915, if (config->compression && !blt_block_copy_supports_compression(i915)) return; - if (config->inplace && !config->compression) - return; - for_each_tiling(tiling) { if (!blt_block_copy_supports_tiling(i915, tiling) || (param.tiling >= 0 && param.tiling != tiling)) @@ -656,7 +639,6 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL) struct igt_collection *set; const intel_ctx_t *ctx; int i915; - igt_hang_t hang; igt_fixture { i915 = drm_open_driver(DRIVER_INTEL); @@ -670,7 +652,6 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL) I915_SYSTEM_MEMORY, I915_DEVICE_MEMORY); ctx = intel_ctx_create_all_physical(i915); - hang = igt_allow_hang(i915, ctx->id, 0); } igt_describe("Check block-copy uncompressed blit"); @@ -694,22 +675,6 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL) block_copy_test(i915, &config, ctx, set, BLOCK_MULTICOPY); } - igt_describe("Check block-copy flatccs inplace decompression blit"); - igt_subtest_with_dynamic("block-copy-inplace") { - struct test_config config = { .compression = true, - .inplace = true }; - - block_copy_test(i915, &config, ctx, set, BLOCK_COPY); - } - - igt_describe("Check block-multicopy flatccs inplace decompression blit"); - igt_subtest_with_dynamic("block-multicopy-inplace") { - struct test_config config = { .compression = true, - .inplace = true }; - - block_copy_test(i915, &config, ctx, set, BLOCK_MULTICOPY); - } - igt_describe("Check flatccs data can be copied from/to surface"); igt_subtest_with_dynamic("ctrl-surf-copy") { struct test_config config = { .compression = true, @@ -738,7 +703,6 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL) } igt_fixture { - igt_disallow_hang(i915, hang); close(i915); } } -- 2.25.1