From: Jens Axboe <axboe@fb.com>
Date: Thu, 19 Jan 2017 14:10:13 +0000 (-0700)
Subject: elevator: fix unnecessary put of elevator in failure case
X-Git-Tag: v4.11-rc1~142^2~1^2~61
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=610d886c0c22fa7504e817b6d03c402de64b0264;p=platform%2Fkernel%2Flinux-exynos.git

elevator: fix unnecessary put of elevator in failure case

We already checked that e is NULL, so no point in calling
elevator_put() to free it.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: dc877dbd088f ("blk-mq-sched: add framework for MQ capable IO schedulers")
Signed-off-by: Jens Axboe <axboe@fb.com>
---

diff --git a/block/elevator.c b/block/elevator.c
index bcba2dd..ef7f594 100644
--- a/block/elevator.c
+++ b/block/elevator.c
@@ -230,10 +230,6 @@ int elevator_init(struct request_queue *q, char *name)
 			printk(KERN_ERR
 				"Default I/O scheduler not found. " \
 				"Using noop/none.\n");
-			if (q->mq_ops) {
-				elevator_put(e);
-				return 0;
-			}
 			e = elevator_get("noop", false);
 		}
 	}