diff options
author | Ralph Campbell <ralphc@pathscale.com> | 2007-05-03 15:40:51 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-05-07 00:18:11 -0400 |
commit | 6ed89b9574776d4178f1ad754d20e4f1e5a4b6c8 (patch) | |
tree | 6aafb2518c2cf83566ef7276b9d4424b8dd31207 /drivers | |
parent | 1a70a05d9d2b30db3e56f8cfbebb175663b41bad (diff) |
IB/ipath: Fix two more spin lock problems
Fix a missing unlock in ipath_rc_rcv_resp() and remove an extra unlock
from ipath_rc_rcv_error().
Signed-off-by: Ralph Campbell <ralph.campbell@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/hw/ipath/ipath_rc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_rc.c b/drivers/infiniband/hw/ipath/ipath_rc.c index 9e68c91130dd..f4d729d5dd1a 100644 --- a/drivers/infiniband/hw/ipath/ipath_rc.c +++ b/drivers/infiniband/hw/ipath/ipath_rc.c | |||
@@ -1257,6 +1257,7 @@ ack_err: | |||
1257 | wc.dlid_path_bits = 0; | 1257 | wc.dlid_path_bits = 0; |
1258 | wc.port_num = 0; | 1258 | wc.port_num = 0; |
1259 | ipath_sqerror_qp(qp, &wc); | 1259 | ipath_sqerror_qp(qp, &wc); |
1260 | spin_unlock_irqrestore(&qp->s_lock, flags); | ||
1260 | bail: | 1261 | bail: |
1261 | return; | 1262 | return; |
1262 | } | 1263 | } |
@@ -1436,7 +1437,6 @@ static inline int ipath_rc_rcv_error(struct ipath_ibdev *dev, | |||
1436 | break; | 1437 | break; |
1437 | } | 1438 | } |
1438 | qp->r_nak_state = 0; | 1439 | qp->r_nak_state = 0; |
1439 | spin_unlock_irq(&qp->s_lock); | ||
1440 | tasklet_hi_schedule(&qp->s_task); | 1440 | tasklet_hi_schedule(&qp->s_task); |
1441 | 1441 | ||
1442 | unlock_done: | 1442 | unlock_done: |