From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4EBBD106B504 for ; Wed, 25 Mar 2026 21:22:18 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id BC2EE10E02B; Wed, 25 Mar 2026 21:22:17 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="e2xKZDsV"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.19]) by gabe.freedesktop.org (Postfix) with ESMTPS id 94A1210E1F6 for ; Wed, 25 Mar 2026 21:22:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1774473731; x=1806009731; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=gZodTOCKlCMLeAPfT0O0r5dWUSHw9y6qyomQ+NR9KTc=; b=e2xKZDsV9LMPsSW0jJNjWVOLtHDulHWp7mUCozteFxBjDhauwqtYGgmO JucIMCakZtulvARAXtOzjI7A+jBqjtAclzi0GoNvzGyzcnxvay+mseDTA uPytzzOkzaLLN0flfFLDa6eUp2CN3KjTyQcIJHYvV0vm/hxn/tWoH22ss C3P3+WOgkfnWGvlYGLjtan67wzJh1X0jJBLAQNkTKQZi/nvvLtlkRx1fL sDWXR98adP6kQYwRT+dNuVtL5B8Kbu93yHKkCkSjc1MwmmEYP97ZBnx67 OcpqUAOrw1txvyJdUFn2HWkYlun1zvuOwuBbo/Ly7nAGh7orssWA5nWu1 w==; X-CSE-ConnectionGUID: M3579GxSTN2iXdUT8dYfWw== X-CSE-MsgGUID: jx/+IHjYSzK5fedE7giDZw== X-IronPort-AV: E=McAfee;i="6800,10657,11740"; a="74551987" X-IronPort-AV: E=Sophos;i="6.23,140,1770624000"; d="scan'208";a="74551987" Received: from fmviesa003.fm.intel.com ([10.60.135.143]) by fmvoesa113.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Mar 2026 14:22:10 -0700 X-CSE-ConnectionGUID: Ceoma4FWRHyio3AccBRg1A== X-CSE-MsgGUID: QJCKhVp4RVu95kCfiobk7A== X-ExtLoop1: 1 Received: from dut4086lnl.fm.intel.com ([10.105.10.101]) by fmviesa003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Mar 2026 14:22:10 -0700 From: Jonathan Cavitt To: igt-dev@lists.freedesktop.org Cc: saurabhg.gupta@intel.com, alex.zuo@intel.com, jonathan.cavitt@intel.com, joonas.lahtinen@linux.intel.com, matthew.brost@intel.com, jianxun.zhang@intel.com, stuart.summers@intel.com, shuicheng.lin@intel.com, nishit.sharma@intel.com Subject: [PATCH v7 4/4] tests/intel/xe_vm: Test DRM_IOCTL_XE_VM_GET_PROPERTY fault reporting Date: Wed, 25 Mar 2026 21:22:13 +0000 Message-ID: <20260325212208.85743-10-jonathan.cavitt@intel.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260325212208.85743-6-jonathan.cavitt@intel.com> References: <20260325212208.85743-6-jonathan.cavitt@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" Add a test to xe_vm that determines if pagefaults are correctly tracked and reported by the DRM_IOCTL_XE_VM_GET_PROPERTY. v2: - s/pageproperty/pagefaults v3: - Use xe_vm_get_property Signed-off-by: Jonathan Cavitt Suggested-by: Jianxun Zhang Suggested-by: Stuart Summers --- tests/intel/xe_vm.c | 156 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 156 insertions(+) diff --git a/tests/intel/xe_vm.c b/tests/intel/xe_vm.c index 778d9ac86d..997c9971db 100644 --- a/tests/intel/xe_vm.c +++ b/tests/intel/xe_vm.c @@ -2466,6 +2466,10 @@ static void test_oom(int fd) * SUBTEST: vm-get-property-invalid-property * Functionality: ioctl_input_validation * Description: Check query with invalid property returns expected error code + * + * SUBTEST: vm-get-property-exercise + * Functionality: drm_xe_vm_get_property + * Description: Check query correctly reports pagefaults on vm */ static void get_property_invalid_reserved(int fd, uint32_t vm) { @@ -2506,6 +2510,157 @@ static void get_property_invalid_property(int fd, uint32_t vm) do_ioctl_err(fd, DRM_IOCTL_XE_VM_GET_PROPERTY, &query, EINVAL); } +static void +gen_pf(int fd, uint32_t vm, struct drm_xe_engine_class_instance *eci) +{ + int n_exec_queues = 2; + int n_execs = 2; + uint64_t addr = 0x1a0000; + struct drm_xe_sync sync[2] = { + { .type = DRM_XE_SYNC_TYPE_SYNCOBJ, .flags = DRM_XE_SYNC_FLAG_SIGNAL, }, + { .type = DRM_XE_SYNC_TYPE_SYNCOBJ, .flags = DRM_XE_SYNC_FLAG_SIGNAL, }, + }; + struct drm_xe_exec exec = { + .num_batch_buffer = 1, + .num_syncs = 2, + .syncs = to_user_pointer(sync), + }; + uint32_t exec_queues[2]; + uint32_t syncobjs[2]; + size_t bo_size; + uint32_t bo = 0; + struct { + struct xe_spin spin; + uint32_t batch[16]; + uint64_t pad; + uint32_t data; + } *data; + struct xe_spin_opts spin_opts = { .preempt = false }; + int i, b; + + bo_size = sizeof(*data) * n_execs; + bo_size = xe_bb_size(fd, bo_size); + + bo = xe_bo_create(fd, vm, bo_size, + vram_if_possible(fd, eci->gt_id), + DRM_XE_GEM_CREATE_FLAG_NEEDS_VISIBLE_VRAM); + data = xe_bo_map(fd, bo, bo_size); + + for (i = 0; i < n_exec_queues; i++) { + exec_queues[i] = xe_exec_queue_create(fd, vm, eci, 0); + syncobjs[i] = syncobj_create(fd, 0); + }; + + sync[0].handle = syncobj_create(fd, 0); + xe_vm_bind_async(fd, vm, 0, bo, 0, addr, bo_size, sync, 1); + + for (i = 0; i < n_execs; i++) { + uint64_t base_addr = !i ? addr + bo_size * 128 : addr; + uint64_t batch_offset = (char *)&data[i].batch - (char *)data; + uint64_t batch_addr = base_addr + batch_offset; + uint64_t spin_offset = (char *)&data[i].spin - (char *)data; + uint64_t sdi_offset = (char *)&data[i].data - (char *)data; + uint64_t sdi_addr = base_addr + sdi_offset; + uint64_t exec_addr; + int e = i % n_exec_queues; + + if (!i) { + spin_opts.addr = base_addr + spin_offset; + xe_spin_init(&data[i].spin, &spin_opts); + exec_addr = spin_opts.addr; + } else { + b = 0; + data[i].batch[b++] = MI_STORE_DWORD_IMM_GEN4; + data[i].batch[b++] = sdi_addr; + data[i].batch[b++] = sdi_addr >> 32; + data[i].batch[b++] = 0xc0ffee; + data[i].batch[b++] = MI_BATCH_BUFFER_END; + igt_assert(b <= ARRAY_SIZE(data[i].batch)); + + exec_addr = batch_addr; + } + + sync[0].flags &= ~DRM_XE_SYNC_FLAG_SIGNAL; + sync[1].flags |= DRM_XE_SYNC_FLAG_SIGNAL; + sync[1].handle = syncobjs[e]; + + exec.exec_queue_id = exec_queues[e]; + exec.address = exec_addr; + if (e != i) + syncobj_reset(fd, &syncobjs[e], 1); + xe_exec(fd, &exec); + } + + for (i = 0; i < n_exec_queues && n_execs; i++) + igt_assert(syncobj_wait(fd, &syncobjs[i], 1, INT64_MAX, 0, + NULL)); + igt_assert(syncobj_wait(fd, &sync[0].handle, 1, INT64_MAX, 0, NULL)); + + sync[0].flags |= DRM_XE_SYNC_FLAG_SIGNAL; + xe_vm_unbind_async(fd, vm, 0, 0, addr, bo_size, sync, 1); + igt_assert(syncobj_wait(fd, &sync[0].handle, 1, INT64_MAX, 0, NULL)); + + syncobj_destroy(fd, sync[0].handle); + for (i = 0; i < n_exec_queues; i++) { + syncobj_destroy(fd, syncobjs[i]); + xe_exec_queue_destroy(fd, exec_queues[i]); + } + + munmap(data, bo_size); + gem_close(fd, bo); +} + +static void print_pf(struct xe_vm_fault *fault) +{ + igt_debug("FAULT:\n"); + igt_debug("address = 0x%08x%08x\n", + upper_32_bits(fault->address), + lower_32_bits(fault->address)); + igt_debug("address precision = %u\n", fault->address_precision); + igt_debug("access type = %u\n", fault->access_type); + igt_debug("fault type = %u\n", fault->fault_type); + igt_debug("fault level = %u\n", fault->fault_level); + igt_debug("\n"); +} + +static void get_property_exercise(int fd, uint32_t vm) +{ + struct drm_xe_engine_class_instance *hwe; + struct xe_vm_fault *faults, f0, f; + struct drm_xe_vm_get_property query = { + .vm_id = vm, + .property = DRM_XE_VM_GET_PROPERTY_FAULTS + }; + int i, fault_count; + + xe_vm_get_property(fd, vm, &query); + + igt_assert_eq(query.size, 0); + + xe_for_each_engine(fd, hwe) + gen_pf(fd, vm, hwe); + + xe_vm_get_property(fd, vm, &query); + igt_assert_lt(0, query.size); + + faults = malloc(query.size); + igt_assert(faults); + + query.data = to_user_pointer(faults); + xe_vm_get_property(fd, vm, &query); + + fault_count = query.size / sizeof(struct xe_vm_fault); + f0 = faults[0]; + for (i = 0; i < fault_count; i++) { + f = faults[i]; + print_pf(&f); + igt_assert_eq(f.address, f0.address); + igt_assert_eq(f.access_type, f0.access_type); + igt_assert_eq(f.fault_type, f0.fault_type); + } + free(faults); +} + static void test_get_property(int fd, void (*func)(int fd, uint32_t vm)) { uint32_t vm; @@ -2635,6 +2790,7 @@ int igt_main() { "invalid-vm-id", get_property_invalid_vm_id }, { "invalid-size", get_property_invalid_size }, { "invalid-property", get_property_invalid_property }, + { "exercise", get_property_exercise }, { } }; -- 2.43.0