[Intel-gfx] [PATCH 1/3] drm/i915/gtt: Reference mappable_end variable from pointer

[Chris Wilson]

On Mon, Mar 21, 2016 at 10:35:08AM +0100, Daniel Vetter wrote:
> On Fri, Mar 18, 2016 at 10:54:03AM +0200, Joonas Lahtinen wrote:
> > On pe, 2016-03-18 at 08:21 +0000, Chris Wilson wrote:
> > > On Fri, Mar 18, 2016 at 09:53:08AM +0200, Joonas Lahtinen wrote:
> > > > 
> > > > Reference variable value from pointer, not assumed pointer destination.
> > > > 
> > > > Since:
> > > > 
> > > > commit c44ef60e437019b8ca1dab8b4d2e8761fd4ce1e9
> > > > Author: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> > > > Date:   Thu Jun 25 18:35:05 2015 +0300
> > > > 
> > > >     drm/i915/gtt: Allow >= 4GB sizes for vm.
> > > > 
> > > > Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> > > > Signed-off-by: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> > > Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> > > 
> > > That reminds, could you pull
> > > 
> > > https://cgit.freedesktop.org/~ickle/linux-2.6/commit/?h=breadcrumbs&id=8ecc77b5864d4911e9fb9b92be8b2e98c3f2261f
> > > [drm/i915: Codify our assumption that the Global GTT is <= 4GiB]
> > > 
> > > into this series?
> > 
> > Done, sent series v2 with one typo fix and that patch rebased (and
> > tweaked 1ull -> 1ULL).
> > 
> > So if somebody would review the new patch, we're good to merge.
> 
> Patch is wrong Chris, you can review it, and then push the entire pile

Which patch? The only one outstanding is my suggestion to add the WARN.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre