RE: [PATCH 02/15] NFSv4.1: Clean up nfs4_setup_sequence

[Trond Myklebust <trond.myklebust@fys.uio.no>]

On Tue, 2010-06-15 at 09:50 -0700, Gilliam, PaulX J wrote:
> > -----Original Message-----
> > From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-owner@vger.kernel.org] On Behalf Of Trond Myklebust
> > Sent: Monday, June 14, 2010 2:51 PM
> > To: linux-nfs@vger.kernel.org
> > Subject: [PATCH 02/15] NFSv4.1: Clean up nfs4_setup_sequence
> > 
> > Firstly, there is little point in first zeroing out the entire struct
> > nfs4_sequence_res, and then initialising all fields save one. Just
> > initialise the last field to zero...
> 
> The reason one may want to zero out the entire struct is that in the future, someone may add elements to the struct.  In that case, 
> 	memset(res, 0, sizeof(*res));
> would not have to be changed, and any new elements will "automatically" be initialized to a known value.
> 
> Just a thought.

That assumes this is a structure that is likely to change and/or be
extended in the future, which is unlikely since the NFSv4.1 protocol
specification is complete and the SEQUENCE results are fully contained
in the current set of fields.

It also assumes that '0' would be a desirable default value for these
new fields.

Trond