diff options
author | Alex Elder <elder@inktank.com> | 2012-07-03 17:01:19 -0400 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2012-07-30 12:29:56 -0400 |
commit | ad4f232f28e8059fa1de51f3127d8a6a2759ef16 (patch) | |
tree | 6821548fb6f8d576f9edd7b60e0c498827afda0a /drivers/block | |
parent | c61a1abd215c1ccd6fa73104c79e79987ed3aa98 (diff) |
rbd: drop a useless local variable
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>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/rbd.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 8f428a8ab003..2ae3bb0c0a34 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c | |||
@@ -1187,7 +1187,6 @@ static int rbd_req_sync_notify_ack(struct rbd_device *dev, | |||
1187 | const char *obj) | 1187 | const char *obj) |
1188 | { | 1188 | { |
1189 | struct ceph_osd_req_op *ops; | 1189 | struct ceph_osd_req_op *ops; |
1190 | struct page **pages = NULL; | ||
1191 | int ret; | 1190 | int ret; |
1192 | 1191 | ||
1193 | ret = rbd_create_rw_ops(&ops, 1, CEPH_OSD_OP_NOTIFY_ACK, 0); | 1192 | 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, | |||
1200 | 1199 | ||
1201 | ret = rbd_do_request(NULL, dev, NULL, CEPH_NOSNAP, | 1200 | ret = rbd_do_request(NULL, dev, NULL, CEPH_NOSNAP, |
1202 | obj, 0, 0, NULL, | 1201 | obj, 0, 0, NULL, |
1203 | pages, 0, | 1202 | NULL, 0, |
1204 | CEPH_OSD_FLAG_READ, | 1203 | CEPH_OSD_FLAG_READ, |
1205 | ops, | 1204 | ops, |
1206 | 1, | 1205 | 1, |