From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Dake Date: Fri, 23 Oct 2009 12:18:09 -0700 Subject: [Cluster-devel] Re: [RFC PATCH] dlm: enhancing dlm_controld (pcmk) to be able to handle redundant rings In-Reply-To: <20091023175520.GC31794@redhat.com> References: <20091023103645.GA6438@linux-jjzhang> <20091023175520.GC31794@redhat.com> Message-ID: <1256325489.2698.7.camel@localhost.localdomain> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit 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 > Dave, The rrp mode can be read from the confdb currently. The other values cannot be read from the confdb currently. interface_count is the number of configured interfaces so that interface_status may be iterated. Long term the plan is to deprecate all of the APIs related to config/diag in favor of using confdb (or some successor to this api if absolutely necessary). This includes things like the above ring status api. We have alot of work to do to get there. Corosync 2.0 material.