drm/i915/gvt: Use common mapping routines for indirect_ctx object
We have the ability to map an object, so use it rather than opencode it
badly.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c b/drivers/gpu/drm/i915/gvt/cmd_parser.c
index d942da9..153943a 100644
--- a/drivers/gpu/drm/i915/gvt/cmd_parser.c
+++ b/drivers/gpu/drm/i915/gvt/cmd_parser.c
@@ -2717,7 +2717,7 @@
unsigned long guest_gma = wa_ctx->indirect_ctx.guest_gma;
struct drm_i915_gem_object *obj;
int ret = 0;
- void *dest = NULL;
+ void *map;
obj = i915_gem_object_create(dev,
roundup(ctx_size + CACHELINE_BYTES,
@@ -2725,18 +2725,12 @@
if (IS_ERR(obj))
return PTR_ERR(obj);
- ret = i915_gem_object_get_pages(obj);
- if (ret)
- goto put_obj;
-
- i915_gem_object_pin_pages(obj);
-
/* get the va of the shadow batch buffer */
- dest = (void *)vmap_batch(obj, 0, ctx_size + CACHELINE_BYTES);
- if (!dest) {
+ map = i915_gem_object_pin_map(obj, I915_MAP_WB);
+ if (IS_ERR(map)) {
gvt_err("failed to vmap shadow indirect ctx\n");
- ret = -ENOMEM;
- goto unpin_src;
+ ret = PTR_ERR(map);
+ goto put_obj;
}
ret = i915_gem_object_set_to_cpu_domain(obj, false);
@@ -2745,25 +2739,21 @@
goto unmap_src;
}
- wa_ctx->indirect_ctx.shadow_va = dest;
-
- memset(dest, 0, round_up(ctx_size + CACHELINE_BYTES, PAGE_SIZE));
-
ret = copy_gma_to_hva(wa_ctx->workload->vgpu,
wa_ctx->workload->vgpu->gtt.ggtt_mm,
- guest_gma, guest_gma + ctx_size, dest);
+ guest_gma, guest_gma + ctx_size,
+ map);
if (ret) {
gvt_err("fail to copy guest indirect ctx\n");
goto unmap_src;
}
wa_ctx->indirect_ctx.obj = obj;
+ wa_ctx->indirect_ctx.shadow_va = map;
return 0;
unmap_src:
- vunmap(dest);
-unpin_src:
- i915_gem_object_unpin_pages(wa_ctx->indirect_ctx.obj);
+ i915_gem_object_unpin_map(obj);
put_obj:
i915_gem_object_put(wa_ctx->indirect_ctx.obj);
return ret;