[igt-dev] [PATCH i-g-t v7 09/17] Add test where both src and dest are protected
Alan Previn
alan.previn.teres.alexis at intel.com
Wed Aug 25 05:31:01 UTC 2021
When both the source and destination surfaces are
protected, the destination pixel result of the 3d
copy operation would be the same as the source. By
appending this test case to the end of the prior
test (raw-src to protected-dest) and reusing the
previous' test destination as the current source,
we can prove that new-source was decrypted properly
as we would see the difference in results: repeating
the same render operation but with a src buffer
that is protected in this case yields matching
(but still encrypted) output rendered pixels.
Signed-off-by: Alan Previn <alan.previn.teres.alexis at intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
---
tests/i915/gem_pxp.c | 144 +++++++++++++++++++++++++++++++++++++++----
1 file changed, 133 insertions(+), 11 deletions(-)
diff --git a/tests/i915/gem_pxp.c b/tests/i915/gem_pxp.c
index 396d6d5f..4c77051e 100644
--- a/tests/i915/gem_pxp.c
+++ b/tests/i915/gem_pxp.c
@@ -311,29 +311,51 @@ static void fill_bo_content(int i915, uint32_t bo, uint32_t size, uint32_t initc
#define COMPARE_COLOR_READIBLE 1
#define COMPARE_COLOR_UNREADIBLE 2
+#define COMPARE_BUFFER_READIBLE 3
+#define COMPARE_BUFFER_UNREADIBLE 4
+#define COPY_BUFFER 5
#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)
+static void assert_bo_content_check(int i915, uint32_t bo, int compare_op, uint32_t size,
+ uint32_t color, uint32_t *auxptr, int auxsize)
{
- uint32_t *ptr, *ptrtmp;
+ uint32_t *ptr, *ptrtmp, *auxtmp;
int loop = 0, num_matches = 0;
uint32_t value;
- bool op_readible = (compare_op == COMPARE_COLOR_READIBLE);
+ bool op_readible = ((compare_op == COMPARE_COLOR_READIBLE) ||
+ (compare_op == COMPARE_BUFFER_READIBLE));
+ bool chk_buff = ((compare_op == COMPARE_BUFFER_READIBLE) ||
+ (compare_op == COMPARE_BUFFER_UNREADIBLE));
+ bool copy_buff = (compare_op == COPY_BUFFER);
ptr = gem_mmap__device_coherent(i915, bo, 0, size, PROT_READ);
ptrtmp = ptr;
+ if (chk_buff || copy_buff) {
+ if (auxsize < size)
+ auxptr = NULL;
+ igt_assert(auxptr);
+ auxtmp = auxptr;
+ }
+
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);
+ if (chk_buff)
+ color = *auxtmp;
+ if (copy_buff)
+ igt_info("Color copy = 0x%08x\n", value);
+ else {
+ igt_info("Color read = 0x%08x ", value);
+ igt_info("expected %c= 0x%08x)\n", op_readible?'=':'!', color);
+ }
+ ++auxtmp;
++ptrtmp;
++loop;
}
igt_info("<<<---------\n");
+ auxtmp = auxptr;
ptrtmp = ptr;
loop = 0;
}
@@ -341,8 +363,25 @@ static void assert_bo_content_check(int i915, uint32_t bo, int compare_op,
/* count all pixels for matches */
while (loop++ < (size/4)) {
value = *ptrtmp;
- if (value == color)
- ++num_matches;
+ switch (compare_op) {
+ case COMPARE_COLOR_READIBLE:
+ case COMPARE_COLOR_UNREADIBLE:
+ if (value == color)
+ ++num_matches;
+ break;
+ case COMPARE_BUFFER_READIBLE:
+ case COMPARE_BUFFER_UNREADIBLE:
+ if (value == (*auxtmp))
+ ++num_matches;
+ ++auxtmp;
+ break;
+ case COPY_BUFFER:
+ *auxtmp = value;
+ ++auxtmp;
+ break;
+ default:
+ break;
+ }
++ptrtmp;
}
@@ -364,7 +403,8 @@ static uint32_t alloc_and_fill_dest_buff(int i915, bool protected, uint32_t size
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);
+ assert_bo_content_check(i915, bo, COMPARE_COLOR_READIBLE,
+ size, init_color, NULL, 0);
return bo;
}
@@ -382,6 +422,7 @@ static uint32_t alloc_and_fill_dest_buff(int i915, bool protected, uint32_t size
#define TSTSURF_FILLCOLOR2 0xdeaddead
#define TSTSURF_INITCOLOR1 0x12341234
#define TSTSURF_INITCOLOR2 0x56785678
+#define TSTSURF_INITCOLOR3 0xabcdabcd
static void test_render_baseline(int i915)
{
@@ -416,7 +457,7 @@ static void test_render_baseline(int i915)
gem_sync(i915, dstbo);
assert_bo_content_check(i915, dstbo, COMPARE_COLOR_READIBLE,
- TSTSURF_SIZE, TSTSURF_FILLCOLOR1);
+ TSTSURF_SIZE, TSTSURF_FILLCOLOR1, NULL, 0);
intel_bb_destroy(ibb);
intel_buf_destroy(srcbuf);
@@ -467,13 +508,92 @@ static void test_render_pxp_src_to_protdest(int i915)
gem_sync(i915, dstbo);
assert_bo_content_check(i915, dstbo, COMPARE_COLOR_UNREADIBLE,
- TSTSURF_SIZE, TSTSURF_FILLCOLOR2);
+ TSTSURF_SIZE, TSTSURF_FILLCOLOR2, NULL, 0);
+
+ 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);
+}
+
+static void test_render_pxp_protsrc_to_protdest(int i915)
+{
+ uint32_t ctx, srcbo, dstbo, dstbo2;
+ struct intel_buf *srcbuf, *dstbuf, *dstbuf2;
+ struct buf_ops *bops;
+ struct intel_bb *ibb;
+ uint32_t devid;
+ int ret;
+ uint32_t encrypted[TSTSURF_SIZE/TSTSURF_BYTESPP];
+
+ devid = intel_get_drm_devid(i915);
+ igt_assert(devid);
+
+ bops = buf_ops_create(i915);
+ igt_assert(bops);
+
+ /*
+ * Perform a protected render operation but only label
+ * the dest as protected. After rendering, the content
+ * should be encrypted
+ */
+ ret = create_ctx_with_params(i915, true, true, true, false, &ctx);
+ igt_assert_eq(ret, 0);
+ igt_assert_eq(get_ctx_protected_param(i915, ctx), 1);
+ ibb = intel_bb_create_with_context(i915, ctx, 4096);
+ igt_assert(ibb);
+ intel_bb_set_pxp(ibb, true, DISPLAY_APPTYPE, I915_PROTECTED_CONTENT_DEFAULT_SESSION);
+
+ dstbo = alloc_and_fill_dest_buff(i915, true, TSTSURF_SIZE, TSTSURF_INITCOLOR2);
+ dstbuf = intel_buf_create_using_handle(bops, dstbo, TSTSURF_WIDTH, TSTSURF_HEIGHT,
+ TSTSURF_BYTESPP*8, 0, I915_TILING_NONE, 0);
+ intel_buf_set_pxp(dstbuf, true);
+
+ srcbo = alloc_and_fill_dest_buff(i915, false, TSTSURF_SIZE, TSTSURF_FILLCOLOR2);
+ 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_UNREADIBLE,
+ TSTSURF_SIZE, TSTSURF_FILLCOLOR2, NULL, 0);
+
+ /*
+ * Reuse prior dst as the new-src and create dst2 as the new-dest.
+ * Take a copy of encrypted content from new-src for comparison after render
+ * operation. After the rendering, we should find no difference in content
+ * since both new-src and new-dest are labelled as encrypted. HW should read
+ * and decrypt new-src, perform the render and re-encrypt when going into
+ * new-dest
+ */
+ assert_bo_content_check(i915, dstbo, COPY_BUFFER,
+ TSTSURF_SIZE, 0, encrypted, TSTSURF_SIZE);
+
+ dstbo2 = alloc_and_fill_dest_buff(i915, true, TSTSURF_SIZE, TSTSURF_INITCOLOR3);
+ dstbuf2 = intel_buf_create_using_handle(bops, dstbo2, TSTSURF_WIDTH, TSTSURF_HEIGHT,
+ TSTSURF_BYTESPP*8, 0, I915_TILING_NONE, 0);
+ intel_buf_set_pxp(dstbuf2, true);
+ intel_buf_set_pxp(dstbuf, true);/*this time, src is protected*/
+
+ intel_bb_set_pxp(ibb, true, DISPLAY_APPTYPE, I915_PROTECTED_CONTENT_DEFAULT_SESSION);
+
+ gen12_render_copyfunc(ibb, dstbuf, 0, 0, TSTSURF_WIDTH, TSTSURF_HEIGHT, dstbuf2, 0, 0);
+ gem_sync(i915, dstbo2);
+
+ assert_bo_content_check(i915, dstbo2, COMPARE_BUFFER_READIBLE,
+ TSTSURF_SIZE, 0, encrypted, TSTSURF_SIZE);
intel_bb_destroy(ibb);
intel_buf_destroy(srcbuf);
gem_close(i915, srcbo);
intel_buf_destroy(dstbuf);
gem_close(i915, dstbo);
+ intel_buf_destroy(dstbuf2);
+ gem_close(i915, dstbo2);
gem_context_destroy(i915, ctx);
buf_ops_destroy(bops);
}
@@ -551,6 +671,8 @@ igt_main
test_render_baseline(i915);
igt_subtest("protected-raw-src-copy-not-readible")
test_render_pxp_src_to_protdest(i915);
+ igt_subtest("protected-encrypted-src-copy-not-readible")
+ test_render_pxp_protsrc_to_protdest(i915);
}
igt_fixture {
--
2.25.1
More information about the igt-dev
mailing list