From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhang Rui Subject: Re: [regression,bisected] restore of disks after suspend-to-disk broken in 3.9.x Date: Thu, 30 May 2013 18:50:36 +0800 Message-ID: <1369911036.2636.5.camel@rzhang1-mobl4> References: <1369631226.2159.5.camel@rzhang1-mobl4> <1369878386.3368.16.camel@rzhang1-mobl4> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-pm-owner@vger.kernel.org To: Giacomo Perale Cc: linux-acpi@vger.kernel.org, linux-pm@vger.kernel.org List-Id: linux-acpi@vger.kernel.org On Thu, 2013-05-30 at 10:04 +0200, Giacomo Perale wrote: > 2013/5/30 Zhang Rui : > > On Mon, 2013-05-27 at 22:07 +0200, Giacomo Perale wrote: > >> 2013/5/27 Zhang Rui : > >> > On Sun, 2013-05-26 at 15:51 +0200, Giacomo Perale wrote: > >> >> Hello, > >> >> > >> >> after upgrading from 3.8.7 to 3.9.x I noticed some slightly longer > >> >> delays when resuming from suspend-to-disk and a few new error messages > >> >> in the logs: > >> >> > >> >> > >> >> Not having connected the issue with the previous error messages I > >> >> initially blamed the disk and replaced it with a new one thinking it > >> >> was broken, but the problem persisted so I started a bisection run > >> >> that pointed to this commit: > >> >> > >> >> > >> >> b8bb6cb999858043489c1ddef08eed2127559169 is the first bad commit > >> >> commit b8bb6cb999858043489c1ddef08eed2127559169 > >> >> Author: Zhang Rui > >> >> Date: Thu Nov 22 15:45:02 2012 +0800 > >> >> > >> >> step_wise: Unify the code for both throttle and dethrottle > >> >> > >> >> Signed-off-by: Zhang Rui > >> >> > >> > I do not see how this would affect the sata hibernation/resume. > >> > but anyway, would you please try the four patches in comment #10, #11, > >> > #12 and #13 in > >> > https://bugzilla.kernel.org/show_bug.cgi?id=58301 > >> > and check if they help? > >> > > >> > thanks, > >> > rui > >> > > >> > >> No luck, with git head and those four patches hibernation is > >> completely broken, the system doesn't even shut down, it just loses > >> the disks (I'm attaching what I could get from /proc/kmsg to this > >> email). > >> > > then can you please try the patch at > > https://patchwork.kernel.org/patch/2633071/ > > on top of the four patches and see if it helps? > > > > thanks, > > rui > > > > Do you mean without Rafael's patch? > right. thanks, rui > Giacomo