aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/infiniband
diff options
context:
space:
mode:
authorRalph Campbell <ralph.campbell@qlogic.com>2007-03-15 17:44:50 -0400
committerRoland Dreier <rolandd@cisco.com>2007-04-18 23:20:54 -0400
commit7b21d26ddad6912bf345e8e88a51a5ce98a036ad (patch)
tree1f560dcf8c088416bc4838edbeb15364cafd59ff /drivers/infiniband
parent9f9630d5e12a51f38513de0d64320a55ab6f02d5 (diff)
IB/ipath: NMI cpu lockup if local loopback used
If a post send is done in loopback and there is no receive queue entry, the sending QP is put on a timeout list for a while so the receiver has a chance to post a receive buffer. If the another post send is done, the code incorrectly tried to put the QP on the timeout list again an corrupted the timeout list. This eventually leads to a spin lock deadlock NMI due to the timer function looping forever with the lock held. Signed-off-by: Bryan O'Sullivan <bryan.osullivan@qlogic.com> Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r--drivers/infiniband/hw/ipath/ipath_ruc.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_ruc.c b/drivers/infiniband/hw/ipath/ipath_ruc.c
index 146db8516997..cda84933bb43 100644
--- a/drivers/infiniband/hw/ipath/ipath_ruc.c
+++ b/drivers/infiniband/hw/ipath/ipath_ruc.c
@@ -265,7 +265,8 @@ static void ipath_ruc_loopback(struct ipath_qp *sqp)
265again: 265again:
266 spin_lock_irqsave(&sqp->s_lock, flags); 266 spin_lock_irqsave(&sqp->s_lock, flags);
267 267
268 if (!(ib_ipath_state_ops[sqp->state] & IPATH_PROCESS_SEND_OK)) { 268 if (!(ib_ipath_state_ops[sqp->state] & IPATH_PROCESS_SEND_OK) ||
269 qp->s_rnr_timeout) {
269 spin_unlock_irqrestore(&sqp->s_lock, flags); 270 spin_unlock_irqrestore(&sqp->s_lock, flags);
270 goto done; 271 goto done;
271 } 272 }