yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
76daa3
From 1ffa7d12a15fdc4f1bbed59bf5d3292ac2b574b3 Mon Sep 17 00:00:00 2001
76daa3
From: Jeffrey Cody <jcody@redhat.com>
76daa3
Date: Mon, 12 Jun 2017 14:46:56 +0200
76daa3
Subject: [PATCH 11/13] block: fix external snapshot abort permission error
76daa3
76daa3
RH-Author: Jeffrey Cody <jcody@redhat.com>
76daa3
Message-id: <14e6e82657b93fb6bec16eddce051ced36db5ff9.1497278737.git.jcody@redhat.com>
76daa3
Patchwork-id: 75593
76daa3
O-Subject: [RHEV-7.4 qemu-kvm-rhev PATCH 1/1] block: fix external snapshot abort permission error
76daa3
Bugzilla: 1447184
76daa3
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
76daa3
RH-Acked-by: Max Reitz <mreitz@redhat.com>
76daa3
RH-Acked-by: John Snow <jsnow@redhat.com>
76daa3
76daa3
In external_snapshot_abort(), we try to undo what was done in
76daa3
external_snapshot_prepare() calling bdrv_replace_node() to swap the
76daa3
nodes back.  However, we receive a permissions error as writers are
76daa3
blocked on the old node, which is now the new node backing file.
76daa3
76daa3
An easy fix (initially suggested by Kevin Wolf) is to call
76daa3
bdrv_set_backing_hd() on the new node, to set the backing node to NULL.
76daa3
76daa3
Signed-off-by: Jeff Cody <jcody@redhat.com>
76daa3
Reviewed-by: Eric Blake <eblake@redhat.com>
76daa3
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
76daa3
(cherry picked from commit 719fc28c80a22ab9f1533d775bae09c14442bbbe)
76daa3
Signed-off-by: Jeff Cody <jcody@redhat.com>
76daa3
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
---
76daa3
 blockdev.c | 4 ++++
76daa3
 1 file changed, 4 insertions(+)
76daa3
76daa3
diff --git a/blockdev.c b/blockdev.c
76daa3
index 4432143..a3a0731 100644
76daa3
--- a/blockdev.c
76daa3
+++ b/blockdev.c
76daa3
@@ -1847,7 +1847,11 @@ static void external_snapshot_abort(BlkActionState *common)
76daa3
                              DO_UPCAST(ExternalSnapshotState, common, common);
76daa3
     if (state->new_bs) {
76daa3
         if (state->overlay_appended) {
76daa3
+            bdrv_ref(state->old_bs);   /* we can't let bdrv_set_backind_hd()
76daa3
+                                          close state->old_bs; we need it */
76daa3
+            bdrv_set_backing_hd(state->new_bs, NULL, &error_abort);
76daa3
             bdrv_replace_node(state->new_bs, state->old_bs, &error_abort);
76daa3
+            bdrv_unref(state->old_bs); /* bdrv_replace_node() ref'ed old_bs */
76daa3
         }
76daa3
     }
76daa3
 }
76daa3
-- 
76daa3
1.8.3.1
76daa3