From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932508AbbFQPSG (ORCPT ); Wed, 17 Jun 2015 11:18:06 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:43605 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754576AbbFQPSD (ORCPT ); Wed, 17 Jun 2015 11:18:03 -0400 Date: Wed, 17 Jun 2015 17:18:00 +0200 From: Pavel Machek To: Paul Bolle , imre.deak@intel.com, ville.syrjala@linux.intel.com, daniel.vetter@ffwll.ch, Linus Torvalds Cc: "Rafael J. Wysocki" , linus , lenb , kernel list , linux-acpi , linux-pm Subject: da2bc1b9db3351addd293e5b82757efe1f77ed1d breaks hibernation was Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing Message-ID: <20150617151800.GA12312@amd> References: <20150617121352.GA20553@amd> <7336539.OpACL0uN6J@vostro.rjw.lan> <20150617123333.GA21389@amd> <2490787.R6LzZId8tR@vostro.rjw.lan> <20150617131608.GA24030@amd> <1434549754.2400.5.camel@x220> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1434549754.2400.5.camel@x220> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! > And this all reminded me of https://lkml.org/lkml/2015/3/18/133 . So I > think that's another system not covered by commit ab3be73fa7b4 > ("drm/i915: gen4: work around hang during hibernation"). Yes, another system broken by commit da2bc1b9db3351addd293e5b82757efe1f77ed1d Author: Imre Deak Date: Thu Oct 23 19:23:26 2014 +0300 drm/i915: add poweroff_late handler The suspend_late handler saves some registers and powers off the device, so it doesn't have a big overhead. Calling it at S4 poweroff_late time makes the power off handling identical to the S3 suspend and S4 freeze handling, so do this for consistency. And no, adding another system to the "blacklist" is a not good idea, because we know there are other broken systems out there. Can we revert da2bc1b9db3351addd293e5b82757efe1f77ed1d, please? Consistency is nice, but not having regressions is even nicer. Pavel PS: Now I have real Linus and fake Linus on the Cc list. Or maybe two real Linuses, its hard to tell. -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html