]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
scout: display fetch and store counts as unsigned
authorMichael Meffie <mmeffie@sinenomine.net>
Thu, 23 Sep 2010 14:15:57 +0000 (10:15 -0400)
committerJeffrey Altman <jaltman@openafs.org>
Fri, 24 Sep 2010 13:49:08 +0000 (06:49 -0700)
Fetches and stores are already defined as unsigned, so format
them as unsigned values when displaying in scout. This fixes
the bug where scout shows those counts as negative values on
busy servers which have been running for a while.

Change-Id: I460172720d76e081ede4381b2718f75386af4285
Reviewed-on: http://gerrit.openafs.org/2843
Reviewed-by: Simon Wilkinson <sxw@inf.ed.ac.uk>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
src/scout/scout.c

index 85fced53043d1dfe98bb00aa853b899e9d70f5fe..78e16d5a05c035764cca7538e5cc59a76e982631 100644 (file)
@@ -1379,7 +1379,7 @@ FS_Handler(void)
        lightdata = (struct gator_lightobj *)(curr_line->fetches_lp->o_data);
        if (*curr_probeOK == 0) {
            sp = s;
-           sprintf(sp, "%d", curr_stats->TotalFetchs);
+           sprintf(sp, "%u", curr_stats->TotalFetchs);
        } else
            sp = sblank;
        code = mini_justify(sp, /*Src buffer */
@@ -1398,7 +1398,7 @@ FS_Handler(void)
        lightdata = (struct gator_lightobj *)(curr_line->stores_lp->o_data);
        if (*curr_probeOK == 0) {
            sp = s;
-           sprintf(sp, "%d", curr_stats->TotalStores);
+           sprintf(sp, "%u", curr_stats->TotalStores);
        } else
            sp = sblank;
        code = mini_justify(sp, /*Src buffer */