On Tue, Jun 30, 2015 at 11:31:54AM +0300, Jani Nikula wrote:
> On Mon, 29 Jun 2015, ville.syrjala@xxxxxxxxxxxxxxx wrote:
> > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> >
> > Stolen gets trashed during hibernation, so storing contexts there
> > is not a very good idea. On my IVB machines this leads to a totally
> > dead GPU on resume. A reboot is required to resurrect it. So let's
> > not store contexts where they will get trampled.
> >
> > This reverts commit 149c86e74fe44dcbac5e9f8d145c5fbc5dc21261.
>
> Looks like a *partial* revert of
>
> commit 149c86e74fe44dcbac5e9f8d145c5fbc5dc21261
> Author: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Date: Tue Apr 7 16:21:11 2015 +0100
>
> drm/i915: Allocate context objects from stolen
>
> and the not reverted hunk has been touched since in
>
> commit 3126a660f352b3fe48125a8a0b4fdbf85935d8bf
> Author: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Date: Thu Apr 30 17:30:50 2015 +0300
>
> drm/i915: checking IS_ERR() instead of NULL
>
> The commit message should say it's a partial revert and not claim
> otherwise, and document why not everything was reverted.
>
> The above commits are in Linus' tree.
The fixes I have planned to migrate context objects on hibernation are
not of the simple variety! Revert now, fix later.
Given Ville's discovery, it also makes
commit 8040513870399f1cb032cb8bc805df5042fedcdf
Author: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
Date: Thu Nov 15 11:32:29 2012 +0000
drm/i915: Allocate overlay registers from stolen memory
suspect.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
