From f56cb9da50ea8c38d397534cd45aff53d7f693dc Mon Sep 17 00:00:00 2001 From: Chris Rorvick Date: Tue, 16 Dec 2014 22:24:02 -0600 Subject: [PATCH] drivers: staging: lustre: Track sign separately The `mult' parameter is negated if the user data begins with a '-' so that the final value has the appropriate sign. But `mult' is only used if the user data does not include a "units" suffix. In this case, `mult' is overridden with the numeric scale conveyed by the units suffix, but retains the sign of the original value. Having `mult' serving double-duty works but is confusing. Use a new local variable to store the sign of the user data instead. This also fixes a pitfall of passing 0 to `mult', expecting it to be ignored when a units suffix is specified, but having the effect of taking the absolute value of the user-provided data. Signed-off-by: Chris Rorvick Reviewed-by: Andreas Dilger Signed-off-by: Greg Kroah-Hartman --- Reading git-format-patch failed