Fixes network stats printf strings
authorBarret Rhoden <brho@cs.berkeley.edu>
Tue, 10 Jun 2014 19:09:24 +0000 (12:09 -0700)
committerBarret Rhoden <brho@cs.berkeley.edu>
Tue, 10 Jun 2014 19:09:24 +0000 (12:09 -0700)
All of these protocols are using u32s for their stats and other stuff.  The
format strings were looking for longs instead of ints.  This fixes them to use ints.

Alternatively, we could change the stats fields to u64.  I'm indifferent.

I didn't both changing any of the protocols that we don't use or even compile.

kern/src/net/icmp.c
kern/src/net/icmp6.c
kern/src/net/ip.c
kern/src/net/tcp.c
kern/src/net/udp.c

index 61ae3c5..740993c 100644 (file)
@@ -450,13 +450,13 @@ int icmpstats(struct Proto *icmp, char *buf, int len)
        p = buf;
        e = p + len;
        for (i = 0; i < Nstats; i++)
-               p = seprintf(p, e, "%s: %lu\n", statnames[i], priv->stats[i]);
+               p = seprintf(p, e, "%s: %u\n", statnames[i], priv->stats[i]);
        for (i = 0; i <= Maxtype; i++) {
                if (icmpnames[i])
-                       p = seprintf(p, e, "%s: %lu %lu\n", icmpnames[i], priv->in[i],
+                       p = seprintf(p, e, "%s: %u %u\n", icmpnames[i], priv->in[i],
                                                 priv->out[i]);
                else
-                       p = seprintf(p, e, "%d: %lu %lu\n", i, priv->in[i], priv->out[i]);
+                       p = seprintf(p, e, "%d: %u %u\n", i, priv->in[i], priv->out[i]);
        }
        return p - buf;
 }
index ba40c1f..678f0e3 100644 (file)
@@ -903,13 +903,13 @@ int icmpstats6(struct Proto *icmp6, char *buf, int len)
        p = buf;
        e = p + len;
        for (i = 0; i < Nstats6; i++)
-               p = seprintf(p, e, "%s: %lu\n", statnames6[i], priv->stats[i]);
+               p = seprintf(p, e, "%s: %u\n", statnames6[i], priv->stats[i]);
        for (i = 0; i <= Maxtype6; i++) {
                if (icmpnames6[i])
-                       p = seprintf(p, e, "%s: %lu %lu\n", icmpnames6[i], priv->in[i],
+                       p = seprintf(p, e, "%s: %u %u\n", icmpnames6[i], priv->in[i],
                                                 priv->out[i]);
 /*             else
-                       p = seprintf(p, e, "%d: %lu %lu\n", i, priv->in[i], priv->out[i]);
+                       p = seprintf(p, e, "%d: %u %u\n", i, priv->in[i], priv->out[i]);
 */
        }
        return p - buf;
index 27e16ca..ee0888f 100644 (file)
@@ -561,7 +561,7 @@ int ipstats(struct Fs *f, char *buf, int len)
        p = buf;
        e = p + len;
        for (i = 0; i < Nstats; i++)
-               p = seprintf(p, e, "%s: %lu\n", statnames[i], ip->stats[i]);
+               p = seprintf(p, e, "%s: %u\n", statnames[i], ip->stats[i]);
        return p - buf;
 }
 
index 9fe9ef3..b2bce39 100644 (file)
@@ -3054,7 +3054,7 @@ int tcpstats(struct Proto *tcp, char *buf, int len)
        p = buf;
        e = p + len;
        for (i = 0; i < Nstats; i++)
-               p = seprintf(p, e, "%s: %lu\n", statnames[i], priv->stats[i]);
+               p = seprintf(p, e, "%s: %u\n", statnames[i], priv->stats[i]);
        return p - buf;
 }
 
index 3759040..ff22719 100644 (file)
@@ -632,10 +632,10 @@ int udpstats(struct Proto *udp, char *buf, int len)
        upriv = udp->priv;
        p = buf;
        e = p + len;
-       p = seprintf(p, e, "InDatagrams: %lu\n", upriv->ustats.udpInDatagrams);
-       p = seprintf(p, e, "NoPorts: %lu\n", upriv->ustats.udpNoPorts);
-       p = seprintf(p, e, "InErrors: %lu\n", upriv->ustats.udpInErrors);
-       p = seprintf(p, e, "OutDatagrams: %lu\n", upriv->ustats.udpOutDatagrams);
+       p = seprintf(p, e, "InDatagrams: %u\n", upriv->ustats.udpInDatagrams);
+       p = seprintf(p, e, "NoPorts: %u\n", upriv->ustats.udpNoPorts);
+       p = seprintf(p, e, "InErrors: %u\n", upriv->ustats.udpInErrors);
+       p = seprintf(p, e, "OutDatagrams: %u\n", upriv->ustats.udpOutDatagrams);
        return p - buf;
 }