3 files changed, 11 insertions, 30 deletions

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index dee8d5c7a235..d3274164143d 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -406,6 +406,7 @@ struct i915_gtt {
                                    unsigned int pg_start,
                                    enum i915_cache_level cache_level);
 };
+#define gtt_total_entries(gtt) ((gtt).total >> PAGE_SHIFT)
 
 #define I915_PPGTT_PD_ENTRIES 512
 #define I915_PPGTT_PT_ENTRIES 1024
@@ -696,8 +697,6 @@ struct intel_l3_parity {
 };
 
 struct i915_gem_mm {
-        /** Bridge to intel-gtt-ko */
-        struct intel_gtt *gtt;
         /** Memory allocator for GTT stolen memory */
         struct drm_mm stolen;
         /** Memory allocator for GTT */
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 87249e8cd4a0..f7e0018626bd 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -182,7 +182,8 @@ static int gen6_ppgtt_init(struct i915_hw_ppgtt *ppgtt)
         /* ppgtt PDEs reside in the global gtt pagetable, which has 512*1024
          * entries. For aliasing ppgtt support we just steal them at the end for
          * now. */
-        first_pd_entry_in_global_pt = dev_priv->mm.gtt->gtt_total_entries - I915_PPGTT_PD_ENTRIES;
+        first_pd_entry_in_global_pt =
+                gtt_total_entries(dev_priv->gtt) - I915_PPGTT_PD_ENTRIES;
 
         ppgtt->num_pd_entries = I915_PPGTT_PD_ENTRIES;
         ppgtt->clear_range = gen6_ppgtt_clear_range;
@@ -473,7 +474,7 @@ static void gen6_ggtt_clear_range(struct drm_device *dev,
         struct drm_i915_private *dev_priv = dev->dev_private;
         gtt_pte_t scratch_pte;
         gtt_pte_t __iomem *gtt_base = (gtt_pte_t __iomem *) dev_priv->gtt.gsm + first_entry;
-        const int max_entries = dev_priv->mm.gtt->gtt_total_entries - first_entry;
+        const int max_entries = gtt_total_entries(dev_priv->gtt) - first_entry;
         int i;
 
         if (WARN(num_entries > max_entries,
@@ -634,7 +635,7 @@ void i915_gem_init_global_gtt(struct drm_device *dev)
         unsigned long gtt_size, mappable_size;
         int ret;
 
-        gtt_size = dev_priv->mm.gtt->gtt_total_entries << PAGE_SHIFT;
+        gtt_size = dev_priv->gtt.total;
         mappable_size = dev_priv->gtt.mappable_end;
 
         if (intel_enable_ppgtt(dev) && HAS_ALIASING_PPGTT(dev)) {
@@ -778,7 +779,6 @@ void gen6_gmch_remove(struct drm_device *dev)
         struct drm_i915_private *dev_priv = dev->dev_private;
         iounmap(dev_priv->gtt.gsm);
         teardown_scratch_page(dev_priv->dev);
-        kfree(dev_priv->mm.gtt);
 }
 
 static int i915_gmch_probe(struct drm_device *dev,
@@ -788,22 +788,13 @@ static int i915_gmch_probe(struct drm_device *dev,
         struct drm_i915_private *dev_priv = dev->dev_private;
         int ret;
 
-        /* This is a temporary hack to make the code cleaner in
-         * i915_gem_gtt_init. I promise it will go away very shortly. */
-        kfree(dev_priv->mm.gtt);
-
         ret = intel_gmch_probe(dev_priv->bridge_dev, dev_priv->dev->pdev, NULL);
         if (!ret) {
                 DRM_ERROR("failed to set up gmch\n");
                 return -EIO;
         }
 
-        dev_priv->mm.gtt = intel_gtt_get();
-        if (!dev_priv->mm.gtt) {
-                DRM_ERROR("Failed to initialize GTT\n");
-                intel_gmch_remove();
-                return -ENODEV;
-        }
+        intel_gtt_get(gtt_total, stolen);
 
         dev_priv->gtt.do_idle_maps = needs_idle_maps(dev_priv->dev);
         dev_priv->gtt.gtt_clear_range = i915_ggtt_clear_range;
@@ -824,10 +815,6 @@ int i915_gem_gtt_init(struct drm_device *dev)
         unsigned long gtt_size;
         int ret;
 
-        dev_priv->mm.gtt = kzalloc(sizeof(*dev_priv->mm.gtt), GFP_KERNEL);
-        if (!dev_priv->mm.gtt)
-                return -ENOMEM;
-
         gtt->mappable_base = pci_resource_start(dev->pdev, 2);
         gtt->mappable_end = pci_resource_len(dev->pdev, 2);
 
@@ -841,13 +828,8 @@ int i915_gem_gtt_init(struct drm_device *dev)
 
         ret = dev_priv->gtt.gtt_probe(dev, &dev_priv->gtt.total,
                                      &dev_priv->gtt.stolen_size);
-        if (ret) {
-                kfree(dev_priv->mm.gtt);
+        if (ret)
                 return ret;
-        }
-
-        dev_priv->mm.gtt->gtt_total_entries = dev_priv->gtt.total >> PAGE_SHIFT;
-        dev_priv->mm.gtt->stolen_size = dev_priv->gtt.stolen_size;
 
         gtt_size = (dev_priv->gtt.total >> PAGE_SHIFT) * sizeof(gtt_pte_t);
 
diff --git a/drivers/gpu/drm/i915/i915_gem_stolen.c b/drivers/gpu/drm/i915/i915_gem_stolen.c
index f21ae17e298f..69d97cbac13c 100644
--- a/drivers/gpu/drm/i915/i915_gem_stolen.c
+++ b/drivers/gpu/drm/i915/i915_gem_stolen.c
@@ -187,11 +187,11 @@ int i915_gem_init_stolen(struct drm_device *dev)
         if (dev_priv->mm.stolen_base == 0)
                 return 0;
 
-        DRM_DEBUG_KMS("found %d bytes of stolen memory at %08lx\n",
-                      dev_priv->mm.gtt->stolen_size, dev_priv->mm.stolen_base);
+        DRM_DEBUG_KMS("found %zd bytes of stolen memory at %08lx\n",
+                      dev_priv->gtt.stolen_size, dev_priv->mm.stolen_base);
 
         /* Basic memrange allocator for stolen space */
-        drm_mm_init(&dev_priv->mm.stolen, 0, dev_priv->mm.gtt->stolen_size);
+        drm_mm_init(&dev_priv->mm.stolen, 0, dev_priv->gtt.stolen_size);
 
         return 0;
 }
@@ -205,7 +205,7 @@ i915_pages_create_for_stolen(struct drm_device *dev,
         struct scatterlist *sg;
 
         DRM_DEBUG_DRIVER("offset=0x%x, size=%d\n", offset, size);
-        BUG_ON(offset > dev_priv->mm.gtt->stolen_size - size);
+        BUG_ON(offset > dev_priv->gtt.stolen_size - size);
 
         /* We hide that we have no struct page backing our stolen object
          * by wrapping the contiguous physical allocation with a fake