From: viro@parcelfarce.linux.theplanet.co.uk
* don't take the unnecessary ->host_lock around ->scsi_done() call.
---
25-akpm/drivers/scsi/ppa.c | 4 ----
1 files changed, 4 deletions(-)
diff -puN drivers/scsi/ppa.c~PPA8-ppa_lock_fix-RC1 drivers/scsi/ppa.c
--- 25/drivers/scsi/ppa.c~PPA8-ppa_lock_fix-RC1 Wed Jan 14 13:35:51 2004
+++ 25-akpm/drivers/scsi/ppa.c Wed Jan 14 13:35:51 2004
@@ -627,7 +627,6 @@ static void ppa_interrupt(void *data)
{
ppa_struct *dev = (ppa_struct *) data;
Scsi_Cmnd *cmd = dev->cur_cmd;
- unsigned long flags;
if (!cmd) {
printk("PPA: bug in ppa_interrupt\n");
@@ -680,10 +679,7 @@ static void ppa_interrupt(void *data)
dev->cur_cmd = 0;
- spin_lock_irqsave(cmd->device->host->host_lock, flags);
cmd->scsi_done(cmd);
- spin_unlock_irqrestore(cmd->device->host->host_lock, flags);
- return;
}
static int ppa_engine(ppa_struct *dev, Scsi_Cmnd *cmd)
_