IB/ipoib: Fix transmit queue stalling forever
authorEli Cohen <eli@dev.mellanox.co.il>
Thu, 1 May 2008 03:02:45 +0000 (20:02 -0700)
committerRoland Dreier <rolandd@cisco.com>
Thu, 1 May 2008 03:02:45 +0000 (20:02 -0700)
commit57ce41d1d18279cc90223f3deadca70c7de1cfca
tree39111403527e4744b4a6a15d5ffe3fd038751489
parent3ae15e1623b9d32eb410c2a23d90e47b16e6acd0
IB/ipoib: Fix transmit queue stalling forever

Commit f56bcd80 ("IPoIB: Use separate CQ for UD send completions")
introduced a bug where the transmit queue could get stopped and never
woken up.  The problem is that send completions are only polled at the
end of the xmit function, so if the send queue fills up and the xmit
path stops the queue, then there is no way for send completions to
ever get polled, and so the transmit queue stays stopped forever.

Fix this by arming the send CQ just before posting the last send
request that fills the send queue.  Then, when the completion event
handler is called, drain the send CQ.  Since it is possible that not
enough send completions are in the CQ, verify that the the net queue
has been woken up after draining the send CQ, and if not arm a timer
and drain again at the timer function.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
drivers/infiniband/ulp/ipoib/ipoib.h
drivers/infiniband/ulp/ipoib/ipoib_ib.c
drivers/infiniband/ulp/ipoib/ipoib_verbs.c