]> git.ipfire.org Git - people/pmueller/ipfire-2.x.git/blobdiff - src/patches/suse-2.6.27.31/patches.drivers/libata-ata_piix-blacklist-double-spin-off
Reenabled linux-xen, added patches for Xen Kernel Version 2.6.27.31,
[people/pmueller/ipfire-2.x.git] / src / patches / suse-2.6.27.31 / patches.drivers / libata-ata_piix-blacklist-double-spin-off
diff --git a/src/patches/suse-2.6.27.31/patches.drivers/libata-ata_piix-blacklist-double-spin-off b/src/patches/suse-2.6.27.31/patches.drivers/libata-ata_piix-blacklist-double-spin-off
new file mode 100644 (file)
index 0000000..e5165a8
--- /dev/null
@@ -0,0 +1,76 @@
+From 6cb195762fb4acc79ab5153452be5d44a40419dc Mon Sep 17 00:00:00 2001
+From: Rafael J. Wysocki <rjw@sisk.pl>
+Date: Mon, 3 Nov 2008 19:01:07 +0900
+Subject: [PATCH] SATA PIIX: Blacklist system that spins off disks during ACPI power off
+References: bnc#441721
+
+Some notebooks from HP have the problem that their BIOSes attempt to
+spin down hard drives before entering ACPI system states S4 and S5.
+This leads to a yo-yo effect during system power-off shutdown and the
+last phase of hibernation when the disk is first spun down by the
+kernel and then almost immediately turned on and off by the BIOS.
+This, in turn, may result in shortening the disk's life times.
+
+To prevent this from happening we can blacklist the affected systems
+using DMI information.
+
+Blacklist HP 2510p that uses the ata_piix driver.
+
+Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
+Signed-off-by: Tejun Heo <tj@kernel.org>
+Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
+Signed-off-by: Tejun Heo <teheo@suse.de>
+---
+ drivers/ata/ata_piix.c |   34 ++++++++++++++++++++++++++++++++++
+ 1 file changed, 34 insertions(+)
+
+--- a/drivers/ata/ata_piix.c
++++ b/drivers/ata/ata_piix.c
+@@ -1409,6 +1409,32 @@ static void piix_iocfg_bit18_quirk(struc
+       }
+ }
++static bool piix_broken_system_poweroff(struct pci_dev *pdev)
++{
++      static const struct dmi_system_id broken_systems[] = {
++              {
++                      .ident = "HP Compaq 2510p",
++                      .matches = {
++                              DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
++                              DMI_MATCH(DMI_PRODUCT_NAME, "HP Compaq 2510p"),
++                      },
++                      /* PCI slot number of the controller */
++                      .driver_data = (void *)0x1FUL,
++              },
++
++              { }     /* terminate list */
++      };
++      const struct dmi_system_id *dmi = dmi_first_match(broken_systems);
++
++      if (dmi) {
++              unsigned long slot = (unsigned long)dmi->driver_data;
++              /* apply the quirk only to on-board controllers */
++              return slot == PCI_SLOT(pdev->devfn);
++      }
++
++      return false;
++}
++
+ /**
+  *    piix_init_one - Register PIIX ATA PCI device with kernel services
+  *    @pdev: PCI device to register
+@@ -1444,6 +1470,14 @@ static int __devinit piix_init_one(struc
+       if (!in_module_init)
+               return -ENODEV;
++      if (piix_broken_system_poweroff(pdev)) {
++              piix_port_info[ent->driver_data].flags |=
++                              ATA_FLAG_NO_POWEROFF_SPINDOWN |
++                                      ATA_FLAG_NO_HIBERNATE_SPINDOWN;
++              dev_info(&pdev->dev, "quirky BIOS, skipping spindown "
++                              "on poweroff and hibernation\n");
++      }
++
+       port_info[0] = piix_port_info[ent->driver_data];
+       port_info[1] = piix_port_info[ent->driver_data];