Skip to content

Commit

Permalink
[SCSI] arcmsr: irq handler fixes, cleanups, micro-opts
Browse files Browse the repository at this point in the history
* Remove IRQF_DISABLED, it is clearly wrong for this driver.

* Remove wasteful spin_lock_irqsave() in interrupt handler.
  The lighter-weight spin_lock() is all that's needed.

* Annotate with FIXME where arcmsr_interrupt() is called
  without any spinlock being acquired.

* Eliminate pointless cast from void pointer in arcmsr_do_interrupt()

[jejb: conflict resolution]

Signed-off-by: Jeff Garzik <[email protected]>
Acked-by: Nick Cheng <[email protected]>
Signed-off-by: James Bottomley <[email protected]>
  • Loading branch information
Jeff Garzik authored and James Bottomley committed Oct 12, 2007
1 parent 2443045 commit 488a5c8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/scsi/arcmsr/arcmsr_hba.c
Original file line number Diff line number Diff line change
Expand Up @@ -422,7 +422,7 @@ static int arcmsr_probe(struct pci_dev *pdev,
goto out_release_regions;

error = request_irq(pdev->irq, arcmsr_do_interrupt,
IRQF_SHARED, "arcmsr", acb);
IRQF_SHARED, "arcmsr", acb);
if (error)
goto out_free_ccb_pool;

Expand Down

0 comments on commit 488a5c8

Please sign in to comment.