diff options
author | Matthew Wilcox <matthew@wil.cx> | 2007-10-05 15:55:02 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-23 15:09:27 -0400 |
commit | 6c9746b363b8191587191518a65d5de93df80a92 (patch) | |
tree | f500193c07ad69d8f733586d694b160e3dff61f8 | |
parent | 71c222dc45ddf01b0dabb8a4d3e232e13e1452b5 (diff) | |
download | op-kernel-dev-6c9746b363b8191587191518a65d5de93df80a92.zip op-kernel-dev-6c9746b363b8191587191518a65d5de93df80a92.tar.gz |
[SCSI] sym53c8xx: Don't disable interrupts in the interrupt handler
Interrupts can't be re-entered, so it's sufficient to call spin_lock, not
spin_lock_irqsave().
Signed-off-by: Matthew Wilcox <willy@linux.intel.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r-- | drivers/scsi/sym53c8xx_2/sym_glue.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/scsi/sym53c8xx_2/sym_glue.c b/drivers/scsi/sym53c8xx_2/sym_glue.c index 9fc2ae8..b5e7c64 100644 --- a/drivers/scsi/sym53c8xx_2/sym_glue.c +++ b/drivers/scsi/sym53c8xx_2/sym_glue.c @@ -550,14 +550,13 @@ static int sym53c8xx_queue_command(struct scsi_cmnd *cmd, */ static irqreturn_t sym53c8xx_intr(int irq, void *dev_id) { - unsigned long flags; - struct sym_hcb *np = (struct sym_hcb *)dev_id; + struct sym_hcb *np = dev_id; if (DEBUG_FLAGS & DEBUG_TINY) printf_debug ("["); - spin_lock_irqsave(np->s.host->host_lock, flags); + spin_lock(np->s.host->host_lock); sym_interrupt(np); - spin_unlock_irqrestore(np->s.host->host_lock, flags); + spin_unlock(np->s.host->host_lock); if (DEBUG_FLAGS & DEBUG_TINY) printf_debug ("]\n"); |