diff options
author | 2007-10-05 15:55:02 -0400 | |
---|---|---|
committer | 2007-10-23 15:09:27 -0400 | |
commit | 6c9746b363b8191587191518a65d5de93df80a92 (patch) | |
tree | f500193c07ad69d8f733586d694b160e3dff61f8 /drivers | |
parent | [SCSI] sym53c8xx: Remove unnecessary check in queuecommand (diff) | |
download | linux-6c9746b363b8191587191518a65d5de93df80a92.tar.gz linux-6c9746b363b8191587191518a65d5de93df80a92.tar.bz2 linux-6c9746b363b8191587191518a65d5de93df80a92.zip |
[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>
Diffstat (limited to 'drivers')
-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 9fc2ae836257..b5e7c6478653 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"); |