]> 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)
committerDerrick Brashear <shadow@dementia.org>
Tue, 5 Oct 2010 19:16:33 +0000 (12:16 -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.

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>
(cherry picked from commit e28bb036b85ea90c0ecf8d52a17043374c1fef3a)
Change-Id: If4d0a72b94e743dd336f31b4c21da7e68f800f11
Reviewed-on: http://gerrit.openafs.org/2923
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.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 */