RDS/IB: Do not wait for send ring to be empty on conn shutdown
authorAndy Grover <andy.grover@oracle.com>
Thu, 4 Mar 2010 03:25:21 +0000 (19:25 -0800)
committerAndy Grover <andy.grover@oracle.com>
Thu, 9 Sep 2010 01:12:09 +0000 (18:12 -0700)
Now that we are signaling send completions much less, we are likely
to have dirty entries in the send queue when the connection is
shut down (on rmmod, for example.) These are cleaned up a little
further down in conn_shutdown, but if we wait on the ring_empty_wait
for them, it'll never happen, and we hand on unload.

Signed-off-by: Andy Grover <andy.grover@oracle.com>
net/rds/ib_cm.c

index b5b5ebbc0bb6758e50e15739fe38cbbbd9a6ac0a..1a91af75f4c74b4bb15c6123e26302a1775ce706 100644 (file)
@@ -620,8 +620,11 @@ void rds_ib_conn_shutdown(struct rds_connection *conn)
                                ic->i_cm_id, err);
                }
 
+               /*
+                * Don't wait for the send ring to be empty -- there may be completed
+                * non-signaled entries sitting on there. We unmap these below.
+                */
                wait_event(rds_ib_ring_empty_wait,
-                       rds_ib_ring_empty(&ic->i_send_ring) &&
                        rds_ib_ring_empty(&ic->i_recv_ring));
 
                if (ic->i_send_hdrs)