Attachment 879812 Details for Bug 1080103 – 0001

[patch] 0001

0001.patch (text/plain), 2.95 KB, created by Jan Friesse on 2014-03-28 12:28:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jan Friesse

Created: 2014-03-28 12:28:46 UTC

Size: 2.95 KB

patch

obsolete

>commit e9d9c4be7293c04ca97bfd84f95b76a22690a92a
>Author: Jan Friesse <jfriesse@redhat.com>
>Date:   Wed Feb 19 12:23:16 2014 +0100
>
>    cpg: Refactor mh_req_exec_cpg_procleave
>    
>    Most of functionality is moved to do_proc_leave function to make it
>    reusable.
>    
>    Signed-off-by: Jan Friesse <jfriesse@redhat.com>
>    Reviewed-by: Christine Caulfield <ccaulfie@redhat.com>
>    (backport from needle)
>
>diff --git a/services/cpg.c b/services/cpg.c
>index 7ea06c1..c558c8b 100644
>--- a/services/cpg.c
>+++ b/services/cpg.c
>@@ -288,6 +288,12 @@ static void do_proc_join(
> 	unsigned int nodeid,
> 	int reason);
> 
>+static void do_proc_leave(
>+	const mar_cpg_name_t *name,
>+	uint32_t pid,
>+	unsigned int nodeid,
>+	int reason);
>+
> static int notify_lib_totem_membership (
> 	void *conn,
> 	int member_list_entries,
>@@ -1155,6 +1161,37 @@ static void do_proc_join(
> 			    MESSAGE_RES_CPG_CONFCHG_CALLBACK);
> }
> 
>+static void do_proc_leave(
>+	const mar_cpg_name_t *name,
>+	uint32_t pid,
>+	unsigned int nodeid,
>+	int reason)
>+{
>+	struct process_info *pi;
>+	struct list_head *iter;
>+	mar_cpg_address_t notify_info;
>+
>+	notify_info.pid = pid;
>+	notify_info.nodeid = nodeid;
>+	notify_info.reason = reason;
>+
>+	notify_lib_joinlist(name, NULL,
>+		0, NULL,
>+		1, &notify_info,
>+		MESSAGE_RES_CPG_CONFCHG_CALLBACK);
>+
>+	for (iter = process_info_list_head.next; iter != &process_info_list_head; ) {
>+		pi = list_entry(iter, struct process_info, list);
>+		iter = iter->next;
>+
>+		if (pi->pid == pid && pi->nodeid == nodeid &&
>+			mar_name_compare (&pi->group, name)==0) {
>+			list_del (&pi->list);
>+			free (pi);
>+		}
>+	}
>+}
>+
> static void message_handler_req_exec_cpg_downlist_old (
> 	const void *message,
> 	unsigned int nodeid)
>@@ -1229,31 +1266,15 @@ static void message_handler_req_exec_cpg_procleave (
> 	unsigned int nodeid)
> {
> 	const struct req_exec_cpg_procjoin *req_exec_cpg_procjoin = message;
>-	struct process_info *pi;
>-	struct list_head *iter;
>-	mar_cpg_address_t notify_info;
>-
>-	log_printf(LOGSYS_LEVEL_DEBUG, "got procleave message from cluster node %d\n", nodeid);
>-
>-	notify_info.pid = req_exec_cpg_procjoin->pid;
>-	notify_info.nodeid = nodeid;
>-	notify_info.reason = req_exec_cpg_procjoin->reason;
>-
>-	notify_lib_joinlist(&req_exec_cpg_procjoin->group_name, NULL,
>-		0, NULL,
>-		1, &notify_info,
>-		MESSAGE_RES_CPG_CONFCHG_CALLBACK);
> 
>-	for (iter = process_info_list_head.next; iter != &process_info_list_head; ) {
>-		pi = list_entry(iter, struct process_info, list);
>-		iter = iter->next;
>+	log_printf(LOGSYS_LEVEL_DEBUG, "got procleave message from cluster node %x (%s) for pid %u",
>+		nodeid,
>+		api->totem_ifaces_print(nodeid),
>+		(unsigned int)req_exec_cpg_procjoin->pid);
> 
>-		if (pi->pid == req_exec_cpg_procjoin->pid && pi->nodeid == nodeid &&
>-			mar_name_compare (&pi->group, &req_exec_cpg_procjoin->group_name)==0) {
>-			list_del (&pi->list);
>-			free (pi);
>-		}
>-	}
>+	do_proc_leave (&req_exec_cpg_procjoin->group_name,
>+		req_exec_cpg_procjoin->pid, nodeid,
>+		req_exec_cpg_procjoin->reason);
> }
> 
>

Actions: View | Diff

Attachments on bug 1080103: 879812 | 879813 | 879814