From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by gabe.freedesktop.org (Postfix) with ESMTPS id DBB8F10E329 for ; Mon, 24 Jul 2023 14:16:38 +0000 (UTC) From: Matthew Auld To: igt-dev@lists.freedesktop.org Date: Mon, 24 Jul 2023 15:15:59 +0100 Message-ID: <20230724141602.47733-4-matthew.auld@intel.com> In-Reply-To: <20230724141602.47733-1-matthew.auld@intel.com> References: <20230724141602.47733-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 v5 3/6] lib/xe: handle small-bar systems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: The spinner stuff wants to use mmap. The fb related stuff might want to CPU access the buffer, so just always ask for it. And the batch buffer stuff is pretty much always going to need to CPU access the buffer. Signed-off-by: Matthew Auld Cc: Kamil Konieczny Cc: Gwan-gyeong Mun Reviewed-by: José Roberto de Souza --- lib/igt_draw.c | 2 +- lib/igt_fb.c | 2 +- lib/intel_batchbuffer.c | 4 ++-- lib/xe/xe_spin.c | 6 ++++-- 4 files changed, 8 insertions(+), 6 deletions(-) diff --git a/lib/igt_draw.c b/lib/igt_draw.c index 1f814bfc8..13b90702c 100644 --- a/lib/igt_draw.c +++ b/lib/igt_draw.c @@ -794,7 +794,7 @@ static void draw_rect_render(int fd, struct cmd_data *cmd_data, else tmp.handle = xe_bo_create_flags(fd, 0, ALIGN(tmp.size, xe_get_default_alignment(fd)), - vram_if_possible(fd, 0)); + visible_vram_if_possible(fd, 0)); tmp.stride = rect->w * pixel_size; tmp.bpp = buf->bpp; diff --git a/lib/igt_fb.c b/lib/igt_fb.c index 1814e8db1..17738d7f1 100644 --- a/lib/igt_fb.c +++ b/lib/igt_fb.c @@ -1194,7 +1194,7 @@ static int create_bo_for_fb(struct igt_fb *fb, bool prefer_sysmem) igt_assert(err == 0 || err == -EOPNOTSUPP); } else if (is_xe_device(fd)) { fb->gem_handle = xe_bo_create_flags(fd, 0, fb->size, - vram_if_possible(fd, 0)); + visible_vram_if_possible(fd, 0)); } else if (is_vc4_device(fd)) { fb->gem_handle = igt_vc4_create_bo(fd, fb->size); diff --git a/lib/intel_batchbuffer.c b/lib/intel_batchbuffer.c index 0408d62ba..163d39d6b 100644 --- a/lib/intel_batchbuffer.c +++ b/lib/intel_batchbuffer.c @@ -943,7 +943,7 @@ __intel_bb_create(int fd, uint32_t ctx, uint32_t vm, const intel_ctx_cfg_t *cfg, ibb->alignment = xe_get_default_alignment(fd); size = ALIGN(size, ibb->alignment); - ibb->handle = xe_bo_create_flags(fd, 0, size, vram_if_possible(fd, 0)); + ibb->handle = xe_bo_create_flags(fd, 0, size, visible_vram_if_possible(fd, 0)); /* Limit to 48-bit due to MI_* address limitation */ ibb->gtt_size = 1ull << min_t(uint32_t, xe_va_bits(fd), 48); @@ -1399,7 +1399,7 @@ void intel_bb_reset(struct intel_bb *ibb, bool purge_objects_cache) ibb->handle = gem_create(ibb->fd, ibb->size); else ibb->handle = xe_bo_create_flags(ibb->fd, 0, ibb->size, - vram_if_possible(ibb->fd, 0)); + visible_vram_if_possible(ibb->fd, 0)); /* Reacquire offset for RELOC and SIMPLE */ if (ibb->allocator_type == INTEL_ALLOCATOR_SIMPLE || diff --git a/lib/xe/xe_spin.c b/lib/xe/xe_spin.c index 529b76844..e799fcfc5 100644 --- a/lib/xe/xe_spin.c +++ b/lib/xe/xe_spin.c @@ -126,7 +126,8 @@ xe_spin_create(int fd, const struct igt_spin_factory *opt) spin->engine = xe_engine_create_class(fd, spin->vm, DRM_XE_ENGINE_CLASS_COPY); } - spin->handle = xe_bo_create(fd, 0, spin->vm, bo_size); + spin->handle = xe_bo_create_flags(fd, spin->vm, bo_size, + visible_vram_if_possible(fd, 0)); xe_spin = xe_bo_map(fd, spin->handle, bo_size); addr = intel_allocator_alloc_with_strategy(ahnd, spin->handle, bo_size, 0, ALLOC_STRATEGY_LOW_TO_HIGH); xe_vm_bind_sync(fd, spin->vm, spin->handle, 0, addr, bo_size); @@ -201,7 +202,8 @@ void xe_cork_init(int fd, struct drm_xe_engine_class_instance *hwe, vm = xe_vm_create(fd, 0, 0); - bo = xe_bo_create(fd, hwe->gt_id, vm, bo_size); + bo = xe_bo_create_flags(fd, vm, bo_size, + visible_vram_if_possible(fd, hwe->gt_id)); spin = xe_bo_map(fd, bo, 0x1000); xe_vm_bind_sync(fd, vm, bo, 0, addr, bo_size); -- 2.41.0