From: Chris Wilson <[email protected]>

Since the vma's backing store is flushed upon first creation, remove the
manual calls to set-to-gtt-domain.

v2 (Tvrtko):
 * Rebase.

Signed-off-by: Chris Wilson <[email protected]>
Reviewed-by: Matthew Auld <[email protected]> # v1
Signed-off-by: Tvrtko Ursulin <[email protected]>
---
 .../gpu/drm/i915/gem/selftests/i915_gem_mman.c   | 16 ----------------
 drivers/gpu/drm/i915/selftests/i915_vma.c        |  6 ------
 2 files changed, 22 deletions(-)

diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c 
b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
index 05a3b29f545e..886e2446e081 100644
--- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
+++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
@@ -104,14 +104,6 @@ static int check_partial_mapping(struct 
drm_i915_gem_object *obj,
        GEM_BUG_ON(i915_gem_object_get_tiling(obj) != tile->tiling);
        GEM_BUG_ON(i915_gem_object_get_stride(obj) != tile->stride);
 
-       i915_gem_object_lock(obj, NULL);
-       err = i915_gem_object_set_to_gtt_domain(obj, true);
-       i915_gem_object_unlock(obj);
-       if (err) {
-               pr_err("Failed to flush to GTT write domain; err=%d\n", err);
-               return err;
-       }
-
        page = i915_prandom_u32_max_state(npages, prng);
        view = compute_partial_view(obj, page, MIN_CHUNK_PAGES);
 
@@ -189,14 +181,6 @@ static int check_partial_mappings(struct 
drm_i915_gem_object *obj,
        GEM_BUG_ON(i915_gem_object_get_tiling(obj) != tile->tiling);
        GEM_BUG_ON(i915_gem_object_get_stride(obj) != tile->stride);
 
-       i915_gem_object_lock(obj, NULL);
-       err = i915_gem_object_set_to_gtt_domain(obj, true);
-       i915_gem_object_unlock(obj);
-       if (err) {
-               pr_err("Failed to flush to GTT write domain; err=%d\n", err);
-               return err;
-       }
-
        for_each_prime_number_from(page, 1, npages) {
                struct i915_ggtt_view view =
                        compute_partial_view(obj, page, MIN_CHUNK_PAGES);
diff --git a/drivers/gpu/drm/i915/selftests/i915_vma.c 
b/drivers/gpu/drm/i915/selftests/i915_vma.c
index dd0607254a95..24a806801883 100644
--- a/drivers/gpu/drm/i915/selftests/i915_vma.c
+++ b/drivers/gpu/drm/i915/selftests/i915_vma.c
@@ -987,12 +987,6 @@ static int igt_vma_remapped_gtt(void *arg)
                        u32 __iomem *map;
                        unsigned int x, y;
 
-                       i915_gem_object_lock(obj, NULL);
-                       err = i915_gem_object_set_to_gtt_domain(obj, true);
-                       i915_gem_object_unlock(obj);
-                       if (err)
-                               goto out;
-
                        if (!plane_info[0].dst_stride)
                                plane_info[0].dst_stride = *t == 
I915_GGTT_VIEW_ROTATED ?
                                                                 p->height : 
p->width;
-- 
2.30.2

Reply via email to