[PATCH 13/17] nfsd4: add per-operation server stats

Shankar Anand shanand at novell.com
Wed Jul 5 02:30:16 EDT 2006


Hi,

>>> On 7/4/2006 at 11:48 PM, "J.Bruce Fields" <bfields at fieldses.org>
wrote:
> On Tue, Jul 04, 2006 at 08:51:01AM -0600, Shankar Anand wrote:
>> Here is the modified nfs-utils patch. This was also sent to nfs
mailing
>> list.
> 
> Minor complaint: I think Neil and Tom were suggesting the zero-based
> array only for the kernel /proc interface.  I don't think it makes
sense
> for the nfsstat display; in
> 
> op0-unused   op1-unused   op2-future   access       close       
commit
> 0         0% 0         0% 0         0% 1         5% 0         0%
0	0%
> 
> Could we cut out the first three entries?

To do this we will have to write a special case logic in the file
parsing 
function to ignore the first 3 proc file entries for the proc4ops line
alone, 
which may not be good. 

Also if an (experienced) user sees the proc file and compares it  with
the 
nfsstat output, he should not get confused over the difference in the 
number of operations.

Cheers,
Shankar






More information about the NFSv4 mailing list