nfsstat.c: Implemented --sleep[interval] option.
nfsstat.man: Added information about the --sleep option
Signed-off-by: Kevin Constantine <kevin.constantine-FfNkGbSheRGpB8w63BLUukEOCMrvLtNR@public.gmane.org>
---
utils/nfsstat/nfsstat.c | 158 +++++++++++++++++++++++++++++++++++++--------
utils/nfsstat/nfsstat.man | 6 ++-
2 files changed, 137 insertions(+), 27 deletions(-)
diff --git a/utils/nfsstat/nfsstat.c b/utils/nfsstat/nfsstat.c
index 1517414..c25580c 100644
--- a/utils/nfsstat/nfsstat.c
+++ b/utils/nfsstat/nfsstat.c
@@ -167,10 +167,16 @@ DECLARE_SRV(srvinfo, _old);
DECLARE_CLT(cltinfo);
DECLARE_CLT(cltinfo, _old);
+static void print_server_stats(int, int);
+static void print_client_stats(int, int);
+static void print_stats_list(int);
static void print_numbers(const char *, unsigned int *,
unsigned int);
static void print_callstats(const char *, const char **,
unsigned int *, unsigned int);
+static void print_callstats_list(const char *, const char **,
+ unsigned int *, unsigned int *,
+ unsigned int);
static int parse_raw_statfile(const char *, struct statinfo *);
static int parse_pretty_statfile(const char *, struct statinfo *);
@@ -183,6 +189,7 @@ static void get_stats(const char *, struct statinfo *, int *, int,
static int has_stats(const unsigned int *);
static void diff_stats(struct statinfo *, struct statinfo *, int);
static void unpause(int);
+static void update_old_counters(struct statinfo *, struct statinfo *);
static time_t starttime;
@@ -207,26 +214,29 @@ void usage(char *name)
{
printf("Usage: %s [OPTION]...\n\
\n\
- -m, --mounts\t\tShow statistics on mounted NFS filesystems\n\
- -c, --client\t\tShow NFS client statistics\n\
- -s, --server\t\tShow NFS server statistics\n\
- -2\t\t\tShow NFS version 2 statistics\n\
- -3\t\t\tShow NFS version 3 statistics\n\
- -4\t\t\tShow NFS version 4 statistics\n\
- -o [facility]\t\tShow statistics on particular facilities.\n\
- nfs\tNFS protocol information\n\
- rpc\tGeneral RPC information\n\
- net\tNetwork layer statistics\n\
- fh\t\tUsage information on the server's file handle cache\n\
- rc\t\tUsage information on the server's request reply cache\n\
- all\tSelect all of the above\n\
- -v, --verbose, --all\tSame as '-o all'\n\
- -r, --rpc\t\tShow RPC statistics\n\
- -n, --nfs\t\tShow NFS statistics\n\
- -Z, --sleep\t\tSaves stats, pauses, diffs current and saved\n\
- -S, --since file\tShows difference between current stats and those in 'file'\n\
- --version\t\tShow program version\n\
- --help\t\tWhat you just did\n\
+ -m, --mounts Show statistics on mounted NFS filesystems\n\
+ -c, --client Show NFS client statistics\n\
+ -s, --server Show NFS server statistics\n\
+ -2 Show NFS version 2 statistics\n\
+ -3 Show NFS version 3 statistics\n\
+ -4 Show NFS version 4 statistics\n\
+ -o [facility] Show statistics on particular facilities.\n\
+ nfs NFS protocol information\n\
+ rpc General RPC information\n\
+ net Network layer statistics\n\
+ fh Usage information on the server's file handle cache\n\
+ rc Usage information on the server's request reply cache\n\
+ all Select all of the above\n\
+ -v, --verbose, --all Same as '-o all'\n\
+ -r, --rpc Show RPC statistics\n\
+ -n, --nfs Show NFS statistics\n\
+ -Z[#], --sleep[=#] Collects stats until interrupted.\n\
+ Cumulative stats are then printed\n\
+ If # is provided, stats will be output every\n\
+ # seconds.\n\
+ -S, --since file Shows difference between current stats and those in 'file'\n\
+ --version Show program version\n\
+ --help What you just did\n\
\n", name);
exit(0);
}
@@ -245,7 +255,7 @@ static struct option longopts[] =
{ "zero", 0, 0, 'z' },
{ "help", 0, 0, '\1' },
{ "version", 0, 0, '\2' },
- { "sleep", 0, 0, 'Z' },
+ { "sleep", 2, 0, 'Z' },
{ "since", 1, 0, 'S' },
{ NULL, 0, 0, 0 }
};
@@ -258,6 +268,7 @@ main(int argc, char **argv)
opt_clt = 0,
opt_prt = 0,
opt_sleep = 0,
+ sleep_time = 0,
opt_since = 0;
int c;
char *progname,
@@ -279,7 +290,7 @@ main(int argc, char **argv)
else
progname = argv[0];
- while ((c = getopt_long(argc, argv, "234acmno:ZS:vrsz\1\2", longopts, NULL)) != EOF) {
+ while ((c = getopt_long(argc, argv, "234acmno:Z::S:vrsz\1\2", longopts, NULL)) != EOF) {
switch (c) {
case 'a':
fprintf(stderr, "nfsstat: nfs acls are not yet supported.\n");
@@ -311,6 +322,9 @@ main(int argc, char **argv)
break;
case 'Z':
opt_sleep = 1;
+ if (optarg) {
+ sleep_time = atoi(optarg);
+ }
break;
case 'S':
opt_since = 1;
@@ -384,7 +398,7 @@ main(int argc, char **argv)
if (opt_clt)
get_stats(clientfile, clientinfo, &opt_clt, opt_srv, 0);
- if (opt_sleep) {
+ if (opt_sleep && !sleep_time) {
starttime = time(NULL);
printf("Collecting statistics; press CTRL-C to view results from interval (i.e., from pause to CTRL-C).\n");
if (sigaction(SIGINT, &act, NULL) != 0) {
@@ -404,7 +418,33 @@ main(int argc, char **argv)
diff_stats(clientinfo_tmp, clientinfo, 0);
}
}
+ if(sleep_time) {
+ while(1) {
+ if (opt_srv) {
+ get_stats(NFSSRVSTAT, serverinfo_tmp , &opt_srv, opt_clt, 1);
+ diff_stats(serverinfo_tmp, serverinfo, 1);
+ }
+ if (opt_clt) {
+ get_stats(NFSCLTSTAT, clientinfo_tmp, &opt_clt, opt_srv, 0);
+ diff_stats(clientinfo_tmp, clientinfo, 0);
+ }
+ print_stats_list(opt_prt);
+ fflush(stdout);
+
+ update_old_counters(clientinfo_tmp, clientinfo);
+ sleep(sleep_time);
+ }
+ } else {
+ print_server_stats(opt_srv, opt_prt);
+ print_client_stats(opt_clt, opt_prt);
+ }
+
+ return 0;
+}
+static void
+print_server_stats(int opt_srv, int opt_prt)
+{
if (opt_srv) {
if (opt_prt & PRNT_NET) {
print_numbers(
@@ -479,7 +519,10 @@ main(int argc, char **argv)
}
}
}
-
+}
+static void
+print_client_stats(int opt_clt, int opt_prt)
+{
if (opt_clt) {
if (opt_prt & PRNT_NET) {
print_numbers(
@@ -515,10 +558,43 @@ main(int argc, char **argv)
);
}
}
-
- return 0;
}
+static void
+print_stats_list(int opt_prt)
+{
+ if (opt_prt & PRNT_CALLS) {
+ if ((opt_prt & PRNT_V2) || ((opt_prt & PRNT_AUTO) && has_stats(cltproc2info)))
+ print_callstats_list(
+ "nfs v2 stats:",
+ nfsv2name, srvproc2info + 1, cltproc2info + 1,
+ sizeof(nfsv2name)/sizeof(char *)
+ );
+ printf("\n");
+ if ((opt_prt & PRNT_V3) || ((opt_prt & PRNT_AUTO) && has_stats(cltproc3info)))
+ print_callstats_list(
+ "nfs v3 stats:",
+ nfsv3name, srvproc3info + 1, cltproc3info + 1,
+ sizeof(nfsv3name)/sizeof(char *)
+ );
+ printf("\n");
+ if ((opt_prt & PRNT_V4) || ((opt_prt & PRNT_AUTO) && has_stats(cltproc4info))) {
+ print_callstats(
+ LABEL_srvproc4,
+ nfssrvproc4name, srvproc4info + 1, sizeof(nfssrvproc4name)/sizeof(char *)
+ );
+ print_callstats(
+ LABEL_srvproc4ops,
+ nfssrvproc4opname, srvproc4opsinfo + 1, sizeof(nfssrvproc4opname)/sizeof(char *)
+ );
+ print_callstats(
+ LABEL_cltproc4,
+ nfscltproc4name, cltproc4info + 1, sizeof(nfscltproc4name)/sizeof(char *)
+ );
+ }
+ }
+}
+
static statinfo *
get_stat_info(const char *sp, struct statinfo *statp)
{
@@ -569,6 +645,26 @@ print_callstats(const char *hdr, const char **names,
printf("\n");
}
+static void
+print_callstats_list(const char *hdr, const char **names,
+ unsigned int *srvinfo, unsigned int *cltinfo,
+ unsigned int nr)
+{
+ unsigned long long srvtotal, clttotal;
+ int i;
+
+ for (i = 0, srvtotal = 0, clttotal = 0; i < nr; i++) {
+ srvtotal += srvinfo[i];
+ clttotal += cltinfo[i];
+ }
+ printf("%13s %8s %8s\n", hdr, "Server", "Client");
+ printf("%12s: %8llu %8llu \n", "total", srvtotal, clttotal);
+ for (i = 0; i < nr; i++) {
+ printf("%12s: %8u %8u \n", names[i], srvinfo[i], cltinfo[i]);
+ }
+
+}
+
/* returns 0 on success, 1 otherwise */
static int
parse_raw_statfile(const char *name, struct statinfo *statp)
@@ -846,3 +942,13 @@ unpause(int sig)
seconds = (int)time_diff % 60;
printf("Signal received; displaying (only) statistics gathered over the last %d minutes, %d seconds:\n\n", minutes, seconds);
}
+
+static void
+update_old_counters(struct statinfo *new, struct statinfo *old)
+{
+ int z, i;
+ for (z = 0; old[z].tag; z++)
+ for (i = 0; i <= old[z].nrvals; i++)
+ old[z].valptr[i] += new[z].valptr[i];
+
+}
diff --git a/utils/nfsstat/nfsstat.man b/utils/nfsstat/nfsstat.man
index cb5f89f..461b2c0 100644
--- a/utils/nfsstat/nfsstat.man
+++ b/utils/nfsstat/nfsstat.man
@@ -91,7 +91,7 @@ output
.I file
are treated as zeroes.
.TP
-.B \-Z, \-\-sleep
+.B \-Z[interval], \-\-sleep=[interval]
Instead of printing current statistics and immediately exiting,
.B nfsstat
takes a snapshot of the current statistics and pauses until it receives
@@ -100,6 +100,10 @@ takes a snapshot of the current statistics and pauses until it receives
.BR Ctrl-C ),
at which point it takes another snapshot and displays the difference
between the two.
+If \fIinterval\fR is specified,
+.B nfsstat
+will print the number of \fBNFS\fR calls made since the previous report.
+Stats will be printed repeatedly every \fIinterval\fR seconds.
.\" --------------------- EXAMPLES -------------------------------
.SH EXAMPLES
.TP
--
1.6.0.4.761.g47577
Kevin Constantine wrote:
> nfsstat.c: Implemented --sleep[interval] option.
> nfsstat.man: Added information about the --sleep option
> Signed-off-by: Kevin Constantine <kevin.constantine-FfNkGbSheRGpB8w63BLUukEOCMrvLtNR@public.gmane.org>
> ---
> utils/nfsstat/nfsstat.c | 158 +++++++++++++++++++++++++++++++++++++--------
> utils/nfsstat/nfsstat.man | 6 ++-
> 2 files changed, 137 insertions(+), 27 deletions(-)
>
> diff --git a/utils/nfsstat/nfsstat.c b/utils/nfsstat/nfsstat.c
> index 1517414..c25580c 100644
> --- a/utils/nfsstat/nfsstat.c
> +++ b/utils/nfsstat/nfsstat.c
> @@ -167,10 +167,16 @@ DECLARE_SRV(srvinfo, _old);
> @@ -183,6 +189,7 @@ static void get_stats(const char *, struct statinfo *, int *, int,
> @@ -207,26 +214,29 @@ void usage(char *name)
> @@ -245,7 +255,7 @@ static struct option longopts[] =
> @@ -258,6 +268,7 @@ main(int argc, char **argv)
> @@ -279,7 +290,7 @@ main(int argc, char **argv)
> @@ -311,6 +322,9 @@ main(int argc, char **argv)
> @@ -384,7 +398,7 @@ main(int argc, char **argv)
>
All good.
> @@ -404,7 +418,33 @@ main(int argc, char **argv)
> diff_stats(clientinfo_tmp, clientinfo, 0);
> }
> }
> + if(sleep_time) {
> + while(1) {
> + if (opt_srv) {
> + get_stats(NFSSRVSTAT, serverinfo_tmp , &opt_srv, opt_clt, 1);
> + diff_stats(serverinfo_tmp, serverinfo, 1);
> + }
> + if (opt_clt) {
> + get_stats(NFSCLTSTAT, clientinfo_tmp, &opt_clt, opt_srv, 0);
> + diff_stats(clientinfo_tmp, clientinfo, 0);
> + }
> + print_stats_list(opt_prt);
> + fflush(stdout);
> +
> + update_old_counters(clientinfo_tmp, clientinfo);
> + sleep(sleep_time);
> + }
> + } else {
> + print_server_stats(opt_srv, opt_prt);
> + print_client_stats(opt_clt, opt_prt);
> + }
> +
>
So what I expected to see here, was both the sleep_time != 0 and
sleep_time == 0 cases calling the same function which printed stats in
the old format. Then the next patch could add the --list option which
added a new feature to that function. In other words,
print_stats_list() would be introduced in the 2nd patch.
> @@ -479,7 +519,10 @@ main(int argc, char **argv)
> @@ -515,10 +558,43 @@ main(int argc, char **argv)
> @@ -569,6 +645,26 @@ print_callstats(const char *hdr, const char **names,
> @@ -846,3 +942,13 @@ unpause(int sig)
> diff --git a/utils/nfsstat/nfsstat.man b/utils/nfsstat/nfsstat.man
> index cb5f89f..461b2c0 100644
> --- a/utils/nfsstat/nfsstat.man
> +++ b/utils/nfsstat/nfsstat.man
> @@ -91,7 +91,7 @@ output
>
>
The manpage change looks fine.
--
Greg Banks, P.Engineer, SGI Australian Software Group.
the brightly coloured sporks of revolution.
I don't speak for SGI.
Hey Kevin,
I apologize for taking so long to look at this... but...
Question: I notice that the v3 stats are displayed one
way (in list format) and the v4 stats are display in
another way (in column format) was this on purpose?
Should they be displayed using the same format?
steved.
Kevin Constantine wrote:
> nfsstat.c: Implemented --sleep[interval] option.
> nfsstat.man: Added information about the --sleep option
> Signed-off-by: Kevin Constantine <kevin.constantine-FfNkGbSheRGpB8w63BLUukEOCMrvLtNR@public.gmane.org>
> ---
> utils/nfsstat/nfsstat.c | 158 +++++++++++++++++++++++++++++++++++++--------
> utils/nfsstat/nfsstat.man | 6 ++-
> 2 files changed, 137 insertions(+), 27 deletions(-)
>
> diff --git a/utils/nfsstat/nfsstat.c b/utils/nfsstat/nfsstat.c
> index 1517414..c25580c 100644
> --- a/utils/nfsstat/nfsstat.c
> +++ b/utils/nfsstat/nfsstat.c
> @@ -167,10 +167,16 @@ DECLARE_SRV(srvinfo, _old);
> DECLARE_CLT(cltinfo);
> DECLARE_CLT(cltinfo, _old);
>
> +static void print_server_stats(int, int);
> +static void print_client_stats(int, int);
> +static void print_stats_list(int);
> static void print_numbers(const char *, unsigned int *,
> unsigned int);
> static void print_callstats(const char *, const char **,
> unsigned int *, unsigned int);
> +static void print_callstats_list(const char *, const char **,
> + unsigned int *, unsigned int *,
> + unsigned int);
> static int parse_raw_statfile(const char *, struct statinfo *);
> static int parse_pretty_statfile(const char *, struct statinfo *);
>
> @@ -183,6 +189,7 @@ static void get_stats(const char *, struct statinfo *, int *, int,
> static int has_stats(const unsigned int *);
> static void diff_stats(struct statinfo *, struct statinfo *, int);
> static void unpause(int);
> +static void update_old_counters(struct statinfo *, struct statinfo *);
>
> static time_t starttime;
>
> @@ -207,26 +214,29 @@ void usage(char *name)
> {
> printf("Usage: %s [OPTION]...\n\
> \n\
> - -m, --mounts\t\tShow statistics on mounted NFS filesystems\n\
> - -c, --client\t\tShow NFS client statistics\n\
> - -s, --server\t\tShow NFS server statistics\n\
> - -2\t\t\tShow NFS version 2 statistics\n\
> - -3\t\t\tShow NFS version 3 statistics\n\
> - -4\t\t\tShow NFS version 4 statistics\n\
> - -o [facility]\t\tShow statistics on particular facilities.\n\
> - nfs\tNFS protocol information\n\
> - rpc\tGeneral RPC information\n\
> - net\tNetwork layer statistics\n\
> - fh\t\tUsage information on the server's file handle cache\n\
> - rc\t\tUsage information on the server's request reply cache\n\
> - all\tSelect all of the above\n\
> - -v, --verbose, --all\tSame as '-o all'\n\
> - -r, --rpc\t\tShow RPC statistics\n\
> - -n, --nfs\t\tShow NFS statistics\n\
> - -Z, --sleep\t\tSaves stats, pauses, diffs current and saved\n\
> - -S, --since file\tShows difference between current stats and those in 'file'\n\
> - --version\t\tShow program version\n\
> - --help\t\tWhat you just did\n\
> + -m, --mounts Show statistics on mounted NFS filesystems\n\
> + -c, --client Show NFS client statistics\n\
> + -s, --server Show NFS server statistics\n\
> + -2 Show NFS version 2 statistics\n\
> + -3 Show NFS version 3 statistics\n\
> + -4 Show NFS version 4 statistics\n\
> + -o [facility] Show statistics on particular facilities.\n\
> + nfs NFS protocol information\n\
> + rpc General RPC information\n\
> + net Network layer statistics\n\
> + fh Usage information on the server's file handle cache\n\
> + rc Usage information on the server's request reply cache\n\
> + all Select all of the above\n\
> + -v, --verbose, --all Same as '-o all'\n\
> + -r, --rpc Show RPC statistics\n\
> + -n, --nfs Show NFS statistics\n\
> + -Z[#], --sleep[=#] Collects stats until interrupted.\n\
> + Cumulative stats are then printed\n\
> + If # is provided, stats will be output every\n\
> + # seconds.\n\
> + -S, --since file Shows difference between current stats and those in 'file'\n\
> + --version Show program version\n\
> + --help What you just did\n\
> \n", name);
> exit(0);
> }
> @@ -245,7 +255,7 @@ static struct option longopts[] =
> { "zero", 0, 0, 'z' },
> { "help", 0, 0, '\1' },
> { "version", 0, 0, '\2' },
> - { "sleep", 0, 0, 'Z' },
> + { "sleep", 2, 0, 'Z' },
> { "since", 1, 0, 'S' },
> { NULL, 0, 0, 0 }
> };
> @@ -258,6 +268,7 @@ main(int argc, char **argv)
> opt_clt = 0,
> opt_prt = 0,
> opt_sleep = 0,
> + sleep_time = 0,
> opt_since = 0;
> int c;
> char *progname,
> @@ -279,7 +290,7 @@ main(int argc, char **argv)
> else
> progname = argv[0];
>
> - while ((c = getopt_long(argc, argv, "234acmno:ZS:vrsz\1\2", longopts, NULL)) != EOF) {
> + while ((c = getopt_long(argc, argv, "234acmno:Z::S:vrsz\1\2", longopts, NULL)) != EOF) {
> switch (c) {
> case 'a':
> fprintf(stderr, "nfsstat: nfs acls are not yet supported.\n");
> @@ -311,6 +322,9 @@ main(int argc, char **argv)
> break;
> case 'Z':
> opt_sleep = 1;
> + if (optarg) {
> + sleep_time = atoi(optarg);
> + }
> break;
> case 'S':
> opt_since = 1;
> @@ -384,7 +398,7 @@ main(int argc, char **argv)
> if (opt_clt)
> get_stats(clientfile, clientinfo, &opt_clt, opt_srv, 0);
>
> - if (opt_sleep) {
> + if (opt_sleep && !sleep_time) {
> starttime = time(NULL);
> printf("Collecting statistics; press CTRL-C to view results from interval (i.e., from pause to CTRL-C).\n");
> if (sigaction(SIGINT, &act, NULL) != 0) {
> @@ -404,7 +418,33 @@ main(int argc, char **argv)
> diff_stats(clientinfo_tmp, clientinfo, 0);
> }
> }
> + if(sleep_time) {
> + while(1) {
> + if (opt_srv) {
> + get_stats(NFSSRVSTAT, serverinfo_tmp , &opt_srv, opt_clt, 1);
> + diff_stats(serverinfo_tmp, serverinfo, 1);
> + }
> + if (opt_clt) {
> + get_stats(NFSCLTSTAT, clientinfo_tmp, &opt_clt, opt_srv, 0);
> + diff_stats(clientinfo_tmp, clientinfo, 0);
> + }
> + print_stats_list(opt_prt);
> + fflush(stdout);
> +
> + update_old_counters(clientinfo_tmp, clientinfo);
> + sleep(sleep_time);
> + }
> + } else {
> + print_server_stats(opt_srv, opt_prt);
> + print_client_stats(opt_clt, opt_prt);
> + }
> +
> + return 0;
> +}
>
> +static void
> +print_server_stats(int opt_srv, int opt_prt)
> +{
> if (opt_srv) {
> if (opt_prt & PRNT_NET) {
> print_numbers(
> @@ -479,7 +519,10 @@ main(int argc, char **argv)
> }
> }
> }
> -
> +}
> +static void
> +print_client_stats(int opt_clt, int opt_prt)
> +{
> if (opt_clt) {
> if (opt_prt & PRNT_NET) {
> print_numbers(
> @@ -515,10 +558,43 @@ main(int argc, char **argv)
> );
> }
> }
> -
> - return 0;
> }
>
> +static void
> +print_stats_list(int opt_prt)
> +{
> + if (opt_prt & PRNT_CALLS) {
> + if ((opt_prt & PRNT_V2) || ((opt_prt & PRNT_AUTO) && has_stats(cltproc2info)))
> + print_callstats_list(
> + "nfs v2 stats:",
> + nfsv2name, srvproc2info + 1, cltproc2info + 1,
> + sizeof(nfsv2name)/sizeof(char *)
> + );
> + printf("\n");
> + if ((opt_prt & PRNT_V3) || ((opt_prt & PRNT_AUTO) && has_stats(cltproc3info)))
> + print_callstats_list(
> + "nfs v3 stats:",
> + nfsv3name, srvproc3info + 1, cltproc3info + 1,
> + sizeof(nfsv3name)/sizeof(char *)
> + );
> + printf("\n");
> + if ((opt_prt & PRNT_V4) || ((opt_prt & PRNT_AUTO) && has_stats(cltproc4info))) {
> + print_callstats(
> + LABEL_srvproc4,
> + nfssrvproc4name, srvproc4info + 1, sizeof(nfssrvproc4name)/sizeof(char *)
> + );
> + print_callstats(
> + LABEL_srvproc4ops,
> + nfssrvproc4opname, srvproc4opsinfo + 1, sizeof(nfssrvproc4opname)/sizeof(char *)
> + );
> + print_callstats(
> + LABEL_cltproc4,
> + nfscltproc4name, cltproc4info + 1, sizeof(nfscltproc4name)/sizeof(char *)
> + );
> + }
> + }
> +}
> +
> static statinfo *
> get_stat_info(const char *sp, struct statinfo *statp)
> {
> @@ -569,6 +645,26 @@ print_callstats(const char *hdr, const char **names,
> printf("\n");
> }
>
> +static void
> +print_callstats_list(const char *hdr, const char **names,
> + unsigned int *srvinfo, unsigned int *cltinfo,
> + unsigned int nr)
> +{
> + unsigned long long srvtotal, clttotal;
> + int i;
> +
> + for (i = 0, srvtotal = 0, clttotal = 0; i < nr; i++) {
> + srvtotal += srvinfo[i];
> + clttotal += cltinfo[i];
> + }
> + printf("%13s %8s %8s\n", hdr, "Server", "Client");
> + printf("%12s: %8llu %8llu \n", "total", srvtotal, clttotal);
> + for (i = 0; i < nr; i++) {
> + printf("%12s: %8u %8u \n", names[i], srvinfo[i], cltinfo[i]);
> + }
> +
> +}
> +
> /* returns 0 on success, 1 otherwise */
> static int
> parse_raw_statfile(const char *name, struct statinfo *statp)
> @@ -846,3 +942,13 @@ unpause(int sig)
> seconds = (int)time_diff % 60;
> printf("Signal received; displaying (only) statistics gathered over the last %d minutes, %d seconds:\n\n", minutes, seconds);
> }
> +
> +static void
> +update_old_counters(struct statinfo *new, struct statinfo *old)
> +{
> + int z, i;
> + for (z = 0; old[z].tag; z++)
> + for (i = 0; i <= old[z].nrvals; i++)
> + old[z].valptr[i] += new[z].valptr[i];
> +
> +}
> diff --git a/utils/nfsstat/nfsstat.man b/utils/nfsstat/nfsstat.man
> index cb5f89f..461b2c0 100644
> --- a/utils/nfsstat/nfsstat.man
> +++ b/utils/nfsstat/nfsstat.man
> @@ -91,7 +91,7 @@ output
> .I file
> are treated as zeroes.
> .TP
> -.B \-Z, \-\-sleep
> +.B \-Z[interval], \-\-sleep=[interval]
> Instead of printing current statistics and immediately exiting,
> .B nfsstat
> takes a snapshot of the current statistics and pauses until it receives
> @@ -100,6 +100,10 @@ takes a snapshot of the current statistics and pauses until it receives
> .BR Ctrl-C ),
> at which point it takes another snapshot and displays the difference
> between the two.
> +If \fIinterval\fR is specified,
> +.B nfsstat
> +will print the number of \fBNFS\fR calls made since the previous report.
> +Stats will be printed repeatedly every \fIinterval\fR seconds.
> .\" --------------------- EXAMPLES -------------------------------
> .SH EXAMPLES
> .TP
Steve Dickson wrote:
> Hey Kevin,
>
> I apologize for taking so long to look at this... but...
>
> Question: I notice that the v3 stats are displayed one
> way (in list format) and the v4 stats are display in
> another way (in column format) was this on purpose?
> Should they be displayed using the same format?
>
Steve-
I re-worked, and re-submitted the patches per some suggestions from
Greg. I've since found an output bug that if --sleep=1 outputs a
newline when there's no nfs traffic. It'd probably just be easiest if I
re-send the patches at this point if that works for you.
-kevin
> steved.
>
> Kevin Constantine wrote:
>> nfsstat.c: Implemented --sleep[interval] option.
>> nfsstat.man: Added information about the --sleep option
>> Signed-off-by: Kevin Constantine <kevin.constantine-FfNkGbSheRGpB8w63BLUukEOCMrvLtNR@public.gmane.org>
>> ---
>> utils/nfsstat/nfsstat.c | 158 +++++++++++++++++++++++++++++++++++++--------
>> utils/nfsstat/nfsstat.man | 6 ++-
>> 2 files changed, 137 insertions(+), 27 deletions(-)
>>
>> diff --git a/utils/nfsstat/nfsstat.c b/utils/nfsstat/nfsstat.c
>> index 1517414..c25580c 100644
>> --- a/utils/nfsstat/nfsstat.c
>> +++ b/utils/nfsstat/nfsstat.c
>> @@ -167,10 +167,16 @@ DECLARE_SRV(srvinfo, _old);
Kevin Constantine wrote:
>
> Steve Dickson wrote:
>> Hey Kevin,
>>
>> I apologize for taking so long to look at this... but...
>>
>> Question: I notice that the v3 stats are displayed one
>> way (in list format) and the v4 stats are display in
>> another way (in column format) was this on purpose? Should they be
>> displayed using the same format?
> Steve-
> I re-worked, and re-submitted the patches per some suggestions from
> Greg. I've since found an output bug that if --sleep=1 outputs a
> newline when there's no nfs traffic. It'd probably just be easiest if I
> re-send the patches at this point if that works for you.
Yeah I did notice that newline bug as well... It seem to echo
newlines even when there was not data show... It was an easy
enough to fix to I just when ahead and fixed it..
I'll wait for the repost.. will you be reposting the
"nfsstat: Add --list flag" patch as well or is that
read to go?
steved.