[igt-dev] [RFC i-g-t 09/11] Add test to verify session key renewed after suspend-resume cycle
Alan Previn
alan.previn.teres.alexis at intel.com
Tue Mar 2 22:53:32 UTC 2021
Add a subtest to verify that before and after a suspend-resume
cycle, repeating the same protected rendering operation yields
different encrypted output reflecting updated encryption keys
after auto-teardown.
Signed-off-by: Alan Previn <alan.previn.teres.alexis at intel.com>
---
tests/i915/i915_pxp.c | 107 +++++++++++++++++++++++++++++++++++++++++-
1 file changed, 105 insertions(+), 2 deletions(-)
diff --git a/tests/i915/i915_pxp.c b/tests/i915/i915_pxp.c
index 2290fe34..11ca05df 100644
--- a/tests/i915/i915_pxp.c
+++ b/tests/i915/i915_pxp.c
@@ -6,6 +6,8 @@
#include <sys/ioctl.h>
#include "igt.h"
+#include "igt_device.h"
+#include "igt_sysfs.h"
#include "i915/gem.h"
IGT_TEST_DESCRIPTION("Test PXP (Protected Xe Path), which is the component "
@@ -32,6 +34,19 @@ IGT_TEST_DESCRIPTION("Test PXP (Protected Xe Path), which is the component "
#define COPY3D_BASELINE_SRC_TO_DST 1
#define COPY3D_PROTECTED_SRC_TO_PROTDST 2
#define COPY3D_PROTECTED_PROTSRC_TO_PROTDST 3
+#define COPY3D_PROTECTED_SRC_TO_PROTDST_SAMPLED 4
+
+/* test-configs for power-management triggered protected session teardown */
+#define SESSION_PMSUSPEND_TEARDOWN_KEY_CHANGE 1
+#define SESSION_PMSUSPEND_TEARDOWN_PROTCTX_BAN_CHK 2
+#define SESSION_PMSUSPEND_TEARDOWN_PROTCTX_BAN_EXEC 3
+#define SESSION_PMSUSPEND_TEARDOWN_PROTBO_BAN_EXEC 4
+
+/* Struct and defintions for power management. */
+struct powermgt_data {
+ int debugfsdir;
+ bool has_runtime_pm;
+};
static bool is_pxp_hw_supported(int i915)
{
@@ -472,7 +487,8 @@ static uint32_t alloc_and_fill_dest_buff(int i915, bool protected,
#define TSTSURF_INITCOLOR1 0x12341234
#define TSTSURF_INITCOLOR2 0x56785678
#define TSTSURF_INITCOLOR3 0xabcdabcd
-static void test_render_protected_buffer(int i915, uint32_t test_cfg)
+static void __test_render_protected_buffer(int i915, uint32_t test_cfg,
+ uint32_t *outpixels, int outsize)
{
uint32_t ctx, srcbo, dstbo, dstbo2;
struct intel_buf *srcbuf, *dstbuf, *dstbuf2;
@@ -532,6 +548,7 @@ static void test_render_protected_buffer(int i915, uint32_t test_cfg)
case COPY3D_PROTECTED_SRC_TO_PROTDST:
case COPY3D_PROTECTED_PROTSRC_TO_PROTDST:
+ case COPY3D_PROTECTED_SRC_TO_PROTDST_SAMPLED:
/* Perform a protected render operation but only label
* the dest as protected. After rendering, the content
* should be encrypted
@@ -567,7 +584,12 @@ static void test_render_protected_buffer(int i915, uint32_t test_cfg)
assert_bo_content_check(i915, dstbo, COMPARE_COLOR_UNREADIBLE,
TSTSURF_SIZE, TSTSURF_FILLCOLOR2, NULL, 0);
- if (test_cfg == COPY3D_PROTECTED_SRC_TO_PROTDST) {
+ if (test_cfg == COPY3D_PROTECTED_SRC_TO_PROTDST_SAMPLED)
+ assert_bo_content_check(i915, dstbo, COPY_BUFFER,
+ TSTSURF_SIZE, 0, outpixels, outsize);
+
+ if ((test_cfg == COPY3D_PROTECTED_SRC_TO_PROTDST) ||
+ (test_cfg == COPY3D_PROTECTED_SRC_TO_PROTDST_SAMPLED)) {
intel_bb_destroy(ibb);
intel_buf_destroy(srcbuf);
gem_close(i915, srcbo);
@@ -627,10 +649,75 @@ static void test_render_protected_buffer(int i915, uint32_t test_cfg)
buf_ops_destroy(bops);
}
+static void test_render_protected_buffer(int i915, uint32_t test_cfg)
+{
+ __test_render_protected_buffer(i915, test_cfg, NULL, 0);
+}
+
+static void init_powermgt_resources(int i915, struct powermgt_data *pm)
+{
+ pm->debugfsdir = igt_debugfs_dir(i915);
+ igt_require(pm->debugfsdir != -1);
+ pm->has_runtime_pm = igt_setup_runtime_pm(i915);
+ igt_require(pm->has_runtime_pm);
+
+}
+
+static void trigger_powermgt_suspend_cycle(int i915,
+ struct powermgt_data *pm)
+{
+ igt_pm_enable_sata_link_power_management();
+
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_MEM, SUSPEND_TEST_DEVICES);
+
+ igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_SUSPENDED);
+ gem_test_engine(i915, I915_EXEC_RENDER);
+ while ((!igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_ACTIVE))) {
+ igt_info("Waited 10 secs for resume - kick render again!\n");
+ gem_test_engine(i915, I915_EXEC_RENDER);
+ }
+}
+
+static void test_protected_session_teardown(int i915, uint32_t test_cfg,
+ struct powermgt_data *pm)
+{
+ uint32_t encrypted_pixels_b4[TSTSURF_SIZE/TSTSURF_BYTESPP];
+ uint32_t encrypted_pixels_aft[TSTSURF_SIZE/TSTSURF_BYTESPP];
+ int matched_after_keychange = 0, loop = 0;
+
+ switch (test_cfg) {
+ case SESSION_PMSUSPEND_TEARDOWN_KEY_CHANGE:
+ __test_render_protected_buffer(i915,
+ COPY3D_PROTECTED_SRC_TO_PROTDST_SAMPLED,
+ encrypted_pixels_b4, TSTSURF_SIZE);
+
+ trigger_powermgt_suspend_cycle(i915, pm);
+
+ __test_render_protected_buffer(i915,
+ COPY3D_PROTECTED_SRC_TO_PROTDST_SAMPLED,
+ encrypted_pixels_aft, TSTSURF_SIZE);
+
+ while (loop < (TSTSURF_SIZE/TSTSURF_BYTESPP)) {
+ if (encrypted_pixels_b4[loop] ==
+ encrypted_pixels_aft[loop])
+ ++matched_after_keychange;
+ ++loop;
+ }
+ igt_assert_eq(matched_after_keychange, 0);
+ break;
+ default:
+ igt_info("Skipping unknown power-mgt test_cfg = %d\n",
+ test_cfg);
+ break;
+ }
+}
+
igt_main
{
int i915 = -1;
bool pxp_supported = false;
+ struct powermgt_data pm = {0};
igt_render_copyfunc_t rendercopy = NULL;
uint32_t devid = 0;
@@ -719,6 +806,22 @@ igt_main
test_render_protected_buffer(i915,
COPY3D_PROTECTED_PROTSRC_TO_PROTDST);
}
+ 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);
+ init_powermgt_resources(i915, &pm);
+ }
+ igt_describe("Verify suspend-resume teardown management:");
+ igt_subtest("verify-pxp-key-change-after-suspend-resume") {
+ test_protected_session_teardown(i915,
+ SESSION_PMSUSPEND_TEARDOWN_KEY_CHANGE,
+ &pm);
+ }
+ }
igt_fixture {
close(i915);
--
2.25.1
More information about the igt-dev
mailing list