Re: [Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_queue_child()

2018-10-09 Thread Alberto Garcia
On Mon 08 Oct 2018 08:46:57 PM CEST, Max Reitz wrote: > On 08.10.18 20:13, Alberto Garcia wrote: >> On Mon 08 Oct 2018 04:48:50 AM CEST, Max Reitz wrote: >> +/* Old values are used for options that aren't set yet */ +old_options = qdict_clone_shallow(bs->options); +

Re: [Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_queue_child()

2018-10-08 Thread Max Reitz
On 08.10.18 20:13, Alberto Garcia wrote: > On Mon 08 Oct 2018 04:48:50 AM CEST, Max Reitz wrote: > >>> +/* Old values are used for options that aren't set yet */ >>> +old_options = qdict_clone_shallow(bs->options); >>> +bdrv_join_options(bs, options, old_options); >>> +

Re: [Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_queue_child()

2018-10-08 Thread Alberto Garcia
On Mon 08 Oct 2018 04:48:50 AM CEST, Max Reitz wrote: >> +/* Old values are used for options that aren't set yet */ >> +old_options = qdict_clone_shallow(bs->options); >> +bdrv_join_options(bs, options, old_options); >> +qobject_unref(old_options); >> + >> +/* We have the

Re: [Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_queue_child()

2018-10-07 Thread Max Reitz
On 19.09.18 16:47, Alberto Garcia wrote: > Now that all callers are passing the new options using the QDict we no > longer need the 'flags' parameter. > > This patch makes the following changes: > >1) The update_options_from_flags() call is no longer necessary > so it can be removed. >

[Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_queue_child()

2018-09-19 Thread Alberto Garcia
Now that all callers are passing the new options using the QDict we no longer need the 'flags' parameter. This patch makes the following changes: 1) The update_options_from_flags() call is no longer necessary so it can be removed. 2) The update_flags_from_options() call is now used