rbd: drop a useless local variable
authorAlex Elder <elder@inktank.com>
Tue, 3 Jul 2012 21:01:19 +0000 (16:01 -0500)
committerSage Weil <sage@inktank.com>
Mon, 30 Jul 2012 16:29:56 +0000 (09:29 -0700)
In rbd_req_sync_notify_ack(), a local variable was needlessly being
used to hold a null pointer.  Just pass NULL instead.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
drivers/block/rbd.c

index 8f428a8ab003d8c7a029036eea6a1666a96d7dd3..2ae3bb0c0a340f2d3fbb19fbcdc892306008e2ce 100644 (file)
@@ -1187,7 +1187,6 @@ static int rbd_req_sync_notify_ack(struct rbd_device *dev,
                                   const char *obj)
 {
        struct ceph_osd_req_op *ops;
-       struct page **pages = NULL;
        int ret;
 
        ret = rbd_create_rw_ops(&ops, 1, CEPH_OSD_OP_NOTIFY_ACK, 0);
@@ -1200,7 +1199,7 @@ static int rbd_req_sync_notify_ack(struct rbd_device *dev,
 
        ret = rbd_do_request(NULL, dev, NULL, CEPH_NOSNAP,
                          obj, 0, 0, NULL,
-                         pages, 0,
+                         NULL, 0,
                          CEPH_OSD_FLAG_READ,
                          ops,
                          1,