aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/block/rsxx
diff options
context:
space:
mode:
authorPhilip J Kelleher <pjk1939@linux.vnet.ibm.com>2013-02-25 14:09:40 -0500
committerJens Axboe <axboe@kernel.dk>2013-03-11 14:53:55 -0400
commit1ebfd109822ea35b71aee4efe9ddc2e1b9ac0ed7 (patch)
treef2dd0444d11132e7ebe99a1be5314d34bdc0aa12 /drivers/block/rsxx
parent9bb3c4469e317919b0fde8c0e0a3ebe7bd2cf167 (diff)
block: IBM RamSan 70/80 error message bug fix.
This patch includes a simple change to the rsxx_pci_remove function that caused error messages because traffic was halted too early. Signed-off-by: Philip J Kelleher <pjk1939@linux.vnet.ibm.com> Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block/rsxx')
-rw-r--r--drivers/block/rsxx/core.c7
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/block/rsxx/core.c b/drivers/block/rsxx/core.c
index b82ee7baf0e8..cbbdff113f46 100644
--- a/drivers/block/rsxx/core.c
+++ b/drivers/block/rsxx/core.c
@@ -538,9 +538,6 @@ static void rsxx_pci_remove(struct pci_dev *dev)
538 rsxx_disable_ier_and_isr(card, CR_INTR_EVENT); 538 rsxx_disable_ier_and_isr(card, CR_INTR_EVENT);
539 spin_unlock_irqrestore(&card->irq_lock, flags); 539 spin_unlock_irqrestore(&card->irq_lock, flags);
540 540
541 /* Prevent work_structs from re-queuing themselves. */
542 card->halt = 1;
543
544 cancel_work_sync(&card->event_work); 541 cancel_work_sync(&card->event_work);
545 542
546 rsxx_destroy_dev(card); 543 rsxx_destroy_dev(card);
@@ -549,6 +546,10 @@ static void rsxx_pci_remove(struct pci_dev *dev)
549 spin_lock_irqsave(&card->irq_lock, flags); 546 spin_lock_irqsave(&card->irq_lock, flags);
550 rsxx_disable_ier_and_isr(card, CR_INTR_ALL); 547 rsxx_disable_ier_and_isr(card, CR_INTR_ALL);
551 spin_unlock_irqrestore(&card->irq_lock, flags); 548 spin_unlock_irqrestore(&card->irq_lock, flags);
549
550 /* Prevent work_structs from re-queuing themselves. */
551 card->halt = 1;
552
552 free_irq(dev->irq, card); 553 free_irq(dev->irq, card);
553 554
554 if (!force_legacy) 555 if (!force_legacy)