From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id AA5C310E21E for ; Thu, 9 Nov 2023 13:40:45 +0000 (UTC) From: Matthew Auld To: igt-dev@lists.freedesktop.org Date: Thu, 9 Nov 2023 13:38:22 +0000 Message-ID: <20231109133833.503891-5-matthew.auld@intel.com> In-Reply-To: <20231109133833.503891-1-matthew.auld@intel.com> References: <20231109133833.503891-1-matthew.auld@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t v9 04/15] lib/xe: support explicit cpu_caching gem_create List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Most tests shouldn't about such things, so likely it's just a case of picking the most sane default. However we also add some helpers for the tests that do care. v2: Rebase on coh_mode removal Signed-off-by: Matthew Auld Cc: José Roberto de Souza Cc: Pallavi Mishra Reviewed-by: Pallavi Mishra --- lib/xe/xe_ioctl.c | 55 ++++++++++++++++++++++++++++++++--------- lib/xe/xe_ioctl.h | 5 ++++ tests/intel/xe_create.c | 1 + 3 files changed, 50 insertions(+), 11 deletions(-) diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c index 895e3bd4e..9010380c3 100644 --- a/lib/xe/xe_ioctl.c +++ b/lib/xe/xe_ioctl.c @@ -226,13 +226,24 @@ void xe_vm_destroy(int fd, uint32_t vm) igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_XE_VM_DESTROY, &destroy), 0); } -uint32_t __xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags, - uint32_t *handle) +uint16_t __xe_default_cpu_caching_from_flags(int fd, uint32_t flags) +{ + if ((flags & all_memory_regions(fd)) != system_memory(fd) || + flags & XE_GEM_CREATE_FLAG_SCANOUT) + /* VRAM placements or scanout should always use WC */ + return DRM_XE_GEM_CPU_CACHING_WC; + + return DRM_XE_GEM_CPU_CACHING_WB; +} + +static uint32_t ___xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags, + uint16_t cpu_caching, uint32_t *handle) { struct drm_xe_gem_create create = { .vm_id = vm, .size = size, .flags = flags, + .cpu_caching = cpu_caching, }; int err; @@ -242,6 +253,15 @@ uint32_t __xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags *handle = create.handle; return 0; + +} + +uint32_t __xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags, + uint32_t *handle) +{ + uint16_t cpu_caching = __xe_default_cpu_caching_from_flags(fd, flags); + + return ___xe_bo_create_flags(fd, vm, size, flags, cpu_caching, handle); } uint32_t xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags) @@ -253,19 +273,32 @@ uint32_t xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags) return handle; } +uint32_t __xe_bo_create_caching(int fd, uint32_t vm, uint64_t size, uint32_t flags, + uint16_t cpu_caching, uint32_t *handle) +{ + return ___xe_bo_create_flags(fd, vm, size, flags, cpu_caching, + handle); +} + +uint32_t xe_bo_create_caching(int fd, uint32_t vm, uint64_t size, uint32_t flags, + uint16_t cpu_caching) +{ + uint32_t handle; + + igt_assert_eq(__xe_bo_create_caching(fd, vm, size, flags, + cpu_caching, &handle), 0); + + return handle; +} + uint32_t xe_bo_create(int fd, int gt, uint32_t vm, uint64_t size) { - struct drm_xe_gem_create create = { - .vm_id = vm, - .size = size, - .flags = vram_if_possible(fd, gt), - }; - int err; + uint32_t handle; - err = igt_ioctl(fd, DRM_IOCTL_XE_GEM_CREATE, &create); - igt_assert_eq(err, 0); + igt_assert_eq(__xe_bo_create_flags(fd, vm, size, vram_if_possible(fd, gt), + &handle), 0); - return create.handle; + return handle; } uint32_t xe_bind_exec_queue_create(int fd, uint32_t vm, uint64_t ext, bool async) diff --git a/lib/xe/xe_ioctl.h b/lib/xe/xe_ioctl.h index a8dbcf376..3681a6977 100644 --- a/lib/xe/xe_ioctl.h +++ b/lib/xe/xe_ioctl.h @@ -67,6 +67,11 @@ void xe_vm_destroy(int fd, uint32_t vm); uint32_t __xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags, uint32_t *handle); uint32_t xe_bo_create_flags(int fd, uint32_t vm, uint64_t size, uint32_t flags); +uint32_t __xe_bo_create_caching(int fd, uint32_t vm, uint64_t size, uint32_t flags, + uint16_t cpu_caching, uint32_t *handle); +uint32_t xe_bo_create_caching(int fd, uint32_t vm, uint64_t size, uint32_t flags, + uint16_t cpu_caching); +uint16_t __xe_default_cpu_caching_from_flags(int fd, uint32_t flags); uint32_t xe_bo_create(int fd, int gt, uint32_t vm, uint64_t size); uint32_t xe_exec_queue_create(int fd, uint32_t vm, struct drm_xe_engine_class_instance *instance, diff --git a/tests/intel/xe_create.c b/tests/intel/xe_create.c index d99bd51cf..587df2dfc 100644 --- a/tests/intel/xe_create.c +++ b/tests/intel/xe_create.c @@ -25,6 +25,7 @@ static int __create_bo(int fd, uint32_t vm, uint64_t size, uint32_t flags, .vm_id = vm, .size = size, .flags = flags, + .cpu_caching = __xe_default_cpu_caching_from_flags(fd, flags), }; int ret = 0; -- 2.41.0