[PATCH] [DLM] Clear othercon pointers when a connection is closed

From: Patrick Caulfield <[email protected]>

This patch clears the othercon pointer and frees the memory when a connnection
is closed. This could cause a small memory leak when nodes leave the cluster.

Signed-Off-By: Patrick Caulfield <[email protected]>
Signed-off-by: Steven Whitehouse <[email protected]>

diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c
index dd36273..d15fd5f 100644
--- a/fs/dlm/lowcomms.c
+++ b/fs/dlm/lowcomms.c
@@ -327,6 +327,8 @@ static void close_connection(struct connection *con, bool and_other)
 	if (con->othercon && and_other) {
 		/* Will only re-enter once. */
 		close_connection(con->othercon, false);
+		kmem_cache_free(con_cache, con->othercon);
+		con->othercon = NULL;
 	}
 	if (con->rx_page) {
 		__free_page(con->rx_page);
-- 
1.5.1.2

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • [GFS2/DLM] [0/12] Pre pull request patch posting
    • From: Steven Whitehouse <[email protected]>
  • [PATCH] [GFS2] Revert remounting w/o acl option leaves acls enabled
    • From: [email protected]
  • [PATCH] [GFS2] Fix setting of inherit jdata attr
    • From: [email protected]
  • [PATCH] [GFS2] Fix incorrect error path in prepare_write()
    • From: [email protected]
  • [PATCH] [GFS2] Fix incorrect return code in rgrp.c
    • From: [email protected]
  • [PATCH] [GFS2] soft lockup in rgblk_search
    • From: [email protected]
  • [PATCH] [GFS2] soft lockup detected in databuf_lo_before_commit
    • From: [email protected]
  • [PATCH] [DLM] fix basts for granted PR waiting CW
    • From: [email protected]
  • [PATCH] [DLM] More othercon fixes
    • From: [email protected]
  • [PATCH] [DLM] Fix memory leak in dlm_add_member() when dlm_node_weight() returns less than zero
    • From: [email protected]
  • [PATCH] [DLM] zero unused parts of sockaddr_storage
    • From: [email protected]
  • [PATCH] [DLM] fix NULL ls usage
    • From: [email protected]

• Prev by Date: Re: [PATCH 00/23] per device dirty throttling -v8
• Next by Date: [PATCH] [DLM] fix NULL ls usage
• Previous by thread: HTC as USB gprs/edge modem on linux
• Next by thread: [PATCH] [DLM] fix NULL ls usage
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]