LKML Archive on
help / color / mirror / Atom feed
From: john stultz <>
To: Andrew Morton <>
Cc: Bartlomiej Zolnierkiewicz <>,,,,
	Bob Tracy <>
Subject: [PATCH] correct slow acpi_pm rating (fixes no boot with 2.6.21-rc3 and later)
Date: Thu, 22 Mar 2007 13:25:18 -0700	[thread overview]
Message-ID: <1174595118.5662.16.camel@localhost.localdomain> (raw)
In-Reply-To: <>

Tested by Bob Tracy.

On Wed, 2007-03-21 at 21:45 -0600, Bob Tracy wrote:
> john stultz wrote:
> > > > Also, does booting w/ "clocksource=jiffies" change the behavior?
> Works fine with 2.6.21-rc4.  I'm running on that kernel as I type this.
> > Also trying booting w/ "notsc" would be a useful data point.
> Boot hangs at the point indicated in my original message.  I *did*
> notice the blurb in the console messages about the pit clocksource
> being selected/used.  There was also a complaint about it being
> unstable, with a negative delta.  I think these messages are consistent
> with the pre-bad-commit case other than where they appear in the boot
> messages.
> > (...) a pre-bad-commit dmesg would help.
> Sent under separate cover to John.

Yea. From looking at your boot log, it does look like the PIT
clocksource is being used. 

This does point to an issue w/ that clocksource, however I noticed its
getting selected over the ACPI PM timer, because you have the PIIX4 bug.
That bug drops the ACPI PM timers rating to the same as the PIT, so
that's why you're getting the PIT.

Realistically, the PIT is much slower then even the triple read ACPI PM,
so the de-ranking code is probably dropping it too far.

Try this patch and let me know if it does the right thing. Although I do
still need to dig a bit on the PIT hang issue.


Don't drop ACPI PM quite so low if we see the PIIX4 bug.

Signed-off-by: John Stultz <>

diff --git a/drivers/clocksource/acpi_pm.c b/drivers/clocksource/acpi_pm.c
index d42060e..5ac309e 100644
--- a/drivers/clocksource/acpi_pm.c
+++ b/drivers/clocksource/acpi_pm.c
@@ -90,7 +90,7 @@ __setup("acpi_pm_good", acpi_pm_good_set
 static inline void acpi_pm_need_workaround(void)
 { = acpi_pm_read_slow;
-	clocksource_acpi_pm.rating = 110;
+	clocksource_acpi_pm.rating = 120;

      parent reply	other threads:[~2007-03-22 20:25 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-21 19:54 [BUG] no boot with 2.6.21-rc3 and later Bob Tracy
2007-03-21 20:18 ` Bartlomiej Zolnierkiewicz
2007-03-21 21:34   ` john stultz
2007-03-21 22:00     ` Bob Tracy
2007-03-21 22:24       ` john stultz
2007-03-22  3:45         ` Bob Tracy
2007-03-22 18:35           ` john stultz
2007-03-22 19:14             ` Bob Tracy
2007-03-22 19:25               ` john stultz
2007-03-22 19:39                 ` Bob Tracy
2007-03-22 19:56                   ` john stultz
2007-03-22 20:13                     ` Bob Tracy
2007-03-22 19:43                 ` Thomas Gleixner
2007-03-22 20:25           ` john stultz [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1174595118.5662.16.camel@localhost.localdomain \ \ \ \ \ \ \ \
    --subject='Re: [PATCH] correct slow acpi_pm rating (fixes no boot with 2.6.21-rc3 and later)' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).