Commit 466fc15c authored by Rickard Strandqvist's avatar Rickard Strandqvist Committed by Greg Kroah-Hartman

staging: lustre: lustre: obdclass: lprocfs_status.c: Tidied up some code in a case statement

Tidied up some code in a case statement after a discussion.

This was partly found using a static code analysis program called cppcheck.
Signed-off-by: default avatarRickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e9a86069
...@@ -45,6 +45,7 @@ ...@@ -45,6 +45,7 @@
#include "../include/lprocfs_status.h" #include "../include/lprocfs_status.h"
#include "../include/lustre/lustre_idl.h" #include "../include/lustre/lustre_idl.h"
#include <linux/seq_file.h> #include <linux/seq_file.h>
#include <linux/ctype.h>
static const char * const obd_connect_names[] = { static const char * const obd_connect_names[] = {
"read_only", "read_only",
...@@ -1896,16 +1897,16 @@ int lprocfs_write_frac_u64_helper(const char *buffer, unsigned long count, ...@@ -1896,16 +1897,16 @@ int lprocfs_write_frac_u64_helper(const char *buffer, unsigned long count,
} }
units = 1; units = 1;
switch (*end) { switch (tolower(*end)) {
case 'p': case 'P': case 'p':
units <<= 10; units <<= 10;
case 't': case 'T': case 't':
units <<= 10; units <<= 10;
case 'g': case 'G': case 'g':
units <<= 10; units <<= 10;
case 'm': case 'M': case 'm':
units <<= 10; units <<= 10;
case 'k': case 'K': case 'k':
units <<= 10; units <<= 10;
} }
/* Specified units override the multiplier */ /* Specified units override the multiplier */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment