[3.19.y-ckt stable] Patch "drm/i915: apply the PCI_D0/D3 hibernation workaround everywhere on pre GEN6" has been added to staging queue

[Kamal Mostafa]

On Tue, 2015-10-20 at 14:34 +0200, Pavel Machek wrote:
> On Tue 2015-10-20 09:46:19, Jani Nikula wrote:
> > On Tue, 20 Oct 2015, Pavel Machek <pavel at ucw.cz> wrote:
> > > On Mon 2015-10-19 15:39:48, Kamal Mostafa wrote:
> > >> This is a note to let you know that I have just added a patch titled
> > >> 
> > >>     drm/i915: apply the PCI_D0/D3 hibernation workaround everywhere on pre GEN6
> > >> 
> > >> to the linux-3.19.y-queue branch of the 3.19.y-ckt extended stable tree 
> > >> which can be found at:
> > >> 
> > >>     http://kernel.ubuntu.com/git/ubuntu/linux.git/log/?h=linux-3.19.y-queue
> > >> 
> > >> This patch is scheduled to be released in version 3.19.8-ckt8.
> > >> 
> > >> If you, or anyone else, feels it should not be added to this tree, please 
> > >> reply to this email.
> > >
> > > As this says:
> > >
> > >> commit da2bc1b9db3351addd293e5b82757efe1f77ed1d
> > >> Author: Imre Deak <imre.deak at intel.com>
> > >> Date:   Thu Oct 23 19:23:26 2014 +0300
> > >> 
> > >>     drm/i915: add poweroff_late handler
> > >> 
> > >> introduced a regression on old platforms during hibernation. A workaround was
> > >> added in
> > >
> > > commit da2bc introduced a regression, and it should be simply
> > > reverted, attempts to "fix" it may be suitable for -mainline, but not
> > > for stable.
> > >
> > > Just revert it. It does not provide any real benefit, anyway.
> > 
> > This is not how stable works. If you have problems with current
> > upstream, please report them upstream.
> 
> I don't know about problem with the current upstream.
> 
> But for stable, best solution is to just revert da2bc, rather than try
> luck with fixing it.
> 									Pavel

Hi Pavel and Jani-

Thanks very much for reviewing this.

My inclination is to proceed with applying this patch (5487557
"drm/i915: apply the PCI_D0/D3 hibernation workaround everywhere on pre
GEN6") to 3.19-stable, because

 a. it has also been applied to 4.1-stable and 4.2-stable;

 b. neither mainline nor any other stable has reverted da2bc1b
"drm/i915: add poweroff_late handler"

 -Kamal