From owner-freebsd-bugs@FreeBSD.ORG  Sat Aug 31 17:20:01 2013
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
Delivered-To: freebsd-bugs@smarthost.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115])
 (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTP id D8C4E42C
 for <freebsd-bugs@smarthost.ysv.freebsd.org>;
 Sat, 31 Aug 2013 17:20:01 +0000 (UTC)
 (envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (freefall.freebsd.org
 [IPv6:2001:1900:2254:206c::16:87])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (No client certificate requested)
 by mx1.freebsd.org (Postfix) with ESMTPS id AA6072BBE
 for <freebsd-bugs@smarthost.ysv.freebsd.org>;
 Sat, 31 Aug 2013 17:20:01 +0000 (UTC)
Received: from freefall.freebsd.org (localhost [127.0.0.1])
 by freefall.freebsd.org (8.14.7/8.14.7) with ESMTP id r7VHK1g8081073
 for <freebsd-bugs@freefall.freebsd.org>; Sat, 31 Aug 2013 17:20:01 GMT
 (envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
 by freefall.freebsd.org (8.14.7/8.14.7/Submit) id r7VHK17f081071;
 Sat, 31 Aug 2013 17:20:01 GMT (envelope-from gnats)
Date: Sat, 31 Aug 2013 17:20:01 GMT
Message-Id: <201308311720.r7VHK17f081071@freefall.freebsd.org>
To: freebsd-bugs@FreeBSD.org
Cc: 
From: Mike Harding <mvharding@gmail.com>
Subject: Re: kern/181632: 9.2-RC3 - on resume from suspend, disk operations
 are slower
X-BeenThere: freebsd-bugs@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
Reply-To: Mike Harding <mvharding@gmail.com>
List-Id: Bug reports <freebsd-bugs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-bugs>,
 <mailto:freebsd-bugs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-bugs>
List-Post: <mailto:freebsd-bugs@freebsd.org>
List-Help: <mailto:freebsd-bugs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
 <mailto:freebsd-bugs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 31 Aug 2013 17:20:01 -0000

The following reply was made to PR kern/181632; it has been noted by GNATS.

From: Mike Harding <mvharding@gmail.com>
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/181632: 9.2-RC3 - on resume from suspend, disk operations
 are slower
Date: Sat, 31 Aug 2013 10:12:00 -0700

 --001a11c388b2efa98404e5416fa7
 Content-Type: text/plain; charset=ISO-8859-1
 
 I reverted the single line at
 
 http://svnweb.freebsd.org/base/stable/9/sys/dev/acpica/acpi_cpu.c?annotate=244616&pathrev=244616#l978
 
 to the previous version which was just
 
 ACPI_ENABLE_IRQS();
 
 instead of
 
 acpi_cpu_c1();
 
 and the problem does not occur.  The second line does 'sti; hlt' and the
 former just does 'sti'.
 
 Given that the code says
 
 /* If disabled, take the safe path. */  977 if (is_idle_disabled(sc)) {  978
 acpi_cpu_c1();  979 return;  980 }
 and then does a 'hlt' or idle if idle is disabled, this might be a problem.
 
 --001a11c388b2efa98404e5416fa7
 Content-Type: text/html; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"ltr"><div class=3D""><img class=3D"" id=3D":2re" src=3D"https:/=
 /mail.google.com/mail/u/0/images/cleardot.gif" alt=3D""></div><div><div>I r=
 everted the single line at<br><br><a href=3D"http://svnweb.freebsd.org/base=
 /stable/9/sys/dev/acpica/acpi_cpu.c?annotate=3D244616&amp;pathrev=3D244616#=
 l978" target=3D"_blank">http://svnweb.freebsd.org/base/stable/9/sys/dev/acp=
 ica/acpi_cpu.c?annotate=3D244616&amp;pathrev=3D244616#l978</a><br>
 
 <br></div>to the previous version which was just<br><br>ACPI_ENABLE_IRQS();=
 <br><br></div><div>instead of<br><br>acpi_cpu_c1();<br><br></div><div>and t=
 he problem does not occur.=A0 The second line does &#39;sti; hlt&#39; and t=
 he former just does &#39;sti&#39;.<br>
 
 <br></div><div>Given that the code says<br></div><div><br></div><div class=
 =3D"gmail_extra"><table cellpadding=3D"0" cellspacing=3D"0"><tbody><tr><td>=
     /* If disabled, take the safe path. */</td>
 </tr>
 
 
 
 
 <tr>
 <td>977</td>
 
 <td>    if (is_idle_disabled(sc)) {</td>
 </tr>
 
 
 
 
 <tr>
 <td>978</td>
 
 <td>        acpi_cpu_c1();</td>
 </tr>
 
 
 
 
 <tr>
 <td>979</td>
 
 <td>        return;</td>
 </tr>
 
 
 
 
 <tr>
 <td>980</td>
 
 <td>    }</td></tr></tbody></table><br></div>and then does a &#39;hlt&#39; =
 or idle if idle is disabled, this might be a problem.</div>
 
 --001a11c388b2efa98404e5416fa7--