rbd: drop needless test in rbd_rq_fn()
authorAlex Elder <elder@inktank.com>
Thu, 9 Aug 2012 17:33:25 +0000 (10:33 -0700)
committerAlex Elder <elder@inktank.com>
Mon, 1 Oct 2012 19:30:50 +0000 (14:30 -0500)
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>
drivers/block/rbd.c

index 43f6ef8d696f093f3ef28df4456c971a227f32c4..3475bb2e03ab497e289928f4d2bd3a2a46154050 100644 (file)
@@ -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 */