diff options
| author | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2023-10-05 23:21:51 -0400 |
|---|---|---|
| committer | Francois Dugast <francois.dugast@intel.com> | 2023-12-05 09:26:37 +0100 |
| commit | edd4b9325c4ec436440274f5872c4524d70915e8 (patch) | |
| tree | e95a9ec306a79117138b37c59638eae2e16bedfe /benchmarks | |
| parent | 405e1569a002dccf29513f4c810e95d918be0647 (diff) | |
xe_query: Kill visible_vram_if_possible
Let the caller set the flag and the xe_bo_query clear if
not needed.
Although the current helper makes the code cleaner, the
goal is to split the flags into placement and flags as two
different arguments on xe_bo_create. So, the flag decision
cannot be hidden under the helper.
v2: Fix one comment (Kamil Konieczny)
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Diffstat (limited to 'benchmarks')
| -rw-r--r-- | benchmarks/gem_wsim.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c index cb19ad505..47692e94f 100644 --- a/benchmarks/gem_wsim.c +++ b/benchmarks/gem_wsim.c @@ -1746,7 +1746,8 @@ xe_alloc_step_batch(struct workload *wrk, struct w_step *w) w->bb_size = ALIGN(PAGE_SIZE + xe_cs_prefetch_size(fd), xe_get_default_alignment(fd)); w->bb_handle = xe_bo_create(fd, vm->id, w->bb_size, - visible_vram_if_possible(fd, eq->hwe_list[0].gt_id)); + vram_if_possible(fd, eq->hwe_list[0].gt_id) | + DRM_XE_GEM_CREATE_FLAG_NEEDS_VISIBLE_VRAM); w->xe.data = xe_bo_map(fd, w->bb_handle, w->bb_size); w->xe.exec.address = intel_allocator_alloc_with_strategy(vm->ahnd, w->bb_handle, w->bb_size, |