[PATCH] PM-Timer: don't use workaround if chipset is not buggy
authorOGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Mon, 27 Mar 2006 09:14:26 +0000 (01:14 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Mon, 27 Mar 2006 16:44:37 +0000 (08:44 -0800)
Current timer_pm.c reads I/O port triple times, in order to avoid the bug
of chipset.  But I/O port is slow.

2.6.16 (pmtmr)
Simple gettimeofday: 3.6532 microseconds

2.6.16+patch (pmtmr)
Simple gettimeofday: 1.4582 microseconds

[if chip is buggy, probably it will be 7us or more in 4.2% of probability.]

This patch adds blacklist of buggy chip, and if chip is not buggy, this
uses fast normal version instead of slow workaround version.

If chip is buggy, warnings "pmtmr is slow".  But sounds like there is gray
zone.  I found the PIIX4 errata, but I couldn't find the ICH4 errata.  But
some motherboard seems to have problem.

So, if we found a ICH4, generate warnings, and use a workaround version.
If user's ICH4 is good, the user can specify the "pmtmr_good" boot
parameter to use fast version.

Acked-by: John Stultz <johnstul@us.ibm.com>
Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
arch/i386/kernel/timers/timer_pm.c

index 264edaaac315b3e75c0781cb0a016e275151dc67..144e94a049330a7941d9d90767077ae93b666841 100644 (file)
@@ -15,6 +15,7 @@
 #include <linux/module.h>
 #include <linux/device.h>
 #include <linux/init.h>
+#include <linux/pci.h>
 #include <asm/types.h>
 #include <asm/timer.h>
 #include <asm/smp.h>
@@ -45,24 +46,31 @@ static seqlock_t monotonic_lock = SEQLOCK_UNLOCKED;
 
 #define ACPI_PM_MASK 0xFFFFFF /* limit it to 24 bits */
 
+static int pmtmr_need_workaround __read_mostly = 1;
+
 /*helper function to safely read acpi pm timesource*/
 static inline u32 read_pmtmr(void)
 {
-       u32 v1=0,v2=0,v3=0;
-       /* It has been reported that because of various broken
-        * chipsets (ICH4, PIIX4 and PIIX4E) where the ACPI PM time
-        * source is not latched, so you must read it multiple
-        * times to insure a safe value is read.
-        */
-       do {
-               v1 = inl(pmtmr_ioport);
-               v2 = inl(pmtmr_ioport);
-               v3 = inl(pmtmr_ioport);
-       } while ((v1 > v2 && v1 < v3) || (v2 > v3 && v2 < v1)
-                       || (v3 > v1 && v3 < v2));
-
-       /* mask the output to 24 bits */
-       return v2 & ACPI_PM_MASK;
+       if (pmtmr_need_workaround) {
+               u32 v1, v2, v3;
+
+               /* It has been reported that because of various broken
+                * chipsets (ICH4, PIIX4 and PIIX4E) where the ACPI PM time
+                * source is not latched, so you must read it multiple
+                * times to insure a safe value is read.
+                */
+               do {
+                       v1 = inl(pmtmr_ioport);
+                       v2 = inl(pmtmr_ioport);
+                       v3 = inl(pmtmr_ioport);
+               } while ((v1 > v2 && v1 < v3) || (v2 > v3 && v2 < v1)
+                        || (v3 > v1 && v3 < v2));
+
+               /* mask the output to 24 bits */
+               return v2 & ACPI_PM_MASK;
+       }
+
+       return inl(pmtmr_ioport) & ACPI_PM_MASK;
 }
 
 
@@ -263,6 +271,72 @@ struct init_timer_opts __initdata timer_pmtmr_init = {
        .opts = &timer_pmtmr,
 };
 
+#ifdef CONFIG_PCI
+/*
+ * PIIX4 Errata:
+ *
+ * The power management timer may return improper results when read.
+ * Although the timer value settles properly after incrementing,
+ * while incrementing there is a 3 ns window every 69.8 ns where the
+ * timer value is indeterminate (a 4.2% chance that the data will be
+ * incorrect when read). As a result, the ACPI free running count up
+ * timer specification is violated due to erroneous reads.
+ */
+static int __init pmtmr_bug_check(void)
+{
+       static struct pci_device_id gray_list[] __initdata = {
+               /* these chipsets may have bug. */
+               { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
+                               PCI_DEVICE_ID_INTEL_82801DB_0) },
+               { },
+       };
+       struct pci_dev *dev;
+       int pmtmr_has_bug = 0;
+       u8 rev;
+
+       if (cur_timer != &timer_pmtmr || !pmtmr_need_workaround)
+               return 0;
+
+       dev = pci_get_device(PCI_VENDOR_ID_INTEL,
+                            PCI_DEVICE_ID_INTEL_82371AB_3, NULL);
+       if (dev) {
+               pci_read_config_byte(dev, PCI_REVISION_ID, &rev);
+               /* the bug has been fixed in PIIX4M */
+               if (rev < 3) {
+                       printk(KERN_WARNING "* Found PM-Timer Bug on this "
+                               "chipset. Due to workarounds for a bug,\n"
+                               "* this time source is slow.  Consider trying "
+                               "other time sources (clock=)\n");
+                       pmtmr_has_bug = 1;
+               }
+               pci_dev_put(dev);
+       }
+
+       if (pci_dev_present(gray_list)) {
+               printk(KERN_WARNING "* This chipset may have PM-Timer Bug.  Due"
+                       " to workarounds for a bug,\n"
+                       "* this time source is slow. If you are sure your timer"
+                       " does not have\n"
+                       "* this bug, please use \"pmtmr_good\" to disable the "
+                       "workaround\n");
+               pmtmr_has_bug = 1;
+       }
+
+       if (!pmtmr_has_bug)
+               pmtmr_need_workaround = 0;
+
+       return 0;
+}
+device_initcall(pmtmr_bug_check);
+#endif
+
+static int __init pmtr_good_setup(char *__str)
+{
+       pmtmr_need_workaround = 0;
+       return 1;
+}
+__setup("pmtmr_good", pmtr_good_setup);
+
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Dominik Brodowski <linux@brodo.de>");
 MODULE_DESCRIPTION("Power Management Timer (PMTMR) as primary timing source for x86");