From d21cb85060bc0b8235ff393e9a28c0e627d18cc2 Mon Sep 17 00:00:00 2001 From: Jens Taprogge Date: Wed, 12 Sep 2012 14:55:32 +0200 Subject: [PATCH] --- yaml --- r: 325237 b: refs/heads/master c: b8d61d49b2437df3a0aef17d321948783d275bb3 h: refs/heads/master i: 325235: fbf985a8047940e616faa59fd273e777870ae2e9 v: v3 --- [refs] | 2 +- trunk/drivers/staging/ipack/bridges/tpci200.c | 10 ++++++---- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/[refs] b/[refs] index 6c91c0ad0b41..a9c4923a3850 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 699a89f1e291af0d47744200011d5c24d9e462a6 +refs/heads/master: b8d61d49b2437df3a0aef17d321948783d275bb3 diff --git a/trunk/drivers/staging/ipack/bridges/tpci200.c b/trunk/drivers/staging/ipack/bridges/tpci200.c index 24e2a119dd16..9ce577ab9437 100644 --- a/trunk/drivers/staging/ipack/bridges/tpci200.c +++ b/trunk/drivers/staging/ipack/bridges/tpci200.c @@ -132,10 +132,11 @@ static irqreturn_t tpci200_interrupt(int irq, void *dev_id) if (status_reg & TPCI200_SLOT_INT_MASK) { /* callback to the IRQ handler for the corresponding slot */ + rcu_read_lock(); for (i = 0; i < TPCI200_NB_SLOT; i++) { if (!(status_reg & ((TPCI200_A_INT0 | TPCI200_A_INT1) << (2*i)))) continue; - slot_irq = tpci200->slots[i].irq; + slot_irq = rcu_dereference(tpci200->slots[i].irq); if (slot_irq) { ret = tpci200_slot_irq(slot_irq); } else { @@ -147,6 +148,7 @@ static irqreturn_t tpci200_interrupt(int irq, void *dev_id) TPCI200_INT0_EN | TPCI200_INT1_EN); } } + rcu_read_unlock(); } return ret; @@ -303,9 +305,9 @@ static int tpci200_free_irq(struct ipack_device *dev) __tpci200_free_irq(tpci200, dev); slot_irq = tpci200->slots[dev->slot].irq; - tpci200->slots[dev->slot].irq = NULL; + RCU_INIT_POINTER(tpci200->slots[dev->slot].irq, NULL); + synchronize_rcu(); kfree(slot_irq); - mutex_unlock(&tpci200->mutex); return 0; } @@ -490,7 +492,7 @@ static int tpci200_request_irq(struct ipack_device *dev, int vector, slot_irq->arg = arg; slot_irq->holder = dev; - tpci200->slots[dev->slot].irq = slot_irq; + rcu_assign_pointer(tpci200->slots[dev->slot].irq, slot_irq); res = __tpci200_request_irq(tpci200, dev); out_unlock: