rbd: simplify snap_by_name() interface
authorAlex Elder <elder@inktank.com>
Thu, 30 Aug 2012 19:42:15 +0000 (14:42 -0500)
committerAlex Elder <elder@inktank.com>
Mon, 1 Oct 2012 19:30:51 +0000 (14:30 -0500)
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>
drivers/block/rbd.c

index 23fa962fea36f60c857c7f4ea8938132f29e749b..46b8f8e536be9d8d4bb4cd8428dd5fac5a1a2f2d 100644 (file)
@@ -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;