From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiaju Zhang Date: Mon, 26 Oct 2009 18:44:38 +0800 Subject: [Cluster-devel] Re: [RFC PATCH] dlm: enhancing dlm_controld (pcmk) to be able to handle redundant rings In-Reply-To: <20091023205104.GE31794@redhat.com> References: <20091023103645.GA6438@linux-jjzhang> <20091023175520.GC31794@redhat.com> <1256325489.2698.7.camel@localhost.localdomain> <20091023205104.GE31794@redhat.com> Message-ID: List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On Sat, Oct 24, 2009 at 4:51 AM, David Teigland wrote: > On Fri, Oct 23, 2009 at 12:18:09PM -0700, Steven Dake wrote: >> On Fri, 2009-10-23 at 12:55 -0500, David Teigland wrote: >> > On Fri, Oct 23, 2009 at 09:23:20PM +0800, Jiaju Zhang wrote: >> > + ? ?result = corosync_cfg_ring_status_get(handle, >> > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &interface_names, >> > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &interface_status, >> > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &interface_count); >> > + ? ?if (result != CS_OK) { >> > + ? ? ? log_error("Failed to get the ring status (error=%d)", result); >> > + ? ? ? (void)corosync_cfg_finalize(handle); >> > + ? ? ? return -1; >> > + ? ?} >> > + >> > + ? ?(void)corosync_cfg_finalize(handle); >> > + ? ?return interface_count; >> > >> > What is interface_count if there's one ring up but rrp is configured? >> > Could we read the totem/rrp config value from the objdb? > > >> Dave, >> >> The rrp mode can be read from the confdb currently. > > Ah, yeah, that's what I meant. Hello, Attached is the improved patch, which read the rrp mode from the confdb. Thanks a lot for your comments :) Thanks, Jiaju -------------- next part -------------- A non-text attachment was scrubbed... Name: dlm-pcmk-config-protocol.diff Type: text/x-diff Size: 5130 bytes Desc: not available URL: