| Message ID | 20555.29361.138287.766030@pilspetsen.it.uu.se |
|---|---|
| State | Not Applicable |
| Delegated to: | David Miller |
| Headers | show |
>Please try the patch below, which implements the fix I described a >week ago. It's for 3.6-rc4 but should work in any recent kernel. >Without this patch one of my test machines always throws a lockdep >warning involving pdc_sata_hardreset and pdc_interrupt during bootup, >but with the patch the warning is gone, as expected. >If it works for you I'll add your Tested-by: and submit it properly. >/Mikael I will be able to try it no sooner than next week. Thanks ! Regards Adko. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Mikael writes: >Please try the patch below, which implements the fix I described a >week ago. It's for 3.6-rc4 but should work in any recent kernel. >Without this patch one of my test machines always throws a lockdep >warning involving pdc_sata_hardreset and pdc_interrupt during bootup, >but with the patch the warning is gone, as expected. >If it works for you I'll add your Tested-by: and submit it properly. Your patch seems to work, as the warning message is gone after applying it, i tried it with linux-3.5.2, and post here links to dmesg before and after applying the patch: before: http://pastebin.com/Y80AYYVK after: http://pastebin.com/Am0Zuabz Borislav writes: >Hmm, Cool'n'Quiet enabled actually means that you have _PSS objects and >powernow-k8 should be detecting the P-states properly. >Can you catch dmesg from that machine twice? >* one with Cool'n'Quiet disabled >* one with the kernel panic I hope i will be able to do it some time in the future, because i have no any physical contact with this machine, moreover it will require photo of the monitor after panic message, as it happens at boot process - fs not mounted yet, network not set, so i can not catch a message with netconsole. Borislav writes: >Right, so AFAICU, you can run a CONFIG_PREEMPT kernel perfectly fine now >after the BIOS update, right? Right. Borislav writes: >If so and when you say that the oopses started happening after an >electrical instability, I'd venture a guess that something got damaged >in the BIOS EPROM and since you've flashed it anew, the corrupted data >got overwritten and so no more crashes. Correct? The thought crossed my mind, but i still can not imagine how could it happens. Even if bios keeps some of the settings into EPROM, instead of CMOS, it should flash itself just after changing the settings, which is not the case, if i was right informed. There may be some other way that electricity break could damage EPROM content, somehow more directly, beyond the mechanisms by design, i don't know. Anyway, for the known facts we have, this explanation sounds pretty satisfactory, as long as no more crashes appears :). Thanks ! Regards Adko. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- linux-3.6-rc4/drivers/ata/sata_promise.c.~1~ 2012-09-08 12:18:24.000000000 +0200 +++ linux-3.6-rc4/drivers/ata/sata_promise.c 2012-09-08 17:55:49.000000000 +0200 @@ -147,6 +147,10 @@ struct pdc_port_priv { dma_addr_t pkt_dma; }; +struct pdc_host_priv { + spinlock_t hard_reset_lock; +}; + static int pdc_sata_scr_read(struct ata_link *link, unsigned int sc_reg, u32 *val); static int pdc_sata_scr_write(struct ata_link *link, unsigned int sc_reg, u32 val); static int pdc_ata_init_one(struct pci_dev *pdev, const struct pci_device_id *ent); @@ -801,9 +805,10 @@ static void pdc_hard_reset_port(struct a void __iomem *host_mmio = ap->host->iomap[PDC_MMIO_BAR]; void __iomem *pcictl_b1_mmio = host_mmio + PDC_PCI_CTL + 1; unsigned int ata_no = pdc_ata_port_to_ata_no(ap); + struct pdc_host_priv *hpriv = ap->host->private_data; u8 tmp; - spin_lock(&ap->host->lock); + spin_lock(&hpriv->hard_reset_lock); tmp = readb(pcictl_b1_mmio); tmp &= ~(0x10 << ata_no); @@ -814,7 +819,7 @@ static void pdc_hard_reset_port(struct a writeb(tmp, pcictl_b1_mmio); readb(pcictl_b1_mmio); /* flush */ - spin_unlock(&ap->host->lock); + spin_unlock(&hpriv->hard_reset_lock); } static int pdc_sata_hardreset(struct ata_link *link, unsigned int *class, @@ -1182,6 +1187,7 @@ static int pdc_ata_init_one(struct pci_d const struct ata_port_info *pi = &pdc_port_info[ent->driver_data]; const struct ata_port_info *ppi[PDC_MAX_PORTS]; struct ata_host *host; + struct pdc_host_priv *hpriv; void __iomem *host_mmio; int n_ports, i, rc; int is_sataii_tx4; @@ -1218,6 +1224,11 @@ static int pdc_ata_init_one(struct pci_d dev_err(&pdev->dev, "failed to allocate host\n"); return -ENOMEM; } + hpriv = devm_kzalloc(&pdev->dev, sizeof *hpriv, GFP_KERNEL); + if (!hpriv) + return -ENOMEM; + spin_lock_init(&hpriv->hard_reset_lock); + host->private_data = hpriv; host->iomap = pcim_iomap_table(pdev); is_sataii_tx4 = pdc_is_sataii_tx4(pi->flags);