[PATCH] drm/mm: Adjust start/end for coloring first

[Chris Wilson]

On Mon, May 19, 2014 at 10:14:27AM +0200, Daniel Vetter wrote:
> On Mon, May 19, 2014 at 07:52:37AM +0100, Chris Wilson wrote:
> > The current user of the coloring will adjust the end points of the node
> > to leave a hole between disjoint memory types. This adjustment must be
> > performed first or else the derived size will conflict with the
> > adjustment and trigger the BUG_ON sanity checks that the node is within
> > bounds.
> > 
> > Fixes regression from
> > commit 62347f9e0f81d50e9b0923ec8a192f60ab7a1801
> > Author: Lauri Kasanen <cand at gmx.com>
> > Date:   Wed Apr 2 20:03:57 2014 +0300
> > 
> >     drm: Add support for two-ended allocation, v3
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> 
> Do we have a bugzilla for this, or why did igt not scream about this
> failure?

How would igt scream? Look at the patch and think of how many possible
ways the current kernel would explode. Then think about how they are
exposed to userspace.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre