Commit 9e5f2825 authored by Patrick Caulfield's avatar Patrick Caulfield Committed by Steven Whitehouse

[DLM] More othercon fixes

The last patch to clean out 'othercon' structures only fixed half the problem.
The attached addresses the other situations too, and fixes bz#238490
Signed-Off-By: default avatarPatrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: default avatarSteven Whitehouse <swhiteho@redhat.com>
parent 1a2bf2ee
...@@ -328,15 +328,24 @@ static void close_connection(struct connection *con, bool and_other) ...@@ -328,15 +328,24 @@ static void close_connection(struct connection *con, bool and_other)
if (con->othercon && and_other) { if (con->othercon && and_other) {
/* Will only re-enter once. */ /* Will only re-enter once. */
close_connection(con->othercon, false); close_connection(con->othercon, false);
kmem_cache_free(con_cache, con->othercon);
con->othercon = NULL;
} }
if (con->rx_page) { if (con->rx_page) {
__free_page(con->rx_page); __free_page(con->rx_page);
con->rx_page = NULL; con->rx_page = NULL;
} }
con->retries = 0;
mutex_unlock(&con->sock_mutex); /* If we are an 'othercon' then NULL the pointer to us
from the parent and tidy ourself up */
if (test_bit(CF_IS_OTHERCON, &con->flags)) {
struct connection *parent = __nodeid2con(con->nodeid, 0);
parent->othercon = NULL;
kmem_cache_free(con_cache, con);
}
else {
/* Parent connections get reused */
con->retries = 0;
mutex_unlock(&con->sock_mutex);
}
} }
/* We only send shutdown messages to nodes that are not part of the cluster */ /* We only send shutdown messages to nodes that are not part of the cluster */
...@@ -634,7 +643,7 @@ static int receive_from_sock(struct connection *con) ...@@ -634,7 +643,7 @@ static int receive_from_sock(struct connection *con)
out_close: out_close:
mutex_unlock(&con->sock_mutex); mutex_unlock(&con->sock_mutex);
if (ret != -EAGAIN && !test_bit(CF_IS_OTHERCON, &con->flags)) { if (ret != -EAGAIN) {
close_connection(con, false); close_connection(con, false);
/* Reconnect when there is something to send */ /* Reconnect when there is something to send */
} }
...@@ -1125,8 +1134,6 @@ static int tcp_listen_for_all(void) ...@@ -1125,8 +1134,6 @@ static int tcp_listen_for_all(void)
log_print("Using TCP for communications"); log_print("Using TCP for communications");
set_bit(CF_IS_OTHERCON, &con->flags);
sock = tcp_create_listen_sock(con, dlm_local_addr[0]); sock = tcp_create_listen_sock(con, dlm_local_addr[0]);
if (sock) { if (sock) {
add_sock(sock, con); add_sock(sock, con);
...@@ -1410,7 +1417,7 @@ void dlm_lowcomms_stop(void) ...@@ -1410,7 +1417,7 @@ void dlm_lowcomms_stop(void)
for (i = 0; i <= max_nodeid; i++) { for (i = 0; i <= max_nodeid; i++) {
con = __nodeid2con(i, 0); con = __nodeid2con(i, 0);
if (con) { if (con) {
con->flags |= 0xFF; con->flags |= 0x0F;
if (con->sock) if (con->sock)
con->sock->sk->sk_user_data = NULL; con->sock->sk->sk_user_data = NULL;
} }
...@@ -1426,8 +1433,6 @@ void dlm_lowcomms_stop(void) ...@@ -1426,8 +1433,6 @@ void dlm_lowcomms_stop(void)
con = __nodeid2con(i, 0); con = __nodeid2con(i, 0);
if (con) { if (con) {
close_connection(con, true); close_connection(con, true);
if (con->othercon)
kmem_cache_free(con_cache, con->othercon);
kmem_cache_free(con_cache, con); kmem_cache_free(con_cache, con);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment