lkml.org 
[lkml]   [2010]   [Sep]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[125/129] drm/i915: overlay on gen2 cant address above 1G
    2.6.35-stable review patch.  If anyone has any objections, please let us know.

    ------------------

    From: Daniel Vetter <daniel.vetter@ffwll.ch>

    commit 9f82d23846146990d475f6753be733e55788d88d upstream.

    So set the coherent dma mask accordingly. This dma mask is only used
    for physical objects, so it won't really matter allocation-wise.

    Now this never really surfaced because sane 32bit kernels only have 1G
    of lowmem. But some eager testers (distros?) still carry around the patch
    to adjust lowmem via a kconfig option. And the kernel seems to favour
    high allocations on boot-up, hence the overlay blowing up reliably.

    Because the patch is tiny and nicely shows how broken gen2 is it's imho
    worth to merge despite the fact that mucking around with the lowmem/
    highmem division is (no longer) supported.

    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=28318
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/gpu/drm/i915/i915_dma.c | 4 ++++
    1 file changed, 4 insertions(+)

    --- a/drivers/gpu/drm/i915/i915_dma.c
    +++ b/drivers/gpu/drm/i915/i915_dma.c
    @@ -2079,6 +2079,10 @@ int i915_driver_load(struct drm_device *
    goto free_priv;
    }

    + /* overlay on gen2 is broken and can't address above 1G */
    + if (IS_GEN2(dev))
    + dma_set_coherent_mask(&dev->pdev->dev, DMA_BIT_MASK(30));
    +
    dev_priv->regs = ioremap(base, size);
    if (!dev_priv->regs) {
    DRM_ERROR("failed to map registers\n");



    \
     
     \ /
      Last update: 2010-09-18 21:21    [W:4.929 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site