* [Qemu-devel] [PULL 0/2] Block layer patches
@ 2019-04-08 16:34 Kevin Wolf
2019-04-08 16:34 ` [Qemu-devel] [PULL 1/2] block: Forward 'discard' to temporary overlay Kevin Wolf
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Kevin Wolf @ 2019-04-08 16:34 UTC (permalink / raw)
To: qemu-block; +Cc: kwolf, peter.maydell, qemu-devel
The following changes since commit f55a585d1037e5de6088f25e75443c2776786e29:
Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2019-04-07 14:54:55 +0100)
are available in the Git repository at:
git://repo.or.cz/qemu/kevin.git tags/for-upstream
for you to fetch changes up to ab63817119b03b95f7dce6fae036e6d063ad63fb:
hmp: Fix drive_add ... format=help crash (2019-04-08 17:42:06 +0200)
----------------------------------------------------------------
Block layer patches:
- hmp: Fix drive_add ... format=help crash
- block: Forward 'discard' to temporary overlay
----------------------------------------------------------------
Kevin Wolf (1):
block: Forward 'discard' to temporary overlay
Markus Armbruster (1):
hmp: Fix drive_add ... format=help crash
block.c | 3 ++-
device-hotplug.c | 2 +-
tests/test-hmp.c | 1 +
3 files changed, 4 insertions(+), 2 deletions(-)
^ permalink raw reply [flat|nested] 9+ messages in thread* [Qemu-devel] [PULL 1/2] block: Forward 'discard' to temporary overlay
2019-04-08 16:34 [Qemu-devel] [PULL 0/2] Block layer patches Kevin Wolf
@ 2019-04-08 16:34 ` Kevin Wolf
2019-04-08 16:34 ` [Qemu-devel] [PULL 2/2] hmp: Fix drive_add ... format=help crash Kevin Wolf
2019-04-08 19:10 ` Peter Maydell
2 siblings, 0 replies; 9+ messages in thread
From: Kevin Wolf @ 2019-04-08 16:34 UTC (permalink / raw)
To: qemu-block; +Cc: kwolf, peter.maydell, qemu-devel
When bdrv_temp_snapshot_options() is called for snapshot=on, the
'discard' option in the options QDict hasn't been parsed and merged into
the flags yet. So copy the dict entry to make sure that the temporary
overlay enables discard when it was requested for the drive.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
---
block.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/block.c b/block.c
index 3050854528..16615bc876 100644
--- a/block.c
+++ b/block.c
@@ -950,8 +950,9 @@ static void bdrv_temp_snapshot_options(int *child_flags, QDict *child_options,
qdict_set_default_str(child_options, BDRV_OPT_CACHE_DIRECT, "off");
qdict_set_default_str(child_options, BDRV_OPT_CACHE_NO_FLUSH, "on");
- /* Copy the read-only option from the parent */
+ /* Copy the read-only and discard options from the parent */
qdict_copy_default(child_options, parent_options, BDRV_OPT_READ_ONLY);
+ qdict_copy_default(child_options, parent_options, BDRV_OPT_DISCARD);
/* aio=native doesn't work for cache.direct=off, so disable it for the
* temporary snapshot */
--
2.20.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Qemu-devel] [PULL 2/2] hmp: Fix drive_add ... format=help crash
2019-04-08 16:34 [Qemu-devel] [PULL 0/2] Block layer patches Kevin Wolf
2019-04-08 16:34 ` [Qemu-devel] [PULL 1/2] block: Forward 'discard' to temporary overlay Kevin Wolf
@ 2019-04-08 16:34 ` Kevin Wolf
2019-04-08 19:10 ` Peter Maydell
2 siblings, 0 replies; 9+ messages in thread
From: Kevin Wolf @ 2019-04-08 16:34 UTC (permalink / raw)
To: qemu-block; +Cc: kwolf, peter.maydell, qemu-devel
From: Markus Armbruster <armbru@redhat.com>
drive_new() returns null without setting an error when it provided
help. add_init_drive() assumes null means failure, and crashes trying
to report a null error.
Fixes: c4f26c9f37ce511e5fe629c21c180dc6eb7c5a25
Cc: qemu-stable@nongnu.org
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
device-hotplug.c | 2 +-
tests/test-hmp.c | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/device-hotplug.c b/device-hotplug.c
index 6090d5f1e9..6153259d71 100644
--- a/device-hotplug.c
+++ b/device-hotplug.c
@@ -48,7 +48,7 @@ static DriveInfo *add_init_drive(const char *optstr)
mc = MACHINE_GET_CLASS(current_machine);
dinfo = drive_new(opts, mc->block_default_type, &err);
- if (!dinfo) {
+ if (err) {
error_report_err(err);
qemu_opts_del(opts);
return NULL;
diff --git a/tests/test-hmp.c b/tests/test-hmp.c
index 8c49d2fdf1..54a01824dc 100644
--- a/tests/test-hmp.c
+++ b/tests/test-hmp.c
@@ -31,6 +31,7 @@ static const char *hmp_cmds[] = {
"cpu 0",
"device_add ?",
"device_add usb-mouse,id=mouse1",
+ "drive_add ignored format=help",
"mouse_button 7",
"mouse_move 10 10",
"mouse_button 0",
--
2.20.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [Qemu-devel] [PULL 0/2] Block layer patches
@ 2019-04-08 19:10 ` Peter Maydell
0 siblings, 0 replies; 9+ messages in thread
From: Peter Maydell @ 2019-04-08 19:10 UTC (permalink / raw)
To: Kevin Wolf; +Cc: Qemu-block, QEMU Developers
On Mon, 8 Apr 2019 at 17:35, Kevin Wolf <kwolf@redhat.com> wrote:
>
> The following changes since commit f55a585d1037e5de6088f25e75443c2776786e29:
>
> Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2019-04-07 14:54:55 +0100)
>
> are available in the Git repository at:
>
> git://repo.or.cz/qemu/kevin.git tags/for-upstream
>
> for you to fetch changes up to ab63817119b03b95f7dce6fae036e6d063ad63fb:
>
> hmp: Fix drive_add ... format=help crash (2019-04-08 17:42:06 +0200)
>
> ----------------------------------------------------------------
> Block layer patches:
>
> - hmp: Fix drive_add ... format=help crash
> - block: Forward 'discard' to temporary overlay
>
> ----------------------------------------------------------------
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/4.0
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PULL 0/2] Block layer patches
@ 2019-04-08 19:10 ` Peter Maydell
0 siblings, 0 replies; 9+ messages in thread
From: Peter Maydell @ 2019-04-08 19:10 UTC (permalink / raw)
To: Kevin Wolf; +Cc: QEMU Developers, Qemu-block
On Mon, 8 Apr 2019 at 17:35, Kevin Wolf <kwolf@redhat.com> wrote:
>
> The following changes since commit f55a585d1037e5de6088f25e75443c2776786e29:
>
> Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2019-04-07 14:54:55 +0100)
>
> are available in the Git repository at:
>
> git://repo.or.cz/qemu/kevin.git tags/for-upstream
>
> for you to fetch changes up to ab63817119b03b95f7dce6fae036e6d063ad63fb:
>
> hmp: Fix drive_add ... format=help crash (2019-04-08 17:42:06 +0200)
>
> ----------------------------------------------------------------
> Block layer patches:
>
> - hmp: Fix drive_add ... format=help crash
> - block: Forward 'discard' to temporary overlay
>
> ----------------------------------------------------------------
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/4.0
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Qemu-devel] [PULL 0/2] Block layer patches
@ 2019-07-12 13:52 Kevin Wolf
2019-07-12 15:32 ` Peter Maydell
0 siblings, 1 reply; 9+ messages in thread
From: Kevin Wolf @ 2019-07-12 13:52 UTC (permalink / raw)
To: qemu-block; +Cc: kwolf, peter.maydell, qemu-devel
The following changes since commit a2a9d4adabe340617a24eb73a8b2a116d28a6b38:
Merge remote-tracking branch 'remotes/dgibson/tags/ppc-for-4.1-20190712' into staging (2019-07-12 11:06:48 +0100)
are available in the Git repository at:
git://repo.or.cz/qemu/kevin.git tags/for-upstream
for you to fetch changes up to 867eccfed84f96b54f4a432c510a02c2ce03b430:
file-posix: Use max transfer length/segment count only for SCSI passthrough (2019-07-12 15:42:23 +0200)
----------------------------------------------------------------
Block layer patches:
- file-posix: Fix max transfer length for non-SCSI-passthrough
- iotests: Fix 082 reference output
----------------------------------------------------------------
Eric Blake (1):
iotests: Update 082 expected output
Maxim Levitsky (1):
file-posix: Use max transfer length/segment count only for SCSI passthrough
block/file-posix.c | 54 ++++++++++++++++++++++++----------------------
tests/qemu-iotests/082.out | 54 +++++++++++++++++++++++-----------------------
2 files changed, 55 insertions(+), 53 deletions(-)
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [Qemu-devel] [PULL 0/2] Block layer patches
2019-07-12 13:52 Kevin Wolf
@ 2019-07-12 15:32 ` Peter Maydell
0 siblings, 0 replies; 9+ messages in thread
From: Peter Maydell @ 2019-07-12 15:32 UTC (permalink / raw)
To: Kevin Wolf; +Cc: QEMU Developers, Qemu-block
On Fri, 12 Jul 2019 at 14:53, Kevin Wolf <kwolf@redhat.com> wrote:
>
> The following changes since commit a2a9d4adabe340617a24eb73a8b2a116d28a6b38:
>
> Merge remote-tracking branch 'remotes/dgibson/tags/ppc-for-4.1-20190712' into staging (2019-07-12 11:06:48 +0100)
>
> are available in the Git repository at:
>
> git://repo.or.cz/qemu/kevin.git tags/for-upstream
>
> for you to fetch changes up to 867eccfed84f96b54f4a432c510a02c2ce03b430:
>
> file-posix: Use max transfer length/segment count only for SCSI passthrough (2019-07-12 15:42:23 +0200)
>
> ----------------------------------------------------------------
> Block layer patches:
>
> - file-posix: Fix max transfer length for non-SCSI-passthrough
> - iotests: Fix 082 reference output
>
> ----------------------------------------------------------------
> Eric Blake (1):
> iotests: Update 082 expected output
>
> Maxim Levitsky (1):
> file-posix: Use max transfer length/segment count only for SCSI passthrough
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/4.1
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Qemu-devel] [PULL 0/2] Block layer patches
@ 2018-12-03 16:58 Kevin Wolf
2018-12-03 17:43 ` Peter Maydell
0 siblings, 1 reply; 9+ messages in thread
From: Kevin Wolf @ 2018-12-03 16:58 UTC (permalink / raw)
To: qemu-block; +Cc: kwolf, peter.maydell, qemu-devel
The following changes since commit 83ea23cd207a03c5736be0231acbf7f8b05dbf52:
i386: hvf: Fix overrun of _decode_tbl1 (2018-12-03 15:09:55 +0000)
are available in the Git repository at:
git://repo.or.cz/qemu/kevin.git tags/for-upstream
for you to fetch changes up to db5e8210adbafe9c6383d8364345a8c545d38e62:
iotests: simple mirror test with kvm on 1G image (2018-12-03 16:51:53 +0100)
----------------------------------------------------------------
Block layer patches:
- mirror: Fix deadlock
----------------------------------------------------------------
Vladimir Sementsov-Ogievskiy (2):
mirror: fix dead-lock
iotests: simple mirror test with kvm on 1G image
block/mirror.c | 13 +++-----
tests/qemu-iotests/235 | 76 ++++++++++++++++++++++++++++++++++++++++++++++
tests/qemu-iotests/235.out | 3 ++
tests/qemu-iotests/group | 1 +
4 files changed, 85 insertions(+), 8 deletions(-)
create mode 100755 tests/qemu-iotests/235
create mode 100644 tests/qemu-iotests/235.out
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [Qemu-devel] [PULL 0/2] Block layer patches
2018-12-03 16:58 Kevin Wolf
@ 2018-12-03 17:43 ` Peter Maydell
0 siblings, 0 replies; 9+ messages in thread
From: Peter Maydell @ 2018-12-03 17:43 UTC (permalink / raw)
To: Kevin Wolf; +Cc: Qemu-block, QEMU Developers
On Mon, 3 Dec 2018 at 16:58, Kevin Wolf <kwolf@redhat.com> wrote:
>
> The following changes since commit 83ea23cd207a03c5736be0231acbf7f8b05dbf52:
>
> i386: hvf: Fix overrun of _decode_tbl1 (2018-12-03 15:09:55 +0000)
>
> are available in the Git repository at:
>
> git://repo.or.cz/qemu/kevin.git tags/for-upstream
>
> for you to fetch changes up to db5e8210adbafe9c6383d8364345a8c545d38e62:
>
> iotests: simple mirror test with kvm on 1G image (2018-12-03 16:51:53 +0100)
>
> ----------------------------------------------------------------
> Block layer patches:
>
> - mirror: Fix deadlock
>
Applied, thanks.
-- PMM
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-07-12 15:33 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-08 16:34 [Qemu-devel] [PULL 0/2] Block layer patches Kevin Wolf
2019-04-08 16:34 ` [Qemu-devel] [PULL 1/2] block: Forward 'discard' to temporary overlay Kevin Wolf
2019-04-08 16:34 ` [Qemu-devel] [PULL 2/2] hmp: Fix drive_add ... format=help crash Kevin Wolf
2019-04-08 19:10 ` [Qemu-devel] [PULL 0/2] Block layer patches Peter Maydell
2019-04-08 19:10 ` Peter Maydell
-- strict thread matches above, loose matches on Subject: below --
2019-07-12 13:52 Kevin Wolf
2019-07-12 15:32 ` Peter Maydell
2018-12-03 16:58 Kevin Wolf
2018-12-03 17:43 ` Peter Maydell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.