From: john stultz <johnstul@us.ibm.com>
To: Bob Tracy <rct@gherkin.frus.com>
Cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>,
linux-kernel@vger.kernel.org, bunk@stusta.de, jikos@jikos.cz
Subject: Re: [BUG] no boot with 2.6.21-rc3 and later
Date: Thu, 22 Mar 2007 11:35:12 -0700 [thread overview]
Message-ID: <1174588512.5662.3.camel@localhost.localdomain> (raw)
In-Reply-To: <20070322034523.90050DBA1@gherkin.frus.com>
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.
-john
Don't drop ACPI PM quite so low if we see the PIIX4 bug.
Signed-off-by: John Stultz <johnstul@us.ibm.com>
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)
{
clocksource_acpi_pm.read = acpi_pm_read_slow;
- clocksource_acpi_pm.rating = 110;
+ clocksource_acpi_pm.rating = 120;
}
/*
next prev parent reply other threads:[~2007-03-22 18:35 UTC|newest]
Thread overview: 18+ 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 [this message]
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 ` [PATCH] correct slow acpi_pm rating (fixes no boot with 2.6.21-rc3 and later) john stultz
-- strict thread matches above, loose matches on Subject: below --
2007-03-20 1:43 [BUG] no boot with 2.6.21-rc3 and later Bob Tracy
2007-03-20 7:14 ` Jiri Kosina
2007-03-18 1:09 Bob Tracy
2007-03-18 18:59 ` Jiri Kosina
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:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1174588512.5662.3.camel@localhost.localdomain \
--to=johnstul@us.ibm.com \
--cc=bunk@stusta.de \
--cc=bzolnier@gmail.com \
--cc=jikos@jikos.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=rct@gherkin.frus.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox