From a66c72c37140e17f600f9cab8f8f3b09374a96c4 Mon Sep 17 00:00:00 2001 From: Stephen Champion Date: Fri, 30 May 2014 17:35:45 -0700 Subject: [PATCH] LU-5123 procfs: u64 helper does not respect multiplier lprocfs_write_frac_u64_helper should apply the multiplier argument to the value if units are not specified by the user. It has been overriding the multiplier with '1' regardless. Signed-off-by: Stephen Champion Change-Id: Iaacbb6923122479ba1ba4a5ee0088c670245d081 Reviewed-on: http://review.whamcloud.com/10535 Tested-by: Jenkins Reviewed-by: John L. Hammond Reviewed-by: James Simmons Tested-by: Maloo Reviewed-by: Keith Mannthey Reviewed-by: Andreas Dilger --- lustre/obdclass/lprocfs_status.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/obdclass/lprocfs_status.c b/lustre/obdclass/lprocfs_status.c index d81ebc3..885b4d4 100644 --- a/lustre/obdclass/lprocfs_status.c +++ b/lustre/obdclass/lprocfs_status.c @@ -2487,7 +2487,7 @@ int lprocfs_write_frac_u64_helper(const char *buffer, unsigned long count, } } /* Specified units override the multiplier */ - if (units) + if (units > 1) mult = mult < 0 ? -units : units; frac *= mult; -- 1.8.3.1