diff options
| author | Vasily Tarasov <vtaras@openvz.org> | 2006-10-11 09:24:27 +0200 |
|---|---|---|
| committer | Jens Axboe <axboe@nelson.home.kernel.dk> | 2006-10-12 15:08:51 +0200 |
| commit | a22b169df1b9f259391cf3b8ad8bfeea3d7be3f1 (patch) | |
| tree | c03c0fb3de87afc0c67a4a973a826deed4bb9d20 | |
| parent | [PATCH] elevator: elevator_type member not used (diff) | |
| download | linux-a22b169df1b9f259391cf3b8ad8bfeea3d7be3f1.tar.xz linux-a22b169df1b9f259391cf3b8ad8bfeea3d7be3f1.zip | |
[PATCH] block layer: elevator_find function cleanup
We can easily produce search through the elevator list
without introducing additional elevator_type variable.
Signed-off-by: Vasily Tarasov <vtaras@openvz.org>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
| -rw-r--r-- | block/elevator.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/block/elevator.c b/block/elevator.c index 487dd3da8853..d8030a84773a 100644 --- a/block/elevator.c +++ b/block/elevator.c @@ -93,21 +93,18 @@ static inline int elv_try_merge(struct request *__rq, struct bio *bio) static struct elevator_type *elevator_find(const char *name) { - struct elevator_type *e = NULL; + struct elevator_type *e; struct list_head *entry; list_for_each(entry, &elv_list) { - struct elevator_type *__e; - __e = list_entry(entry, struct elevator_type, list); + e = list_entry(entry, struct elevator_type, list); - if (!strcmp(__e->elevator_name, name)) { - e = __e; - break; - } + if (!strcmp(e->elevator_name, name)) + return e; } - return e; + return NULL; } static void elevator_put(struct elevator_type *e) |