Re: nfs-utils 1.0.9-rc1 - hopefully -final in a week

["Shankar Anand" <shanand@novell.com>]

Hi Neil,

>>> On 7/5/2006 at 11:33 AM,  Neil Brown <neilb@suse.de> wrote:
> On Tuesday July 4, shanand@novell.com wrote:
>> 
>> The patch attached here reads /proc/net/rpc/nfsd for nfsv4 server
>> statistics and displays them.
>> 
>> To maintain uniformity with older nfs versions operations 0 1 and 2
>> are also printed though they are not used by nfsv4.
> 
> Good..
> 
>> @@ -115,6 +126,7 @@ static statinfo		svcinfo[] = {
>>  	{ "proc2",      STRUCTSIZE(svcv2info),  svcv2info  },
>>  	{ "proc3",      STRUCTSIZE(svcv3info),  svcv3info  },
>>  	{ "proc4",      STRUCTSIZE(svcv4info),  svcv4info  },
>> +	{ "proc4ops",   STRUCTSIZE(svcv4opinfo),svcv4opinfo},
>>  	{ NULL,         0,                      NULL       }
>>  };
>>  
> 
> As it is called 'proc4ops', does that mean it will apply to all
> minor-versions, or do we expect a
> 	proc4.1ops
> 
> for NFSv4.1?  Or maybe proc4ops.1 ??
> 
> It doesn't really matter one way or the other, but I would like to
> know that the issue has been considered, and to know what the
current
> decision is.

Though the issue has not been discussed earlier on the mailing list,
my thought when I wrote the patch was that instead of creating 
new lines for each minor version (which may lead to confusion), 
we can add the additional procedures, if at all, to the same line. 

Anyway the server is going to have either nfs4 or nfs4.1 running at
one point in time. So it should be fine.

> (Meanwhile, The current patch is in .git waiting for the 1.0.9
> release).

Thanks.

Shankar

Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs