drm/i915: Remove use of gtt_mappable_entries
authorBen Widawsky <ben@bwidawsk.net>
Thu, 17 Jan 2013 20:45:17 +0000 (12:45 -0800)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Sun, 20 Jan 2013 12:09:20 +0000 (13:09 +0100)
Mappable_end, ie. size is almost always what you want as opposed to the
number of entries. Since we already have that information, we can scrap
the number of entries and only calculate it when needed.

If gtt_start is !0, this will have slightly different behavior. This
difference can only occur in DRI1, and exists when we try to kick out
the firmware fb. The new code seems like a bugfix to me.

The other case where we've changed the behavior is during init we check
the mappable region against our current known upper and lower limits
(64MB, and 512MB). This now matches the comment, and makes things more
convenient after removing gtt_mappable_entries.

Also worth noting is the setting of mappable_end is taken out of setup
because we do it earlier now in the DRI2 case and therefore need to add
that tiny hunk to support the DRI1 IOCTL.

v2: Move up mappable end to before legacy AGP init

v3: Add the dev_priv inclusion here from previous rebase error in patch
5

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@gmail.com> (v2)
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
[danvet: squash in fix for a printk format flag mismatch warning.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_debugfs.c
drivers/gpu/drm/i915/i915_dma.c
drivers/gpu/drm/i915/i915_gem.c
drivers/gpu/drm/i915/i915_gem_gtt.c
drivers/gpu/drm/i915/intel_fb.c

index 773b23ecc83b4b5a3c20b9db6e6e1f7c8586adc2..90a6fc506dd5b2c39d724e0b7f508d991562d531 100644 (file)
@@ -258,7 +258,7 @@ static int i915_gem_object_info(struct seq_file *m, void* data)
        seq_printf(m, "%u fault mappable objects, %zu bytes\n",
                   count, size);
 
-       seq_printf(m, "%zu [%zu] gtt total\n",
+       seq_printf(m, "%zu [%lu] gtt total\n",
                   dev_priv->gtt.total,
                   dev_priv->gtt.mappable_end - dev_priv->gtt.start);
 
index 468d2a0fc37833df2f0966258e6c7c133f872c6b..3f70178c63caeb79f5445c3b2de3c18871ec01db 100644 (file)
@@ -1427,8 +1427,8 @@ static void i915_kick_out_firmware_fb(struct drm_i915_private *dev_priv)
                return;
 
        ap->ranges[0].base = dev_priv->gtt.mappable_base;
-       ap->ranges[0].size =
-               dev_priv->mm.gtt->gtt_mappable_entries << PAGE_SHIFT;
+       ap->ranges[0].size = dev_priv->gtt.mappable_end - dev_priv->gtt.start;
+
        primary =
                pdev->resource[PCI_ROM_RESOURCE].flags & IORESOURCE_ROM_SHADOW;
 
@@ -1542,7 +1542,7 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
                goto put_gmch;
        }
 
-       aperture_size = dev_priv->mm.gtt->gtt_mappable_entries << PAGE_SHIFT;
+       aperture_size = dev_priv->gtt.mappable_end;
 
        dev_priv->gtt.mappable =
                io_mapping_create_wc(dev_priv->gtt.mappable_base,
@@ -1699,7 +1699,7 @@ int i915_driver_unload(struct drm_device *dev)
        if (dev_priv->mm.gtt_mtrr >= 0) {
                mtrr_del(dev_priv->mm.gtt_mtrr,
                         dev_priv->gtt.mappable_base,
-                        dev_priv->mm.gtt->gtt_mappable_entries * PAGE_SIZE);
+                        dev_priv->gtt.mappable_end);
                dev_priv->mm.gtt_mtrr = -1;
        }
 
index 51fdf16181a749ec6ae95ac7cac2052814d51cbe..e4132ffb760914d0c739a60525480dbbdb8a6e06 100644 (file)
@@ -149,6 +149,7 @@ int
 i915_gem_init_ioctl(struct drm_device *dev, void *data,
                    struct drm_file *file)
 {
+       struct drm_i915_private *dev_priv = dev->dev_private;
        struct drm_i915_gem_init *args = data;
 
        if (drm_core_check_feature(dev, DRIVER_MODESET))
@@ -165,6 +166,7 @@ i915_gem_init_ioctl(struct drm_device *dev, void *data,
        mutex_lock(&dev->struct_mutex);
        i915_gem_setup_global_gtt(dev, args->gtt_start, args->gtt_end,
                                  args->gtt_end);
+       dev_priv->gtt.mappable_end = args->gtt_end;
        mutex_unlock(&dev->struct_mutex);
 
        return 0;
index 0b89305779536e7bf1a7696439612badcff43848..0f0db02c1e56d090ef3d0d53dca7ea99a31e10b2 100644 (file)
@@ -557,7 +557,6 @@ void i915_gem_setup_global_gtt(struct drm_device *dev,
        }
 
        dev_priv->gtt.start = start;
-       dev_priv->gtt.mappable_end = mappable_end;
        dev_priv->gtt.total = end - start;
 
        /* Clear any non-preallocated blocks */
@@ -596,7 +595,7 @@ void i915_gem_init_global_gtt(struct drm_device *dev)
        int ret;
 
        gtt_size = dev_priv->mm.gtt->gtt_total_entries << PAGE_SHIFT;
-       mappable_size = dev_priv->mm.gtt->gtt_mappable_entries << PAGE_SHIFT;
+       mappable_size = dev_priv->gtt.mappable_end;
 
        if (intel_enable_ppgtt(dev) && HAS_ALIASING_PPGTT(dev)) {
                /* PPGTT pdes are stolen from global gtt ptes, so shrink the
@@ -692,6 +691,7 @@ int i915_gem_gtt_init(struct drm_device *dev)
        int ret;
 
        dev_priv->gtt.mappable_base = pci_resource_start(dev->pdev, 2);
+       dev_priv->gtt.mappable_end = pci_resource_len(dev->pdev, 2);
 
        /* On modern platforms we need not worry ourself with the legacy
         * hostbridge query stuff. Skip it entirely
@@ -735,14 +735,13 @@ int i915_gem_gtt_init(struct drm_device *dev)
        else
                dev_priv->mm.gtt->stolen_size = gen7_get_stolen_size(snb_gmch_ctl);
 
-       dev_priv->mm.gtt->gtt_mappable_entries = pci_resource_len(dev->pdev, 2) >> PAGE_SHIFT;
        /* 64/512MB is the current min/max we actually know of, but this is just a
         * coarse sanity check.
         */
-       if ((dev_priv->mm.gtt->gtt_mappable_entries >> 8) < 64 ||
-           dev_priv->mm.gtt->gtt_mappable_entries > dev_priv->mm.gtt->gtt_total_entries) {
-               DRM_ERROR("Unknown GMADR entries (%d)\n",
-                         dev_priv->mm.gtt->gtt_mappable_entries);
+       if ((dev_priv->gtt.mappable_end < (64<<20) ||
+           (dev_priv->gtt.mappable_end > (512<<20)))) {
+               DRM_ERROR("Unknown GMADR size (%lx)\n",
+                         dev_priv->gtt.mappable_end);
                ret = -ENXIO;
                goto err_out;
        }
@@ -764,7 +763,7 @@ int i915_gem_gtt_init(struct drm_device *dev)
 
        /* GMADR is the PCI aperture used by SW to access tiled GFX surfaces in a linear fashion. */
        DRM_INFO("Memory usable by graphics device = %dM\n", dev_priv->mm.gtt->gtt_total_entries >> 8);
-       DRM_DEBUG_DRIVER("GMADR size = %dM\n", dev_priv->mm.gtt->gtt_mappable_entries >> 8);
+       DRM_DEBUG_DRIVER("GMADR size = %ldM\n", dev_priv->gtt.mappable_end >> 20);
        DRM_DEBUG_DRIVER("GTT stolen size = %dM\n", dev_priv->mm.gtt->stolen_size >> 20);
 
        return 0;
index ce02af8ca96e4b550ab5737ba4e867d99e0279cc..ce5f54498426eb62266054245623abddd9fbadec 100644 (file)
@@ -135,8 +135,7 @@ static int intelfb_create(struct intel_fbdev *ifbdev,
                goto out_unpin;
        }
        info->apertures->ranges[0].base = dev->mode_config.fb_base;
-       info->apertures->ranges[0].size =
-               dev_priv->mm.gtt->gtt_mappable_entries << PAGE_SHIFT;
+       info->apertures->ranges[0].size = dev_priv->gtt.mappable_end;
 
        info->fix.smem_start = dev->mode_config.fb_base + obj->gtt_offset;
        info->fix.smem_len = size;