76daa3
From e9c5d55aa2132484144d231261d128f57e7ec9dd Mon Sep 17 00:00:00 2001
76daa3
From: Jeffrey Cody <jcody@redhat.com>
76daa3
Date: Wed, 26 Apr 2017 20:08:48 +0200
76daa3
Subject: [PATCH 02/11] block: add bdrv_set_read_only() helper function
76daa3
76daa3
RH-Author: Jeffrey Cody <jcody@redhat.com>
76daa3
Message-id: <90cc08632a1a1a4b57830ba63f430506952dd49c.1493236824.git.jcody@redhat.com>
76daa3
Patchwork-id: 74895
76daa3
O-Subject: [RHEV-7.4 qemu-kvm-rhev 1/8] block: add bdrv_set_read_only() helper function
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
We have a helper wrapper for checking for the BDS read_only flag,
76daa3
add a helper wrapper to set the read_only flag as well.
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: 9b18972d05f5fa2ac16c014f0af98d680553048d.1491597120.git.jcody@redhat.com
76daa3
(cherry picked from commit fe5241bfe3fb61ec3f589ceacd91c1469bfd400f)
76daa3
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
---
76daa3
 block.c               | 5 +++++
76daa3
 block/bochs.c         | 2 +-
76daa3
 block/cloop.c         | 2 +-
76daa3
 block/dmg.c           | 2 +-
76daa3
 block/rbd.c           | 2 +-
76daa3
 block/vvfat.c         | 4 ++--
76daa3
 include/block/block.h | 1 +
76daa3
 7 files changed, 12 insertions(+), 6 deletions(-)
76daa3
76daa3
diff --git a/block.c b/block.c
76daa3
index 1fbbb8d..25a4cfd 100644
76daa3
--- a/block.c
76daa3
+++ b/block.c
76daa3
@@ -192,6 +192,11 @@ void path_combine(char *dest, int dest_size,
76daa3
     }
76daa3
 }
76daa3
 
76daa3
+void bdrv_set_read_only(BlockDriverState *bs, bool read_only)
76daa3
+{
76daa3
+    bs->read_only = read_only;
76daa3
+}
76daa3
+
76daa3
 void bdrv_get_full_backing_filename_from_filename(const char *backed,
76daa3
                                                   const char *backing,
76daa3
                                                   char *dest, size_t sz,
76daa3
diff --git a/block/bochs.c b/block/bochs.c
76daa3
index 516da56..bdc2831 100644
76daa3
--- a/block/bochs.c
76daa3
+++ b/block/bochs.c
76daa3
@@ -110,7 +110,7 @@ static int bochs_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
         return -EINVAL;
76daa3
     }
76daa3
 
76daa3
-    bs->read_only = true; /* no write support yet */
76daa3
+    bdrv_set_read_only(bs, true); /* no write support yet */
76daa3
 
76daa3
     ret = bdrv_pread(bs->file, 0, &bochs, sizeof(bochs));
76daa3
     if (ret < 0) {
76daa3
diff --git a/block/cloop.c b/block/cloop.c
76daa3
index a6c7b9d..11f17c8 100644
76daa3
--- a/block/cloop.c
76daa3
+++ b/block/cloop.c
76daa3
@@ -72,7 +72,7 @@ static int cloop_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
         return -EINVAL;
76daa3
     }
76daa3
 
76daa3
-    bs->read_only = true;
76daa3
+    bdrv_set_read_only(bs, true);
76daa3
 
76daa3
     /* read header */
76daa3
     ret = bdrv_pread(bs->file, 128, &s->block_size, 4);
76daa3
diff --git a/block/dmg.c b/block/dmg.c
76daa3
index a7d25fc..27ce4a6 100644
76daa3
--- a/block/dmg.c
76daa3
+++ b/block/dmg.c
76daa3
@@ -420,7 +420,7 @@ static int dmg_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
     }
76daa3
 
76daa3
     block_module_load_one("dmg-bz2");
76daa3
-    bs->read_only = true;
76daa3
+    bdrv_set_read_only(bs, true);
76daa3
 
76daa3
     s->n_chunks = 0;
76daa3
     s->offsets = s->lengths = s->sectors = s->sectorcounts = NULL;
76daa3
diff --git a/block/rbd.c b/block/rbd.c
76daa3
index 1ceeeb5..6ad2904 100644
76daa3
--- a/block/rbd.c
76daa3
+++ b/block/rbd.c
76daa3
@@ -641,7 +641,7 @@ static int qemu_rbd_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
         goto failed_open;
76daa3
     }
76daa3
 
76daa3
-    bs->read_only = (s->snap != NULL);
76daa3
+    bdrv_set_read_only(bs, (s->snap != NULL));
76daa3
 
76daa3
     qemu_opts_del(opts);
76daa3
     return 0;
76daa3
diff --git a/block/vvfat.c b/block/vvfat.c
76daa3
index af5153d..d4ce6d7 100644
76daa3
--- a/block/vvfat.c
76daa3
+++ b/block/vvfat.c
76daa3
@@ -1157,7 +1157,7 @@ static int vvfat_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
     s->current_cluster=0xffffffff;
76daa3
 
76daa3
     /* read only is the default for safety */
76daa3
-    bs->read_only = true;
76daa3
+    bdrv_set_read_only(bs, true);
76daa3
     s->qcow = NULL;
76daa3
     s->qcow_filename = NULL;
76daa3
     s->fat2 = NULL;
76daa3
@@ -1173,7 +1173,7 @@ static int vvfat_open(BlockDriverState *bs, QDict *options, int flags,
76daa3
         if (ret < 0) {
76daa3
             goto fail;
76daa3
         }
76daa3
-        bs->read_only = false;
76daa3
+        bdrv_set_read_only(bs, false);
76daa3
     }
76daa3
 
76daa3
     bs->total_sectors = cyls * heads * secs;
76daa3
diff --git a/include/block/block.h b/include/block/block.h
76daa3
index 5ddc0cf..16eb909 100644
76daa3
--- a/include/block/block.h
76daa3
+++ b/include/block/block.h
76daa3
@@ -434,6 +434,7 @@ int bdrv_is_allocated_above(BlockDriverState *top, BlockDriverState *base,
76daa3
                             int64_t sector_num, int nb_sectors, int *pnum);
76daa3
 
76daa3
 bool bdrv_is_read_only(BlockDriverState *bs);
76daa3
+void bdrv_set_read_only(BlockDriverState *bs, bool read_only);
76daa3
 bool bdrv_is_sg(BlockDriverState *bs);
76daa3
 bool bdrv_is_inserted(BlockDriverState *bs);
76daa3
 int bdrv_media_changed(BlockDriverState *bs);
76daa3
-- 
76daa3
1.8.3.1
76daa3