[igt-dev] [PATCH i-g-t v9 15/16] Verify protected surfaces are dma buffer sharable
Alan Previn
alan.previn.teres.alexis at intel.com
Thu Sep 2 17:30:42 UTC 2021
Verify we can export a protected surface from
protected context A into protected context B (with
different client driver handles) and protected
rendering is successful even after prior file
handle is closed (i.e. pxp specific refcounting
works).
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 | 102 +++++++++++++++++++++++++++++++++++++++++++
1 file changed, 102 insertions(+)
diff --git a/tests/i915/gem_pxp.c b/tests/i915/gem_pxp.c
index d538e78c..9c5aa33a 100644
--- a/tests/i915/gem_pxp.c
+++ b/tests/i915/gem_pxp.c
@@ -619,6 +619,106 @@ static void test_render_pxp_protsrc_to_protdest(int i915)
buf_ops_destroy(bops);
}
+static int export_handle(int fd, uint32_t handle, int *outfd)
+{
+ struct drm_prime_handle args;
+ int ret;
+
+ args.handle = handle;
+ args.flags = DRM_CLOEXEC;
+ args.fd = -1;
+
+ ret = drmIoctl(fd, DRM_IOCTL_PRIME_HANDLE_TO_FD, &args);
+ if (ret)
+ ret = errno;
+ *outfd = args.fd;
+
+ return ret;
+}
+
+static void test_pxp_dmabuffshare_refcnt(void)
+{
+ uint32_t ctx[2], sbo[2], dbo[2];
+ struct intel_buf *sbuf[2], *dbuf[2];
+ struct buf_ops *bops[2];
+ struct intel_bb *ibb[2];
+ int fd[2], dmabuf_fd = 0, ret, n, num_matches = 0;
+ uint32_t encrypted[2][TSTSURF_SIZE/TSTSURF_BYTESPP];
+
+ /* First, create the client driver handles and
+ * protected dest buffer (is exported via dma-buff
+ * from first handle and imported to the second).
+ */
+ for (n = 0; n < 2; ++n) {
+ fd[n] = drm_open_driver(DRIVER_INTEL);
+ igt_require(fd[n]);
+ if (n == 0) {
+ dbo[0] = alloc_and_fill_dest_buff(fd[0], true, TSTSURF_SIZE,
+ TSTSURF_INITCOLOR1);
+ } else {
+ ret = export_handle(fd[0], dbo[0], &dmabuf_fd);
+ igt_assert(ret == 0);
+ dbo[1] = prime_fd_to_handle(fd[1], dmabuf_fd);
+ igt_assert(dbo[1]);
+ }
+ }
+ /* Repeat twice: Create a full set of assets to perform
+ * a protected 3D session but using the same dest buffer
+ * from above.
+ */
+ for (n = 0; n < 2; ++n) {
+ ret = create_ctx_with_params(fd[n], true, true, true, false, &ctx[n]);
+ igt_assert_eq(ret, 0);
+ igt_assert_eq(get_ctx_protected_param(fd[n], ctx[n]), 1);
+ ibb[n] = intel_bb_create_with_context(fd[n], ctx[n], 4096);
+ intel_bb_set_pxp(ibb[n], true, DISPLAY_APPTYPE,
+ I915_PROTECTED_CONTENT_DEFAULT_SESSION);
+
+ bops[n] = buf_ops_create(fd[n]);
+ if (n == 1)
+ fill_bo_content(fd[1], dbo[1], TSTSURF_SIZE, TSTSURF_INITCOLOR2);
+
+ dbuf[n] = intel_buf_create_using_handle(bops[n], dbo[n], TSTSURF_WIDTH,
+ TSTSURF_HEIGHT, TSTSURF_BYTESPP*8, 0,
+ I915_TILING_NONE, 0);
+ intel_buf_set_pxp(dbuf[n], true);
+
+ sbo[n] = alloc_and_fill_dest_buff(fd[n], false, TSTSURF_SIZE, TSTSURF_FILLCOLOR1);
+ sbuf[n] = intel_buf_create_using_handle(bops[n], sbo[n], TSTSURF_WIDTH,
+ TSTSURF_HEIGHT, TSTSURF_BYTESPP*8, 0,
+ I915_TILING_NONE, 0);
+
+ gen12_render_copyfunc(ibb[n], sbuf[n], 0, 0, TSTSURF_WIDTH, TSTSURF_HEIGHT,
+ dbuf[n], 0, 0);
+ gem_sync(fd[n], dbo[n]);
+
+ assert_bo_content_check(fd[n], dbo[n], COMPARE_COLOR_UNREADIBLE, TSTSURF_SIZE,
+ TSTSURF_FILLCOLOR1, NULL, 0);
+ assert_bo_content_check(fd[n], dbo[n], COPY_BUFFER, TSTSURF_SIZE, 0, encrypted[n],
+ TSTSURF_SIZE);
+
+ /* free up all assets except the dest buffer to
+ * verify dma buff refcounting is performed on
+ * the protected dest buffer on the 2nd loop with
+ * successful reuse in another protected render.
+ */
+ intel_bb_destroy(ibb[n]);
+ intel_buf_destroy(sbuf[n]);
+ intel_buf_destroy(dbuf[n]);
+ gem_close(fd[n], sbo[n]);
+ gem_close(fd[n], dbo[n]);
+ gem_context_destroy(fd[n], ctx[n]);
+ close(fd[n]);
+ }
+
+ /* Verify that encrypted output across loops were equal */
+ for (n = 0; n < (TSTSURF_SIZE/4); ++n)
+ if (encrypted[0][n] == encrypted[1][n])
+ ++num_matches;
+ igt_assert(num_matches == (TSTSURF_SIZE/4));
+}
+
+
static void init_powermgt_resources(int i915, struct powermgt_data *pm)
{
pm->debugfsdir = igt_debugfs_dir(i915);
@@ -1052,6 +1152,8 @@ igt_main
test_render_pxp_src_to_protdest(i915);
igt_subtest("protected-encrypted-src-copy-not-readible")
test_render_pxp_protsrc_to_protdest(i915);
+ igt_subtest("dmabuf-shared-protected-dst-is-context-refcounted")
+ test_pxp_dmabuffshare_refcnt();
}
igt_subtest_group {
igt_fixture {
--
2.25.1
More information about the igt-dev
mailing list