diff options
author | Ralph Campbell <ralph.campbell@qlogic.com> | 2007-03-15 17:44:50 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-04-18 23:20:54 -0400 |
commit | 7b21d26ddad6912bf345e8e88a51a5ce98a036ad (patch) | |
tree | 1f560dcf8c088416bc4838edbeb15364cafd59ff /drivers/infiniband | |
parent | 9f9630d5e12a51f38513de0d64320a55ab6f02d5 (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.c | 3 |
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) | |||
265 | again: | 265 | again: |
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 | } |