drm/i915: Tidy gen6_write_pde()
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 15 Feb 2017 08:43:45 +0000 (08:43 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 15 Feb 2017 10:07:24 +0000 (10:07 +0000)
Stop passing around unused parameters makes the code more compact.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170215084357.19977-11-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_gem_gtt.c

index 52586432d5e8af0bf85232ca41d9a65a130fc185..17c5c1a5b6524016e1c6ffc5e31f73906c2c8092 100644 (file)
@@ -1709,36 +1709,28 @@ static void gen6_dump_ppgtt(struct i915_hw_ppgtt *ppgtt, struct seq_file *m)
 }
 
 /* Write pde (index) from the page directory @pd to the page table @pt */
-static void gen6_write_pde(struct i915_page_directory *pd,
-                           const int pde, struct i915_page_table *pt)
+static inline void gen6_write_pde(const struct i915_hw_ppgtt *ppgtt,
+                                 const unsigned int pde,
+                                 const struct i915_page_table *pt)
 {
        /* Caller needs to make sure the write completes if necessary */
-       struct i915_hw_ppgtt *ppgtt =
-               container_of(pd, struct i915_hw_ppgtt, pd);
-       u32 pd_entry;
-
-       pd_entry = GEN6_PDE_ADDR_ENCODE(px_dma(pt));
-       pd_entry |= GEN6_PDE_VALID;
-
-       writel(pd_entry, ppgtt->pd_addr + pde);
+       writel_relaxed(GEN6_PDE_ADDR_ENCODE(px_dma(pt)) | GEN6_PDE_VALID,
+                      ppgtt->pd_addr + pde);
 }
 
 /* Write all the page tables found in the ppgtt structure to incrementing page
  * directories. */
-static void gen6_write_page_range(struct drm_i915_private *dev_priv,
-                                 struct i915_page_directory *pd,
+static void gen6_write_page_range(struct i915_hw_ppgtt *ppgtt,
                                  uint32_t start, uint32_t length)
 {
-       struct i915_ggtt *ggtt = &dev_priv->ggtt;
        struct i915_page_table *pt;
-       uint32_t pde;
+       unsigned int pde;
 
-       gen6_for_each_pde(pt, pd, start, length, pde)
-               gen6_write_pde(pd, pde, pt);
+       gen6_for_each_pde(pt, &ppgtt->pd, start, length, pde)
+               gen6_write_pde(ppgtt, pde, pt);
+       wmb();
 
-       /* Make sure write is complete before other code can use this page
-        * table. Also require for WC mapped PTEs */
-       readl(ggtt->gsm);
+       mark_tlbs_dirty(ppgtt);
 }
 
 static uint32_t get_pd_offset(struct i915_hw_ppgtt *ppgtt)
@@ -2003,7 +1995,7 @@ static int gen6_alloc_va_range(struct i915_address_space *vm,
                           gen6_pte_count(start, length));
 
                if (__test_and_clear_bit(pde, new_page_tables))
-                       gen6_write_pde(&ppgtt->pd, pde, pt);
+                       gen6_write_pde(ppgtt, pde, pt);
 
                trace_i915_page_table_entry_map(vm, pde, pt,
                                         gen6_pte_index(start),
@@ -2161,7 +2153,7 @@ static int gen6_ppgtt_init(struct i915_hw_ppgtt *ppgtt)
        ppgtt->debug_dump = gen6_dump_ppgtt;
 
        gen6_scratch_va_range(ppgtt, 0, ppgtt->base.total);
-       gen6_write_page_range(dev_priv, &ppgtt->pd, 0, ppgtt->base.total);
+       gen6_write_page_range(ppgtt, 0, ppgtt->base.total);
 
        ret = gen6_alloc_va_range(&ppgtt->base, 0, ppgtt->base.total);
        if (ret) {
@@ -3395,8 +3387,6 @@ void i915_gem_restore_gtt_mappings(struct drm_i915_private *dev_priv)
                struct i915_address_space *vm;
 
                list_for_each_entry(vm, &dev_priv->vm_list, global_link) {
-                       /* TODO: Perhaps it shouldn't be gen6 specific */
-
                        struct i915_hw_ppgtt *ppgtt;
 
                        if (i915_is_ggtt(vm))
@@ -3404,8 +3394,7 @@ void i915_gem_restore_gtt_mappings(struct drm_i915_private *dev_priv)
                        else
                                ppgtt = i915_vm_to_ppgtt(vm);
 
-                       gen6_write_page_range(dev_priv, &ppgtt->pd,
-                                             0, ppgtt->base.total);
+                       gen6_write_page_range(ppgtt, 0, ppgtt->base.total);
                }
        }