diff options
author | Max Kellermann <max@duempel.org> | 2005-09-06 18:18:03 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-09-07 19:57:39 -0400 |
commit | 49e31cbac5be2202f351626fd4fb33ad4d4819b8 (patch) | |
tree | a2d33495131b874e1cc4895ba21723e863ca63b4 /net/sunrpc/stats.c | |
parent | 7ea6040b0eff07d3a9a4e2d248ac137c6ad02d42 (diff) |
[PATCH] sunrpc: print unsigned integers in stats
The sunrpc stats are collected in unsigned integers, but they are printed
with '%d'. That can result in negative numbers in /proc/net/rpc when the
highest bit of a counter is set. The following patch changes '%d' to '%u'
where appropriate.
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'net/sunrpc/stats.c')
-rw-r--r-- | net/sunrpc/stats.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/net/sunrpc/stats.c b/net/sunrpc/stats.c index 9b67dc19944c..4979f226e285 100644 --- a/net/sunrpc/stats.c +++ b/net/sunrpc/stats.c | |||
@@ -35,13 +35,13 @@ static int rpc_proc_show(struct seq_file *seq, void *v) { | |||
35 | int i, j; | 35 | int i, j; |
36 | 36 | ||
37 | seq_printf(seq, | 37 | seq_printf(seq, |
38 | "net %d %d %d %d\n", | 38 | "net %u %u %u %u\n", |
39 | statp->netcnt, | 39 | statp->netcnt, |
40 | statp->netudpcnt, | 40 | statp->netudpcnt, |
41 | statp->nettcpcnt, | 41 | statp->nettcpcnt, |
42 | statp->nettcpconn); | 42 | statp->nettcpconn); |
43 | seq_printf(seq, | 43 | seq_printf(seq, |
44 | "rpc %d %d %d\n", | 44 | "rpc %u %u %u\n", |
45 | statp->rpccnt, | 45 | statp->rpccnt, |
46 | statp->rpcretrans, | 46 | statp->rpcretrans, |
47 | statp->rpcauthrefresh); | 47 | statp->rpcauthrefresh); |
@@ -50,10 +50,10 @@ static int rpc_proc_show(struct seq_file *seq, void *v) { | |||
50 | const struct rpc_version *vers = prog->version[i]; | 50 | const struct rpc_version *vers = prog->version[i]; |
51 | if (!vers) | 51 | if (!vers) |
52 | continue; | 52 | continue; |
53 | seq_printf(seq, "proc%d %d", | 53 | seq_printf(seq, "proc%u %u", |
54 | vers->number, vers->nrprocs); | 54 | vers->number, vers->nrprocs); |
55 | for (j = 0; j < vers->nrprocs; j++) | 55 | for (j = 0; j < vers->nrprocs; j++) |
56 | seq_printf(seq, " %d", | 56 | seq_printf(seq, " %u", |
57 | vers->procs[j].p_count); | 57 | vers->procs[j].p_count); |
58 | seq_putc(seq, '\n'); | 58 | seq_putc(seq, '\n'); |
59 | } | 59 | } |
@@ -83,13 +83,13 @@ void svc_seq_show(struct seq_file *seq, const struct svc_stat *statp) { | |||
83 | int i, j; | 83 | int i, j; |
84 | 84 | ||
85 | seq_printf(seq, | 85 | seq_printf(seq, |
86 | "net %d %d %d %d\n", | 86 | "net %u %u %u %u\n", |
87 | statp->netcnt, | 87 | statp->netcnt, |
88 | statp->netudpcnt, | 88 | statp->netudpcnt, |
89 | statp->nettcpcnt, | 89 | statp->nettcpcnt, |
90 | statp->nettcpconn); | 90 | statp->nettcpconn); |
91 | seq_printf(seq, | 91 | seq_printf(seq, |
92 | "rpc %d %d %d %d %d\n", | 92 | "rpc %u %u %u %u %u\n", |
93 | statp->rpccnt, | 93 | statp->rpccnt, |
94 | statp->rpcbadfmt+statp->rpcbadauth+statp->rpcbadclnt, | 94 | statp->rpcbadfmt+statp->rpcbadauth+statp->rpcbadclnt, |
95 | statp->rpcbadfmt, | 95 | statp->rpcbadfmt, |
@@ -99,9 +99,9 @@ void svc_seq_show(struct seq_file *seq, const struct svc_stat *statp) { | |||
99 | for (i = 0; i < prog->pg_nvers; i++) { | 99 | for (i = 0; i < prog->pg_nvers; i++) { |
100 | if (!(vers = prog->pg_vers[i]) || !(proc = vers->vs_proc)) | 100 | if (!(vers = prog->pg_vers[i]) || !(proc = vers->vs_proc)) |
101 | continue; | 101 | continue; |
102 | seq_printf(seq, "proc%d %d", i, vers->vs_nproc); | 102 | seq_printf(seq, "proc%d %u", i, vers->vs_nproc); |
103 | for (j = 0; j < vers->vs_nproc; j++, proc++) | 103 | for (j = 0; j < vers->vs_nproc; j++, proc++) |
104 | seq_printf(seq, " %d", proc->pc_count); | 104 | seq_printf(seq, " %u", proc->pc_count); |
105 | seq_putc(seq, '\n'); | 105 | seq_putc(seq, '\n'); |
106 | } | 106 | } |
107 | } | 107 | } |