From mboxrd@z Thu Jan  1 00:00:00 1970
From: David Teigland <teigland@redhat.com>
Date: Fri, 16 Oct 2009 10:59:58 -0500
Subject: [Cluster-devel] Re: gfs2-utils: master - gfs_controld: Remove
	three unused functions
In-Reply-To: <1255704965.6052.589.camel@localhost.localdomain>
References: <20091014145504.9ADCE1201DA@lists.fedorahosted.org>
	<20091014175350.GC28090@redhat.com>
	<1255704965.6052.589.camel@localhost.localdomain>
Message-ID: <20091016155957.GC23459@redhat.com>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

On Fri, Oct 16, 2009 at 03:56:05PM +0100, Steven Whitehouse wrote:
> Hi,
> 
> On Wed, 2009-10-14 at 12:53 -0500, David Teigland wrote:
> > On Wed, Oct 14, 2009 at 02:55:04PM +0000, Steven Whitehouse wrote:
> > > gfs_controld: Remove three unused functions
> > > 
> > > These functions are not called from anywhere and appear
> > > to be left over from earlier times.
> > 
> > They were just added, but in translating the dlm_controld patch to
> > gfs_controld I missed the bits that called them (both in
> > cluster.git/STABLE3 and gfs2-utils.git)  I'll reapply this bit with the
> > bits that are missing.
> > 
> > Dave
> > 
> 
> I'm not sure I understand the purpose of this code. Is there more to
> come yet?
> 
> The function find_mg_id() still seems to be unused. So far as I can
> figure out the purpose of the new code seems to be to maintain two
> timestamps: cluster_add_time whose sole purpose seems to be to check
> against cg->create_time but I'm not quite sure why, and
> cluster_remove_time which seems to not do anything at all at the moment.
> 
> I can't get any clues from dlm_controld because cluster_remove_time
> seems to be unused there as well,

Right, cluster_add_time is used, but cluster_remove_time isn't, although
it can be very useful to know for debugging.

Dave