From: Andrew Deason <adeason@sinenomine.net>
Date: Fri, 2 Jul 2010 19:28:29 +0000 (-0500)
Subject: Fix VPrintDiskStats_r logging
X-Git-Tag: openafs-devel-1_5_76~165
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=c72cd42364206f4fe237d26e582cef86351f75a8;p=packages%2Fo%2Fopenafs.git

Fix VPrintDiskStats_r logging

VPrintDiskStats_r tried to log some information using separate Log()
statements on the same line. This looks very odd when logged, since we
add a timestamp to every Log() call these days. Fix it to print the
line as one call to Log().

Change-Id: I19a5365c24208229201ba54ff04e793b5e50eee5
Reviewed-on: http://gerrit.openafs.org/2359
Tested-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/vol/partition.c b/src/vol/partition.c
index c97668ae3..ca0aa3bc9 100644
--- a/src/vol/partition.c
+++ b/src/vol/partition.c
@@ -1092,12 +1092,15 @@ VPrintDiskStats_r(void)
 {
     struct DiskPartition64 *dp;
     for (dp = DiskPartitionList; dp; dp = dp->next) {
-	Log("Partition %s: %d available 1K blocks (minfree=%d), ", dp->name,
-	    dp->totalUsable, dp->minFree);
-	if (dp->free < 0)
-	    Log("overallocated by %d blocks\n", -dp->free);
-	else
-	    Log("%d free blocks\n", dp->free);
+	if (dp->free < 0) {
+	    Log("Partition %s: %d available 1K blocks (minfree=%d), "
+	        "overallocated by %d blocks\n", dp->name,
+	        dp->totalUsable, dp->minFree, -dp->free);
+	} else {
+	    Log("Partition %s: %d available 1K blocks (minfree=%d), "
+	        "%d free blocks\n", dp->name,
+	        dp->totalUsable, dp->minFree, dp->free);
+	}
     }
 }