Commit 8ac0d9a8 authored by Jens Axboe's avatar Jens Axboe

elevator: allow name aliases

Since we now lookup elevator types with the appropriate multiqueue
capability, allow schedulers to register with an alias alongside
the real name. This is in preparation for allowing 'mq-deadline'
to register an alias of 'deadline' as well.
Reviewed-by: default avatarOmar Sandoval <osandov@fb.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 2527d997
...@@ -83,6 +83,16 @@ bool elv_bio_merge_ok(struct request *rq, struct bio *bio) ...@@ -83,6 +83,16 @@ bool elv_bio_merge_ok(struct request *rq, struct bio *bio)
} }
EXPORT_SYMBOL(elv_bio_merge_ok); EXPORT_SYMBOL(elv_bio_merge_ok);
static bool elevator_match(const struct elevator_type *e, const char *name)
{
if (!strcmp(e->elevator_name, name))
return true;
if (e->elevator_alias && !strcmp(e->elevator_alias, name))
return true;
return false;
}
/* /*
* Return scheduler with name 'name' and with matching 'mq capability * Return scheduler with name 'name' and with matching 'mq capability
*/ */
...@@ -91,7 +101,7 @@ static struct elevator_type *elevator_find(const char *name, bool mq) ...@@ -91,7 +101,7 @@ static struct elevator_type *elevator_find(const char *name, bool mq)
struct elevator_type *e; struct elevator_type *e;
list_for_each_entry(e, &elv_list, list) { list_for_each_entry(e, &elv_list, list) {
if (!strcmp(e->elevator_name, name) && (mq == e->uses_mq)) if (elevator_match(e, name) && (mq == e->uses_mq))
return e; return e;
} }
...@@ -922,9 +932,9 @@ int elv_register(struct elevator_type *e) ...@@ -922,9 +932,9 @@ int elv_register(struct elevator_type *e)
spin_unlock(&elv_list_lock); spin_unlock(&elv_list_lock);
/* print pretty message */ /* print pretty message */
if (!strcmp(e->elevator_name, chosen_elevator) || if (elevator_match(e, chosen_elevator) ||
(!*chosen_elevator && (!*chosen_elevator &&
!strcmp(e->elevator_name, CONFIG_DEFAULT_IOSCHED))) elevator_match(e, CONFIG_DEFAULT_IOSCHED)))
def = " (default)"; def = " (default)";
printk(KERN_INFO "io scheduler %s registered%s\n", e->elevator_name, printk(KERN_INFO "io scheduler %s registered%s\n", e->elevator_name,
...@@ -1077,8 +1087,7 @@ static int __elevator_change(struct request_queue *q, const char *name) ...@@ -1077,8 +1087,7 @@ static int __elevator_change(struct request_queue *q, const char *name)
if (!e) if (!e)
return -EINVAL; return -EINVAL;
if (q->elevator && if (q->elevator && elevator_match(q->elevator->type, elevator_name)) {
!strcmp(elevator_name, q->elevator->type->elevator_name)) {
elevator_put(e); elevator_put(e);
return 0; return 0;
} }
...@@ -1114,6 +1123,7 @@ ssize_t elv_iosched_show(struct request_queue *q, char *name) ...@@ -1114,6 +1123,7 @@ ssize_t elv_iosched_show(struct request_queue *q, char *name)
struct elevator_queue *e = q->elevator; struct elevator_queue *e = q->elevator;
struct elevator_type *elv = NULL; struct elevator_type *elv = NULL;
struct elevator_type *__e; struct elevator_type *__e;
bool uses_mq = q->mq_ops != NULL;
int len = 0; int len = 0;
if (!queue_is_rq_based(q)) if (!queue_is_rq_based(q))
...@@ -1126,7 +1136,8 @@ ssize_t elv_iosched_show(struct request_queue *q, char *name) ...@@ -1126,7 +1136,8 @@ ssize_t elv_iosched_show(struct request_queue *q, char *name)
spin_lock(&elv_list_lock); spin_lock(&elv_list_lock);
list_for_each_entry(__e, &elv_list, list) { list_for_each_entry(__e, &elv_list, list) {
if (elv && !strcmp(elv->elevator_name, __e->elevator_name)) { if (elv && elevator_match(elv, __e->elevator_name) &&
(__e->uses_mq == uses_mq)) {
len += sprintf(name+len, "[%s] ", elv->elevator_name); len += sprintf(name+len, "[%s] ", elv->elevator_name);
continue; continue;
} }
......
...@@ -144,6 +144,7 @@ struct elevator_type ...@@ -144,6 +144,7 @@ struct elevator_type
size_t icq_align; /* ditto */ size_t icq_align; /* ditto */
struct elv_fs_entry *elevator_attrs; struct elv_fs_entry *elevator_attrs;
char elevator_name[ELV_NAME_MAX]; char elevator_name[ELV_NAME_MAX];
const char *elevator_alias;
struct module *elevator_owner; struct module *elevator_owner;
bool uses_mq; bool uses_mq;
#ifdef CONFIG_BLK_DEBUG_FS #ifdef CONFIG_BLK_DEBUG_FS
......
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