diff options
author | Josh Durgin <josh.durgin@dreamhost.com> | 2011-11-21 21:16:52 -0500 |
---|---|---|
committer | Alex Elder <elder@dreamhost.com> | 2012-05-14 13:12:58 -0400 |
commit | b06e6a6be796bc365a19b0ac5176b553c13abf2f (patch) | |
tree | 5a53dceb7eed8932d097ceeb2fcbe9665e7e3b48 /drivers/block | |
parent | 77dfe99fe3cb0b2b0545e19e2d57b7a9134ee3c0 (diff) |
rbd: remove conditional snapid parameters
The snapid parameters passed to rbd_do_op() and rbd_req_sync_op()
are now always either a valid snapid or an explicit CEPH_NOSNAP.
[elder@dreamhost.com: Rephrased the description]
Signed-off-by: Josh Durgin <josh.durgin@dreamhost.com>
Reviewed-by: Alex Elder <elder@dreamhost.com>
Reviewed-by: Yehuda Sadeh <yehuda@hq.newdream.net>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/rbd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index c1650bdf2f6e..4a0a829f79d1 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c | |||
@@ -1152,7 +1152,7 @@ static int rbd_req_read(struct request *rq, | |||
1152 | int coll_index) | 1152 | int coll_index) |
1153 | { | 1153 | { |
1154 | return rbd_do_op(rq, rbd_dev, NULL, | 1154 | return rbd_do_op(rq, rbd_dev, NULL, |
1155 | (snapid ? snapid : CEPH_NOSNAP), | 1155 | snapid, |
1156 | CEPH_OSD_OP_READ, | 1156 | CEPH_OSD_OP_READ, |
1157 | CEPH_OSD_FLAG_READ, | 1157 | CEPH_OSD_FLAG_READ, |
1158 | 2, | 1158 | 2, |
@@ -1171,7 +1171,7 @@ static int rbd_req_sync_read(struct rbd_device *dev, | |||
1171 | u64 *ver) | 1171 | u64 *ver) |
1172 | { | 1172 | { |
1173 | return rbd_req_sync_op(dev, NULL, | 1173 | return rbd_req_sync_op(dev, NULL, |
1174 | (snapid ? snapid : CEPH_NOSNAP), | 1174 | snapid, |
1175 | CEPH_OSD_OP_READ, | 1175 | CEPH_OSD_OP_READ, |
1176 | CEPH_OSD_FLAG_READ, | 1176 | CEPH_OSD_FLAG_READ, |
1177 | NULL, | 1177 | NULL, |