Randy Dunlap wrote: > On Mon, 2 Aug 2010 15:07:11 -0700 Mark Knecht wrote: > > > On Thu, Jul 22, 2010 at 5:39 AM, Tejun Heo <tj@xxxxxxxxxx> wrote: > > > > Hi Tejun, > > I'm finally home and trying to get back to this. I'm really a bad > > programmer so I don't know what I've done wrong but it seems patch > > isn't happy with me. > > > > c2stable linux # patch --dry-run -p1 <../ata_piix-sidpr-lock.patch > > patching file drivers/ata/ata_piix.c > > patch: **** malformed patch at line 13: > > Whenever the patch file was saved on this system, line 13 of it was > split (probably by an email client). Whenever I see this, I just > join (merge) that line and the next one and try again... sometimes > several lines are malformed and have to be fixed like this. Tejun asked Mark to add a printk, and Mark added it directly to the patch. Mark, just apply the original patch as-is, and then add the printk to the source code in ata_piix.c. You should add it somewhere like the piix_init_one function, e.g. right before the "Save IOCFG" comment around line 1575. -jim > > > > > c2stable linux # > > > > Here's the change I tried to make to a copy of the file: > > > > c2stable linux # cat ../ata_piix-sidpr-lock.patch > > diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c > > index 7409f98..3971bc0 100644 > > --- a/drivers/ata/ata_piix.c > > +++ b/drivers/ata/ata_piix.c > > @@ -158,6 +158,7 @@ struct piix_map_db { > > struct piix_host_priv { > > const int *map; > > u32 saved_iocfg; > > + spinlock_t sidpr_lock; /* FIXME: remove once locking in EH is fixed */ > > + printk("MWK - ata_sidpr patch applied!\n"); > > void __iomem *sidpr; > > }; > > > > @@ -951,12 +952,15 @@ static int piix_sidpr_scr_read(struct ata_link *link, > > unsigned int reg, u32 *val) > > { > > struct piix_host_priv *hpriv = link->ap->host->private_data; > > + unsigned long flags; > > > > if (reg >= ARRAY_SIZE(piix_sidx_map)) > > return -EINVAL; > > > > + spin_lock_irqsave(&hpriv->sidpr_lock, flags); > > piix_sidpr_sel(link, reg); > > *val = ioread32(hpriv->sidpr + PIIX_SIDPR_DATA); > > + spin_unlock_irqrestore(&hpriv->sidpr_lock, flags); > > return 0; > > } > > > > @@ -964,12 +968,15 @@ static int piix_sidpr_scr_write(struct ata_link *link, > > unsigned int reg, u32 val) > > { > > struct piix_host_priv *hpriv = link->ap->host->private_data; > > + unsigned long flags; > > > > if (reg >= ARRAY_SIZE(piix_sidx_map)) > > return -EINVAL; > > > > + spin_lock_irqsave(&hpriv->sidpr_lock, flags); > > piix_sidpr_sel(link, reg); > > iowrite32(val, hpriv->sidpr + PIIX_SIDPR_DATA); > > + spin_unlock_irqrestore(&hpriv->sidpr_lock, flags); > > return 0; > > } > > > > @@ -1566,6 +1573,7 @@ static int __devinit piix_init_one(struct pci_dev *pdev, > > hpriv = devm_kzalloc(dev, sizeof(*hpriv), GFP_KERNEL); > > if (!hpriv) > > return -ENOMEM; > > + spin_lock_init(&hpriv->sidpr_lock); > > > > /* Save IOCFG, this will be used for cable detection, quirk > > * detection and restoration on detach. This is necessary > > c2stable linux # > > > > Maybe you can shoot back something that's done correctly and I'll > > start testing. > > > > I've tried booting a few times. I've had 3 cold boot failures so > > far. No warm boot failures. Each time it failed on cold boot a warm > > boot fixed it. > > > --- > ~Randy > *** Remember to use Documentation/SubmitChecklist when testing your code *** > -- > To unsubscribe from this list: send the line "unsubscribe linux-ide" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html