cryptospore / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone

Blame SOURCES/kvm-block-rbd-update-variable-names-to-more-apt-names.patch

76daa3
From e62afefee293b3188ae9b633f5610002eb709d41 Mon Sep 17 00:00:00 2001
76daa3
From: Jeffrey Cody <jcody@redhat.com>
76daa3
Date: Wed, 26 Apr 2017 20:08:54 +0200
76daa3
Subject: [PATCH 08/11] block/rbd - update variable names to more apt names
76daa3
76daa3
RH-Author: Jeffrey Cody <jcody@redhat.com>
76daa3
Message-id: <0a980f47dfd0a02181a5e9fa84f34ed44d9971fe.1493236824.git.jcody@redhat.com>
76daa3
Patchwork-id: 74903
76daa3
O-Subject: [RHEV-7.4 qemu-kvm-rhev 7/8] block/rbd - update variable names to more apt names
76daa3
Bugzilla: 1189998
76daa3
RH-Acked-by: John Snow <jsnow@redhat.com>
76daa3
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
76daa3
RH-Acked-by: Max Reitz <mreitz@redhat.com>
76daa3
76daa3
Update 'clientname' to be 'user', which tracks better with both
76daa3
the QAPI and rados variable naming.
76daa3
76daa3
Update 'name' to be 'image_name', as it indicates the rbd image.
76daa3
Naming it 'image' would have been ideal, but we are using that for
76daa3
the rados_image_t value returned by rbd_open().
76daa3
76daa3
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
76daa3
Signed-off-by: Jeff Cody <jcody@redhat.com>
76daa3
Reviewed-by: John Snow <jsnow@redhat.com>
76daa3
Message-id: b7ec1fb2e1cf36f9b6911631447a5b0422590b7d.1491597120.git.jcody@redhat.com
76daa3
(cherry picked from commit 80b61a27c69c67492ce87ddd63ee497b5bc00e7e)
76daa3
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
---
76daa3
 block/rbd.c | 33 +++++++++++++++++----------------
76daa3
 1 file changed, 17 insertions(+), 16 deletions(-)
76daa3
76daa3
diff --git a/block/rbd.c b/block/rbd.c
76daa3
index 1c43171..35853c9 100644
76daa3
--- a/block/rbd.c
76daa3
+++ b/block/rbd.c
76daa3
@@ -94,7 +94,7 @@ typedef struct BDRVRBDState {
76daa3
     rados_t cluster;
76daa3
     rados_ioctx_t io_ctx;
76daa3
     rbd_image_t image;
76daa3
-    char *name;
76daa3
+    char *image_name;
76daa3
     char *snap;
76daa3
 } BDRVRBDState;
76daa3
 
76daa3
@@ -350,7 +350,7 @@ static int qemu_rbd_create(const char *filename, QemuOpts *opts, Error **errp)
76daa3
     int64_t bytes = 0;
76daa3
     int64_t objsize;
76daa3
     int obj_order = 0;
76daa3
-    const char *pool, *name, *conf, *clientname, *keypairs;
76daa3
+    const char *pool, *image_name, *conf, *user, *keypairs;
76daa3
     const char *secretid;
76daa3
     rados_t cluster;
76daa3
     rados_ioctx_t io_ctx;
76daa3
@@ -393,11 +393,11 @@ static int qemu_rbd_create(const char *filename, QemuOpts *opts, Error **errp)
76daa3
      */
76daa3
     pool       = qdict_get_try_str(options, "pool");
76daa3
     conf       = qdict_get_try_str(options, "conf");
76daa3
-    clientname = qdict_get_try_str(options, "user");
76daa3
-    name       = qdict_get_try_str(options, "image");
76daa3
+    user       = qdict_get_try_str(options, "user");
76daa3
+    image_name = qdict_get_try_str(options, "image");
76daa3
     keypairs   = qdict_get_try_str(options, "=keyvalue-pairs");
76daa3
 
76daa3
-    ret = rados_create(&cluster, clientname);
76daa3
+    ret = rados_create(&cluster, user);
76daa3
     if (ret < 0) {
76daa3
         error_setg_errno(errp, -ret, "error initializing");
76daa3
         goto exit;
76daa3
@@ -434,7 +434,7 @@ static int qemu_rbd_create(const char *filename, QemuOpts *opts, Error **errp)
76daa3
         goto shutdown;
76daa3
     }
