Commit ce4c3e19 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Jens Axboe

bcache: Replace bch_read_string_list() by __sysfs_match_string()

Kernel library has a common function to match user input from sysfs
against an array of strings. Thus, replace bch_read_string_list() by
__sysfs_match_string().
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarColy Li <colyli@suse.de>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent ecb37ce9
...@@ -18,7 +18,6 @@ ...@@ -18,7 +18,6 @@
/* Default is -1; we skip past it for struct cached_dev's cache mode */ /* Default is -1; we skip past it for struct cached_dev's cache mode */
static const char * const bch_cache_modes[] = { static const char * const bch_cache_modes[] = {
"default",
"writethrough", "writethrough",
"writeback", "writeback",
"writearound", "writearound",
...@@ -28,7 +27,6 @@ static const char * const bch_cache_modes[] = { ...@@ -28,7 +27,6 @@ static const char * const bch_cache_modes[] = {
/* Default is -1; we skip past it for stop_when_cache_set_failed */ /* Default is -1; we skip past it for stop_when_cache_set_failed */
static const char * const bch_stop_on_failure_modes[] = { static const char * const bch_stop_on_failure_modes[] = {
"default",
"auto", "auto",
"always", "always",
NULL NULL
...@@ -146,27 +144,6 @@ static ssize_t bch_snprint_string_list(char *buf, size_t size, const char * cons ...@@ -146,27 +144,6 @@ static ssize_t bch_snprint_string_list(char *buf, size_t size, const char * cons
return out - buf; return out - buf;
} }
static ssize_t bch_read_string_list(const char *buf, const char * const list[])
{
size_t i;
char *s, *d = kstrndup(buf, PAGE_SIZE - 1, GFP_KERNEL);
if (!d)
return -ENOMEM;
s = strim(d);
for (i = 0; list[i]; i++)
if (!strcmp(list[i], s))
break;
kfree(d);
if (!list[i])
return -EINVAL;
return i;
}
SHOW(__bch_cached_dev) SHOW(__bch_cached_dev)
{ {
struct cached_dev *dc = container_of(kobj, struct cached_dev, struct cached_dev *dc = container_of(kobj, struct cached_dev,
...@@ -177,12 +154,12 @@ SHOW(__bch_cached_dev) ...@@ -177,12 +154,12 @@ SHOW(__bch_cached_dev)
if (attr == &sysfs_cache_mode) if (attr == &sysfs_cache_mode)
return bch_snprint_string_list(buf, PAGE_SIZE, return bch_snprint_string_list(buf, PAGE_SIZE,
bch_cache_modes + 1, bch_cache_modes,
BDEV_CACHE_MODE(&dc->sb)); BDEV_CACHE_MODE(&dc->sb));
if (attr == &sysfs_stop_when_cache_set_failed) if (attr == &sysfs_stop_when_cache_set_failed)
return bch_snprint_string_list(buf, PAGE_SIZE, return bch_snprint_string_list(buf, PAGE_SIZE,
bch_stop_on_failure_modes + 1, bch_stop_on_failure_modes,
dc->stop_when_cache_set_failed); dc->stop_when_cache_set_failed);
...@@ -306,8 +283,7 @@ STORE(__cached_dev) ...@@ -306,8 +283,7 @@ STORE(__cached_dev)
bch_cached_dev_run(dc); bch_cached_dev_run(dc);
if (attr == &sysfs_cache_mode) { if (attr == &sysfs_cache_mode) {
v = bch_read_string_list(buf, bch_cache_modes + 1); v = __sysfs_match_string(bch_cache_modes, -1, buf);
if (v < 0) if (v < 0)
return v; return v;
...@@ -318,8 +294,7 @@ STORE(__cached_dev) ...@@ -318,8 +294,7 @@ STORE(__cached_dev)
} }
if (attr == &sysfs_stop_when_cache_set_failed) { if (attr == &sysfs_stop_when_cache_set_failed) {
v = bch_read_string_list(buf, bch_stop_on_failure_modes + 1); v = __sysfs_match_string(bch_stop_on_failure_modes, -1, buf);
if (v < 0) if (v < 0)
return v; return v;
...@@ -688,6 +663,7 @@ SHOW_LOCKED(bch_cache_set) ...@@ -688,6 +663,7 @@ SHOW_LOCKED(bch_cache_set)
STORE(__bch_cache_set) STORE(__bch_cache_set)
{ {
struct cache_set *c = container_of(kobj, struct cache_set, kobj); struct cache_set *c = container_of(kobj, struct cache_set, kobj);
ssize_t v;
if (attr == &sysfs_unregister) if (attr == &sysfs_unregister)
bch_cache_set_unregister(c); bch_cache_set_unregister(c);
...@@ -751,8 +727,7 @@ STORE(__bch_cache_set) ...@@ -751,8 +727,7 @@ STORE(__bch_cache_set)
c->congested_write_threshold_us); c->congested_write_threshold_us);
if (attr == &sysfs_errors) { if (attr == &sysfs_errors) {
ssize_t v = bch_read_string_list(buf, error_actions); v = __sysfs_match_string(error_actions, -1, buf);
if (v < 0) if (v < 0)
return v; return v;
...@@ -767,8 +742,7 @@ STORE(__bch_cache_set) ...@@ -767,8 +742,7 @@ STORE(__bch_cache_set)
c->error_decay = strtoul_or_return(buf) / 88; c->error_decay = strtoul_or_return(buf) / 88;
if (attr == &sysfs_io_disable) { if (attr == &sysfs_io_disable) {
int v = strtoul_or_return(buf); v = strtoul_or_return(buf);
if (v) { if (v) {
if (test_and_set_bit(CACHE_SET_IO_DISABLE, if (test_and_set_bit(CACHE_SET_IO_DISABLE,
&c->flags)) &c->flags))
...@@ -982,6 +956,7 @@ SHOW_LOCKED(bch_cache) ...@@ -982,6 +956,7 @@ SHOW_LOCKED(bch_cache)
STORE(__bch_cache) STORE(__bch_cache)
{ {
struct cache *ca = container_of(kobj, struct cache, kobj); struct cache *ca = container_of(kobj, struct cache, kobj);
ssize_t v;
if (attr == &sysfs_discard) { if (attr == &sysfs_discard) {
bool v = strtoul_or_return(buf); bool v = strtoul_or_return(buf);
...@@ -996,8 +971,7 @@ STORE(__bch_cache) ...@@ -996,8 +971,7 @@ STORE(__bch_cache)
} }
if (attr == &sysfs_cache_replacement_policy) { if (attr == &sysfs_cache_replacement_policy) {
ssize_t v = bch_read_string_list(buf, cache_replacement_policies); v = __sysfs_match_string(cache_replacement_policies, -1, buf);
if (v < 0) if (v < 0)
return v; return v;
......
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