summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorAlex Elder <elder@inktank.com>2012-08-30 14:42:15 -0500
committerAlex Elder <elder@inktank.com>2012-10-01 14:30:51 -0500
commit8836b995fd192dba23d312d2a4fba68dd8ca7183 (patch)
treea04165a494e9327bf052b7ed25f94ba86a28e057 /drivers
parent4e1105a299adf7ac421d42a8be05205f51610f3c (diff)
downloadop-kernel-dev-8836b995fd192dba23d312d2a4fba68dd8ca7183.zip
op-kernel-dev-8836b995fd192dba23d312d2a4fba68dd8ca7183.tar.gz
rbd: simplify snap_by_name() interface
There is only one caller of snap_by_name(), and it passes two values to be assigned, both of which are found within an rbd device structure. Change the interface so it just passes the address of the rbd_dev, and make the assignments to its fields directly. Signed-off-by: Alex Elder <elder@inktank.com> Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/block/rbd.c15
1 files changed, 6 insertions, 9 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 23fa962..46b8f8e 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -621,10 +621,10 @@ out_err:
return -ENOMEM;
}
-static int snap_by_name(struct rbd_image_header *header, const char *snap_name,
- u64 *seq, u64 *size)
+static int snap_by_name(struct rbd_device *rbd_dev, const char *snap_name)
{
int i;
+ struct rbd_image_header *header = &rbd_dev->header;
char *p = header->snap_names;
rbd_assert(header->snapc != NULL);
@@ -633,10 +633,9 @@ static int snap_by_name(struct rbd_image_header *header, const char *snap_name,
/* Found it. Pass back its id and/or size */
- if (seq)
- *seq = header->snapc->snaps[i];
- if (size)
- *size = header->snap_sizes[i];
+ rbd_dev->mapping.snap_id = header->snapc->snaps[i];
+ rbd_dev->mapping.size = header->snap_sizes[i];
+
return i;
}
p += strlen(p) + 1; /* Skip ahead to the next name */
@@ -657,9 +656,7 @@ static int rbd_header_set_snap(struct rbd_device *rbd_dev, char *snap_name)
rbd_dev->mapping.snap_exists = false;
rbd_dev->mapping.read_only = rbd_dev->rbd_opts.read_only;
} else {
- ret = snap_by_name(&rbd_dev->header, snap_name,
- &rbd_dev->mapping.snap_id,
- &rbd_dev->mapping.size);
+ ret = snap_by_name(rbd_dev, snap_name);
if (ret < 0)
goto done;
rbd_dev->mapping.snap_exists = true;
OpenPOWER on IntegriCloud