cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
From: lhh@sourceware.org <lhh@sourceware.org>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] cluster/rgmanager src/daemons/main.c ChangeLog
Date: 12 Apr 2007 17:23:06 -0000	[thread overview]
Message-ID: <20070412172306.15012.qmail@sourceware.org> (raw)

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL5
Changes by:	lhh at sourceware.org	2007-04-12 18:23:05

Modified files:
	rgmanager/src/daemons: main.c 
	rgmanager      : ChangeLog 

Log message:
	Fix watchdog race on rgmanager exit; BZ#236204, patch from Andrey Mirkin

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/rgmanager/src/daemons/main.c.diff?cvsroot=cluster&only_with_tag=RHEL5&r1=1.34.2.1&r2=1.34.2.2
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/rgmanager/ChangeLog.diff?cvsroot=cluster&only_with_tag=RHEL5&r1=1.31&r2=1.31.2.1

--- cluster/rgmanager/src/daemons/main.c	2007/03/20 17:09:11	1.34.2.1
+++ cluster/rgmanager/src/daemons/main.c	2007/04/12 17:23:05	1.34.2.2
@@ -881,6 +881,18 @@
 			clulog(LOG_NOTICE, "Failed to start watchdog\n");
 	}
 
+	setup_signal(SIGINT, flag_shutdown);
+	setup_signal(SIGTERM, flag_shutdown);
+	setup_signal(SIGUSR1, statedump);
+	unblock_signal(SIGCHLD);
+	setup_signal(SIGPIPE, SIG_IGN);
+
+	if (debug) {
+		setup_signal(SIGSEGV, segfault);
+	} else {
+		unblock_signal(SIGSEGV);
+	}
+
 	clu_initialize(&clu);
 	if (cman_init_subsys(clu) < 0) {
 		perror("cman_init_subsys");
@@ -916,18 +928,6 @@
 		return -1;
 	}
 
-	setup_signal(SIGINT, flag_shutdown);
-	setup_signal(SIGTERM, flag_shutdown);
-	setup_signal(SIGUSR1, statedump);
-	unblock_signal(SIGCHLD);
-	setup_signal(SIGPIPE, SIG_IGN);
-
-	if (debug) {
-		setup_signal(SIGSEGV, segfault);
-	} else {
-		unblock_signal(SIGSEGV);
-	}
-
 	if (msg_listen(MSG_SOCKET, RGMGR_SOCK, me.cn_nodeid, &local_ctx) < 0) {
 		clulog(LOG_CRIT,
 		       "#10: Couldn't set up cluster message system: %s\n",
--- cluster/rgmanager/ChangeLog	2006/10/25 15:39:41	1.31
+++ cluster/rgmanager/ChangeLog	2007/04/12 17:23:05	1.31.2.1
@@ -1,3 +1,9 @@
+2007-04-12 Lon Hohberger <lhh@redhat.com>
+	* src/daemons/main.c: Fix watchdog race condition causing
+	reboot; patch from Andrey Mirkin; bugzilla #236204
+
+========
+	
 2006-10-25 Lon Hohberger <lhh@redhat.com>
 	* src/resources/clusterfs.sh: Fix unmounting problem (#212074)
 



             reply	other threads:[~2007-04-12 17:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-12 17:23 lhh [this message]
  -- strict thread matches above, loose matches on Subject: below --
2007-04-12 17:25 [Cluster-devel] cluster/rgmanager src/daemons/main.c ChangeLog lhh
2007-04-19 20:21 lhh

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070412172306.15012.qmail@sourceware.org \
    --to=lhh@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).