From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1BAD16EE77 for ; Wed, 2 Jun 2021 21:37:58 +0000 (UTC) Date: Wed, 2 Jun 2021 17:37:53 -0400 From: Rodrigo Vivi Message-ID: References: <20210518103344.2264397-1-alan.previn.teres.alexis@intel.com> <20210518103344.2264397-6-alan.previn.teres.alexis@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210518103344.2264397-6-alan.previn.teres.alexis@intel.com> Subject: Re: [igt-dev] [PATCH i-g-t 05/17] Perform a regular 3d copy as a control checkpoint List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Alan Previn Cc: igt-dev@lists.freedesktop.org List-ID: On Tue, May 18, 2021 at 03:33:32AM -0700, Alan Previn wrote: > As a control checkpoint, allocate buffers to be used > as texture and render target in the 3d engine for a > regular src to dest copy blit and verify buffer pixels. > > Signed-off-by: Alan Previn Reviewed-by: Rodrigo Vivi > --- > tests/i915/gem_pxp.c | 166 ++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 157 insertions(+), 9 deletions(-) > > diff --git a/tests/i915/gem_pxp.c b/tests/i915/gem_pxp.c > index 8779dddc..b1415efc 100644 > --- a/tests/i915/gem_pxp.c > +++ b/tests/i915/gem_pxp.c > @@ -236,6 +236,20 @@ static void test_ctx_alloc_recover_on_protect_on(int i915) > igt_assert_eq(create_ctx_with_params(i915, true, true, false, false, &ctx), -EPERM); > } > > +static void test_ctx_mod_regular_to_all_valid(int i915) > +{ > + uint32_t ctx; > + > + igt_assert_eq(create_ctx_with_params(i915, false, false, false, false, &ctx), 0); > + igt_assert_eq(get_ctx_protected_param(i915, ctx), 0); > + igt_assert_eq(get_ctx_recovery_param(i915, ctx), 1); > + igt_assert_eq(modify_ctx_param(i915, ctx, CHANGE_PARAM_RECOVERY, false), 0); > + igt_assert_eq(modify_ctx_param(i915, ctx, CHANGE_PARAM_PROTECTED, true), -EPERM); > + igt_assert_eq(get_ctx_protected_param(i915, ctx), 0); > + igt_assert_eq(get_ctx_recovery_param(i915, ctx), 0); > + gem_context_destroy(i915, ctx); > +} > + > static void test_ctx_mod_recover_off_to_on(int i915) > { > uint32_t ctx; > @@ -275,24 +289,145 @@ static void test_ctx_mod_protected_to_all_invalid(int i915) > gem_context_destroy(i915, ctx); > } > > -static void test_ctx_mod_regular_to_all_valid(int i915) > +static void fill_bo_content(int i915, uint32_t bo, uint32_t size, uint32_t initcolor) > { > - uint32_t ctx; > + uint32_t *ptr, *ptrtmp; > + int loop = 0; > > - igt_assert_eq(create_ctx_with_params(i915, false, false, false, false, &ctx), 0); > - igt_assert_eq(get_ctx_protected_param(i915, ctx), 0); > - igt_assert_eq(get_ctx_recovery_param(i915, ctx), 1); > - igt_assert_eq(modify_ctx_param(i915, ctx, CHANGE_PARAM_RECOVERY, false), 0); > - igt_assert_eq(modify_ctx_param(i915, ctx, CHANGE_PARAM_PROTECTED, true), -EPERM); > - igt_assert_eq(get_ctx_protected_param(i915, ctx), 0); > - igt_assert_eq(get_ctx_recovery_param(i915, ctx), 0); > + ptr = gem_mmap__device_coherent(i915, bo, 0, size, PROT_WRITE); > + ptrtmp = ptr; > + > + /* read and count all dword matches till size */ > + while (loop++ < (size/4)) { > + *ptrtmp = initcolor; > + ++ptrtmp; > + } > + > + igt_assert(gem_munmap(ptr, size) == 0); > +} > + > +#define COMPARE_COLOR_READIBLE 1 > +#define COMPARE_COLOR_UNREADIBLE 2 > +#define COMPARE_N_PIXELS_VERBOSELY 0 > + > +static void assert_bo_content_check(int i915, uint32_t bo, int compare_op, > + uint32_t size, uint32_t color) > +{ > + uint32_t *ptr, *ptrtmp; > + int loop = 0, num_matches = 0; > + uint32_t value; > + bool op_readible = (compare_op == COMPARE_COLOR_READIBLE); > + > + ptr = gem_mmap__device_coherent(i915, bo, 0, size, PROT_READ); > + ptrtmp = ptr; > + > + if (COMPARE_N_PIXELS_VERBOSELY) { > + igt_info("--------->>>\n"); > + while (loop < COMPARE_N_PIXELS_VERBOSELY && loop < (size/4)) { > + value = *ptrtmp; > + igt_info("Color read = 0x%08x ", value); > + igt_info("expected %c= 0x%08x)\n", op_readible?'=':'!', color); > + ++ptrtmp; > + ++loop; > + } > + igt_info("<<<---------\n"); > + ptrtmp = ptr; > + loop = 0; > + } > + > + /* count all pixels for matches */ > + while (loop++ < (size/4)) { > + value = *ptrtmp; > + if (value == color) > + ++num_matches; > + ++ptrtmp; > + } > + > + if (op_readible) > + igt_assert_eq(num_matches, (size/4)); > + else > + igt_assert_eq(num_matches, 0); > + > + igt_assert(gem_munmap(ptr, size) == 0); > +} > + > +static uint32_t alloc_and_fill_dest_buff(int i915, bool protected, uint32_t size, > + uint32_t init_color) > +{ > + uint32_t bo; > + int ret; > + > + ret = create_bo_ext(i915, size, protected, &bo); > + igt_assert_eq(ret, 0); > + igt_assert(bo); > + fill_bo_content(i915, bo, size, init_color); > + assert_bo_content_check(i915, bo, COMPARE_COLOR_READIBLE, size, init_color); > + > + return bo; > +} > + > +/* > + * Rendering tests surface attributes, keep it simple: > + * page aligned width==stride, thus, and size > + */ > +#define TSTSURF_WIDTH 1024 > +#define TSTSURF_HEIGHT 128 > +#define TSTSURF_BYTESPP 4 > +#define TSTSURF_STRIDE (TSTSURF_WIDTH*TSTSURF_BYTESPP) > +#define TSTSURF_SIZE (TSTSURF_STRIDE*TSTSURF_HEIGHT) > +#define TSTSURF_FILLCOLOR1 0xfaceface > +#define TSTSURF_INITCOLOR1 0x12341234 > + > +static void test_render_baseline(int i915) > +{ > + uint32_t ctx, srcbo, dstbo; > + struct intel_buf *srcbuf, *dstbuf; > + struct buf_ops *bops; > + struct intel_bb *ibb; > + uint32_t devid; > + int ret; > + > + devid = intel_get_drm_devid(i915); > + igt_assert(devid); > + > + bops = buf_ops_create(i915); > + igt_assert(bops); > + > + /* Perform a regular 3d copy as a control checkpoint */ > + ret = create_ctx_with_params(i915, false, false, false, false, &ctx); > + igt_assert_eq(ret, 0); > + ibb = intel_bb_create_with_context(i915, ctx, 4096); > + igt_assert(ibb); > + > + dstbo = alloc_and_fill_dest_buff(i915, false, TSTSURF_SIZE, TSTSURF_INITCOLOR1); > + dstbuf = intel_buf_create_using_handle(bops, dstbo, TSTSURF_WIDTH, TSTSURF_HEIGHT, > + TSTSURF_BYTESPP*8, 0, I915_TILING_NONE, 0); > + > + srcbo = alloc_and_fill_dest_buff(i915, false, TSTSURF_SIZE, TSTSURF_FILLCOLOR1); > + srcbuf = intel_buf_create_using_handle(bops, srcbo, TSTSURF_WIDTH, TSTSURF_HEIGHT, > + TSTSURF_BYTESPP*8, 0, I915_TILING_NONE, 0); > + > + gen12_render_copyfunc(ibb, srcbuf, 0, 0, TSTSURF_WIDTH, TSTSURF_HEIGHT, dstbuf, 0, 0); > + gem_sync(i915, dstbo); > + > + assert_bo_content_check(i915, dstbo, COMPARE_COLOR_READIBLE, > + TSTSURF_SIZE, TSTSURF_FILLCOLOR1); > + > + intel_bb_destroy(ibb); > + intel_buf_destroy(srcbuf); > + gem_close(i915, srcbo); > + intel_buf_destroy(dstbuf); > + gem_close(i915, dstbo); > gem_context_destroy(i915, ctx); > + buf_ops_destroy(bops); > } > > igt_main > { > int i915 = -1; > bool pxp_supported = false; > + igt_render_copyfunc_t rendercopy = NULL; > + uint32_t devid = 0; > > igt_fixture > { > @@ -346,6 +481,19 @@ igt_main > igt_subtest("reject-modify-context-protection-off-3") > test_ctx_mod_protected_to_all_invalid(i915); > } > + igt_subtest_group { > + igt_fixture { > + igt_require(pxp_supported); > + devid = intel_get_drm_devid(i915); > + igt_assert(devid); > + rendercopy = igt_get_render_copyfunc(devid); > + igt_require(rendercopy); > + } > + > + igt_describe("Verify protected render operations:"); > + igt_subtest("regular-baseline-src-copy-readible") > + test_render_baseline(i915); > + } > > igt_fixture { > close(i915); > -- > 2.25.1 > > _______________________________________________ > igt-dev mailing list > igt-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/igt-dev _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev