From: Alex Elder <elder@inktank.com>
Date: Thu, 9 Aug 2012 17:33:25 +0000 (-0700)
Subject: rbd: drop needless test in rbd_rq_fn()
X-Git-Tag: v3.12-rc1~1650^2~121
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=38f5f65e9d25b0a7270c337a35c724ca3d56f4d8;p=kernel%2Fkernel-generic.git

rbd: drop needless test in rbd_rq_fn()

There's a test for null rq pointer inside the while loop in
rbd_rq_fn() that's not needed.  That same test already occurred
in the immediatly preceding loop condition test.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
---

diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 43f6ef8..3475bb2 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -1475,10 +1475,6 @@ static void rbd_rq_fn(struct request_queue *q)
 		struct rbd_req_coll *coll;
 		struct ceph_snap_context *snapc;
 
-		/* peek at request from block layer */
-		if (!rq)
-			break;
-
 		dout("fetched request\n");
 
 		/* filter out block requests we don't understand */