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 58AF3C3DA49 for ; Thu, 25 Jul 2024 17:38:36 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 1A63B10E30E; Thu, 25 Jul 2024 17:38:36 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="fAEDSEAi"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.8]) by gabe.freedesktop.org (Postfix) with ESMTPS id 18B1910E0B7 for ; Thu, 25 Jul 2024 17:38:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1721929114; x=1753465114; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=D/1XMXCm7fFGVI5w0yf+4H/EqHbWXrwW1kL6oDCtbGs=; b=fAEDSEAiikSgFWQIjDp3zfYGNhjq5RqIOTTN9JTxw07clSlmPLen3aRm hcbOX1bn5sqvc4cYTXOfTKcLq9Snn7kyLr3R9pe8bJKKBz9GvWAVC8Uxp XFTGRmATixg65YDF96X6YkUZiXOZ3N1Uyqi5MLbTsvJ2UFnZaC9us/bXl p4lX3PfjokE4UNinQ0LMQ7PXo4iLuo/UlB4a18/v0PbMLrkiAEboOCSqT 1kXfFrNadpoIowkM6UDmFOWHUTnrOpaaXFU3iMtlO34Uq2cJ/e1kQtixi 7NPRBBEZqYWvjDVCWqjls0NIjAdin0UKrIUa9zjzKZvg85liiaZlHY61B g==; X-CSE-ConnectionGUID: T14S/V04QxiMEF3z1Yp91w== X-CSE-MsgGUID: GN9VfIG6StS+WgTYljJX3A== X-IronPort-AV: E=McAfee;i="6700,10204,11144"; a="37205126" X-IronPort-AV: E=Sophos;i="6.09,236,1716274800"; d="scan'208";a="37205126" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by fmvoesa102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Jul 2024 10:38:33 -0700 X-CSE-ConnectionGUID: YoYITGgLTUyr7DPceTa/+g== X-CSE-MsgGUID: vIYt8WCNT1uYV2xWFuVCZA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.09,236,1716274800"; d="scan'208";a="83984794" Received: from lstrano-desk.jf.intel.com ([10.54.39.91]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Jul 2024 10:38:33 -0700 From: Matthew Brost To: igt-dev@lists.freedesktop.org Cc: janga.rahul.kumar@intel.com Subject: [PATCH v2] tests/intel/xe_exec_compute_mode: Add userptr free / munmap sections Date: Thu, 25 Jul 2024 10:39:21 -0700 Message-Id: <20240725173921.1931503-1-matthew.brost@intel.com> X-Mailer: git-send-email 2.34.1 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 userptr free / munmap sections which does the aforementioned while the userptr still has a mapping. The userptr will be invalidated triggering a rebind which will fail pinning pages as the userptr is no longer valid. This is allowed and should not corrupt the VM or future submissions. v2: - s/faill/fail (Janga) - check dummy for mapped failed (Janga) Cc: Janga Rahul Kumar Signed-off-by: Matthew Brost --- tests/intel/xe_exec_compute_mode.c | 43 +++++++++++++++++++++++++----- 1 file changed, 37 insertions(+), 6 deletions(-) diff --git a/tests/intel/xe_exec_compute_mode.c b/tests/intel/xe_exec_compute_mode.c index 389de7ca40..704d975178 100644 --- a/tests/intel/xe_exec_compute_mode.c +++ b/tests/intel/xe_exec_compute_mode.c @@ -29,9 +29,11 @@ #define REBIND (0x1 << 1) #define INVALIDATE (0x1 << 2) #define RACE (0x1 << 3) -#define BIND_EXECQUEUE (0x1 << 4) +#define BIND_EXECQUEUE (0x1 << 4) #define VM_FOR_BO (0x1 << 5) -#define EXEC_QUEUE_EARLY (0x1 << 6) +#define EXEC_QUEUE_EARLY (0x1 << 6) +#define FREE_MAPPPING (0x1 << 7) +#define UNMAP_MAPPPING (0x1 << 8) /** * SUBTEST: twice-%s @@ -51,6 +53,8 @@ * @basic: basic * @preempt-fence-early: preempt fence early * @userptr: userptr + * @userptr-free: userptr free + * @userptr-unmap: userptr unmap * @rebind: rebind * @userptr-rebind: userptr rebind * @userptr-invalidate: userptr invalidate @@ -74,6 +78,8 @@ * @basic: basic * @preempt-fence-early: preempt fence early * @userptr: userptr + * @userptr-free: userptr free + * @userptr-unmap: userptr unmap * @rebind: rebind * @userptr-rebind: userptr rebind * @userptr-invalidate: userptr invalidate @@ -88,7 +94,7 @@ test_exec(int fd, struct drm_xe_engine_class_instance *eci, int n_exec_queues, int n_execs, unsigned int flags) { uint32_t vm; - uint64_t addr = 0x1a0000; + uint64_t addr = 0x1a0000, dummy_addr = 0x10001a0000; #define USER_FENCE_VALUE 0xdeadbeefdeadbeefull struct drm_xe_sync sync[1] = { { .type = DRM_XE_SYNC_TYPE_USER_FENCE, @@ -114,6 +120,7 @@ test_exec(int fd, struct drm_xe_engine_class_instance *eci, int i, j, b; int map_fd = -1; int64_t fence_timeout; + void *dummy; igt_assert(n_exec_queues <= MAX_N_EXECQUEUES); @@ -142,6 +149,17 @@ test_exec(int fd, struct drm_xe_engine_class_instance *eci, bo_size); igt_assert(data); } + if (flags & UNMAP_MAPPPING) { + dummy = mmap((void *)MAP_ADDRESS, bo_size, PROT_READ | + PROT_WRITE, MAP_SHARED | MAP_FIXED | + MAP_ANONYMOUS, -1, 0); + igt_assert(data != MAP_FAILED); + } + if (flags & FREE_MAPPPING) { + dummy = aligned_alloc(xe_get_default_alignment(fd), + bo_size); + igt_assert(dummy); + } } else { bo = xe_bo_create(fd, flags & VM_FOR_BO ? vm : 0, bo_size, vram_if_possible(fd, eci->gt_id), @@ -157,16 +175,21 @@ test_exec(int fd, struct drm_xe_engine_class_instance *eci, xe_bind_exec_queue_create(fd, vm, 0); else bind_exec_queues[i] = 0; - }; + } sync[0].addr = to_user_pointer(&data[0].vm_sync); - if (bo) + if (bo) { xe_vm_bind_async(fd, vm, bind_exec_queues[0], bo, 0, addr, bo_size, sync, 1); - else + } else { + if (flags & (FREE_MAPPPING | UNMAP_MAPPPING)) + xe_vm_bind_userptr_async(fd, vm, bind_exec_queues[0], + to_user_pointer(dummy), + dummy_addr, bo_size, 0, 0); xe_vm_bind_userptr_async(fd, vm, bind_exec_queues[0], to_user_pointer(data), addr, bo_size, sync, 1); + } #define ONE_SEC MS_TO_NS(1000) #define HUNDRED_SEC MS_TO_NS(100000) @@ -197,6 +220,12 @@ test_exec(int fd, struct drm_xe_engine_class_instance *eci, exec.address = batch_addr; xe_exec(fd, &exec); + if (flags & FREE_MAPPPING && !i) + free(dummy); + + if (flags & UNMAP_MAPPPING && !i) + munmap(dummy, bo_size); + if (flags & REBIND && i + 1 != n_execs) { xe_wait_ufence(fd, &data[i].exec_sync, USER_FENCE_VALUE, exec_queues[e], fence_timeout); @@ -489,6 +518,8 @@ igt_main { "basic", 0 }, { "preempt-fence-early", VM_FOR_BO | EXEC_QUEUE_EARLY }, { "userptr", USERPTR }, + { "userptr-free", USERPTR | FREE_MAPPPING }, + { "userptr-unmap", USERPTR | UNMAP_MAPPPING }, { "rebind", REBIND }, { "userptr-rebind", USERPTR | REBIND }, { "userptr-invalidate", USERPTR | INVALIDATE }, -- 2.34.1