yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone

Blame SOURCES/kvm-iscsi-Drop-deprecated-drive-parameter-filename.patch

26ba25
From cd19aec9abcc51c2d0103389c5c4a28649ccaf9f Mon Sep 17 00:00:00 2001
26ba25
From: Markus Armbruster <armbru@redhat.com>
26ba25
Date: Mon, 18 Jun 2018 08:43:14 +0200
26ba25
Subject: [PATCH 016/268] iscsi: Drop deprecated -drive parameter "filename"
26ba25
26ba25
RH-Author: Markus Armbruster <armbru@redhat.com>
26ba25
Message-id: <20180618084330.30009-8-armbru@redhat.com>
26ba25
Patchwork-id: 80740
26ba25
O-Subject: [RHEL-7.6 qemu-kvm-rhev PATCH 07/23] iscsi: Drop deprecated -drive parameter "filename"
26ba25
Bugzilla: 1557995
26ba25
RH-Acked-by: Max Reitz <mreitz@redhat.com>
26ba25
RH-Acked-by: Jeffrey Cody <jcody@redhat.com>
26ba25
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
26ba25
26ba25
Parameter "filename" is deprecated since commit 5c3ad1a6a8f, v2.10.0.
26ba25
Time to get rid of it.
26ba25
26ba25
Signed-off-by: Markus Armbruster <armbru@redhat.com>
26ba25
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
26ba25
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
26ba25
(cherry picked from commit deadbb8ebb5c253da9b8ed02ab51a0fadf60edc7)
26ba25
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
26ba25
---
26ba25
 block/iscsi.c | 23 ++---------------------
26ba25
 1 file changed, 2 insertions(+), 21 deletions(-)
26ba25
26ba25
diff --git a/block/iscsi.c b/block/iscsi.c
26ba25
index d19ae0e..658462b 100644
26ba25
--- a/block/iscsi.c
26ba25
+++ b/block/iscsi.c
26ba25
@@ -1732,10 +1732,6 @@ static QemuOptsList runtime_opts = {
26ba25
             .name = "timeout",
26ba25
             .type = QEMU_OPT_NUMBER,
26ba25
         },
26ba25
-        {
26ba25
-            .name = "filename",
26ba25
-            .type = QEMU_OPT_STRING,
26ba25
-        },
26ba25
         { /* end of list */ }
26ba25
     },
26ba25
 };
26ba25
@@ -1751,27 +1747,12 @@ static int iscsi_open(BlockDriverState *bs, QDict *options, int flags,
26ba25
     char *initiator_name = NULL;
26ba25
     QemuOpts *opts;
26ba25
     Error *local_err = NULL;
26ba25
-    const char *transport_name, *portal, *target, *filename;
26ba25
+    const char *transport_name, *portal, *target;
26ba25
 #if LIBISCSI_API_VERSION >= (20160603)
26ba25
     enum iscsi_transport_type transport;
26ba25
 #endif
26ba25
     int i, ret = 0, timeout = 0, lun;
26ba25
 
26ba25
-    /* If we are given a filename, parse the filename, with precedence given to
26ba25
-     * filename encoded options */
26ba25
-    filename = qdict_get_try_str(options, "filename");
26ba25
-    if (filename) {
26ba25
-        warn_report("'filename' option specified. "
26ba25
-                    "This is an unsupported option, and may be deprecated "
26ba25
-                    "in the future");
26ba25
-        iscsi_parse_filename(filename, options, &local_err);
26ba25
-        if (local_err) {
26ba25
-            ret = -EINVAL;
26ba25
-            error_propagate(errp, local_err);
26ba25
-            goto exit;
26ba25
-        }
26ba25
-    }
26ba25
-
26ba25
     opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
26ba25
     qemu_opts_absorb_qdict(opts, options, &local_err);
26ba25
     if (local_err) {
26ba25
@@ -1989,7 +1970,7 @@ out:
26ba25
         }
26ba25
         memset(iscsilun, 0, sizeof(IscsiLun));
26ba25
     }
26ba25
-exit:
26ba25
+
26ba25
     return ret;
26ba25
 }
26ba25
 
26ba25
-- 
26ba25
1.8.3.1
26ba25