76daa3
 
76daa3
-    ret = rbd_create(io_ctx, name, bytes, &obj_order);
76daa3
+    ret = rbd_create(io_ctx, image_name, bytes, &obj_order);
76daa3
     if (ret < 0) {
76daa3
         error_setg_errno(errp, -ret, "error rbd create");
76daa3
     }
76daa3
@@ -540,7 +540,7 @@ static int qemu_rbd_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
                          Error **errp)
76daa3
 {
76daa3
     BDRVRBDState *s = bs->opaque;
76daa3
-    const char *pool, *snap, *conf, *clientname, *name, *keypairs;
76daa3
+    const char *pool, *snap, *conf, *user, *image_name, *keypairs;
76daa3
     const char *secretid;
76daa3
     QemuOpts *opts;
76daa3
     Error *local_err = NULL;
76daa3
@@ -567,24 +567,24 @@ static int qemu_rbd_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
     pool           = qemu_opt_get(opts, "pool");
76daa3
     conf           = qemu_opt_get(opts, "conf");
76daa3
     snap           = qemu_opt_get(opts, "snapshot");
76daa3
-    clientname     = qemu_opt_get(opts, "user");
76daa3
-    name           = qemu_opt_get(opts, "image");
76daa3
+    user           = qemu_opt_get(opts, "user");
76daa3
+    image_name     = qemu_opt_get(opts, "image");
76daa3
     keypairs       = qemu_opt_get(opts, "=keyvalue-pairs");
76daa3
 
76daa3
-    if (!pool || !name) {
76daa3
+    if (!pool || !image_name) {
76daa3
         error_setg(errp, "Parameters 'pool' and 'image' are required");
76daa3
         r = -EINVAL;
76daa3
         goto failed_opts;
76daa3
     }
76daa3
 
76daa3
-    r = rados_create(&s->cluster, clientname);
76daa3
+    r = rados_create(&s->cluster, user);
76daa3
     if (r < 0) {
76daa3
         error_setg_errno(errp, -r, "error initializing");
76daa3
         goto failed_opts;
76daa3
     }
76daa3
 
76daa3
     s->snap = g_strdup(snap);
76daa3
-    s->name = g_strdup(name);
76daa3
+    s->image_name = g_strdup(image_name);
76daa3
 
76daa3
     /* try default location when conf=NULL, but ignore failure */
76daa3
     r = rados_conf_read_file(s->cluster, conf);
76daa3
@@ -636,9 +636,10 @@ static int qemu_rbd_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
     }
76daa3
 
76daa3
     /* rbd_open is always r/w */
76daa3
-    r = rbd_open(s->io_ctx, s->name, &s->image, s->snap);
76daa3
+    r = rbd_open(s->io_ctx, s->image_name, &s->image, s->snap);
76daa3
     if (r < 0) {
76daa3
-        error_setg_errno(errp, -r, "error reading header from %s", s->name);
76daa3
+        error_setg_errno(errp, -r, "error reading header from %s",
76daa3
+                         s->image_name);
76daa3
         goto failed_open;
76daa3
     }
76daa3
 
76daa3
@@ -660,7 +661,7 @@ failed_open:
76daa3
 failed_shutdown:
76daa3
     rados_shutdown(s->cluster);
76daa3
     g_free(s->snap);
76daa3
-    g_free(s->name);
76daa3
+    g_free(s->image_name);
76daa3
 failed_opts:
76daa3
     qemu_opts_del(opts);
76daa3
     g_free(mon_host);
76daa3
@@ -674,7 +675,7 @@ static void qemu_rbd_close(BlockDriverState *bs)
76daa3
     rbd_close(s->image);
76daa3
     rados_ioctx_destroy(s->io_ctx);
76daa3
     g_free(s->snap);
76daa3
-    g_free(s->name);
76daa3
+    g_free(s->image_name);
76daa3
     rados_shutdown(s->cluster);
76daa3
 }
76daa3
 
76daa3
-- 
76daa3
1.8.3.1
76daa3