Re: [PATCH 3/3] nfsstat: Eliminate the displaying zero stats

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Steve Dickson wrote:
Hey Bruce,

J. Bruce Fields wrote:
On Wed, Apr 08, 2009 at 12:05:08PM -0400, Steve Dickson wrote:
Eliminate the displaying zero stats when the explicit protocol
is specified (-2, -3, -4) the -Z and or --list options.
When a particular protocol is specified and either
the -Z or --list options are used, zeros or blank lines
are echoed to the screen when there is not any NFS traffic.
This cause any useful data to be scroll off the screen.
Sound like good fixes.

With this patch only non-zero stats will be shown, which
makes the output of these options more condensed and
in turn more useful.
Couldn't this make the output harder to parse?  E.g. if you want to know
how many getattr's were sent each second, it might be simpler if you say
a 0 those times when there were no getattrs.

Maybe.. but if one was parsing the output and the parser didn't see a gettr stat, I'm assuming it would not increment the "gettr counter" so it would be know there were zero gettars in that time period.. The main reason I did this was, when watching reads and writes, 90% of the output was zeros stats which made very difficult to see the non zero counts (i.e. read and write stats) since they generally got scrolled of the screen... Also note, when the --list option is not used, zero stats are shown in the column output, but granted, the column output
is a bit harder to parse than the list output.

I guess I could change it so non zero stats are show only the --sleep option is *not* used....

Dare I suggest another commandline argument? I didn't really like the idea of leaving out the stats with 0's, but after trying it, the output does look a lot cleaner. Perhaps giving users the choice would be the way to go.

-kevin

steved.


--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
------------------------------------------------------------
Kevin Constantine

Systems Engineer		t: 818.460.8221
Walt Disney Animation Studios	e: kevin.constantine@xxxxxxxxxx

Any sufficiently advanced technology is indistinguishable from magic.
    - Arthur C. Clarke
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux