From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Yarbrough Date: Fri, 18 Aug 2006 13:11:42 -0700 Subject: [Cluster-devel] Re: [NFS] [PATCH 2/5] NLM failover - per fs grace period References: <1155535221.3416.26.camel@localhost.localdomain> <1155894561.17651.442.camel@hole.melbourne.sgi.com> Message-ID: <44E61EFE.F41140AD@sgi.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Greg Banks wrote: > > On Mon, 2006-08-14 at 16:00, Wendy Cheng wrote: > > This change enables per NFS-export entry lockd grace period.[...] > > > > > +/* Server fsid linked list for NLM lock failover */ > > +struct nlm_serv { > > + struct nlm_serv* s_next; /* linked list */ > > + unsigned long s_grace_period; /* per fsid grace period */ > > + int s_fsid; /* export fsid */ > > +}; > > + > > The name of this structure appears to be left over from your > previous approach; it doesn't really represent a server anymore. > Giving the structure, and list, and the lock that protects it > similar and appropriate names might be nice. > > Also, the s_grace_period field isn't actually a period, it's > the future expiry time expressed in jiffies. The field name > and comment are both confusing. It might be a good idea to change s_grace_period to something like s_grace_end since it actually marks the ending time of the grace period. If you do change the name, it would be a good idea to enhance the commentary to indicate the relationship of the field to the grace period. That should leave enough breadcrumbs for anyone familiar with the NLM terminology to follow. -- jmy at sgi.com 650 933 3124 I need more snakes.