Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-25 Thread Markus Armbruster
Paolo Bonzini writes: > On 23/10/20 15:40, Markus Armbruster wrote: >>> >>> The benefit of the user creatable object approach is that we dont >>> have to add custom CLI args for different types of object, nor write >>> code to populate QOM from QAPI. The downside is that we're divorced >>> from

Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-23 Thread Paolo Bonzini
On 23/10/20 15:40, Markus Armbruster wrote: >> >> The benefit of the user creatable object approach is that we dont >> have to add custom CLI args for different types of object, nor write >> code to populate QOM from QAPI. The downside is that we're divorced >> from the QAPI schema, so loose

Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-23 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Fri, Oct 23, 2020 at 12:12:16PM +0200, Kevin Wolf wrote: >> While the qemu-storage-daemon command line is still considered unstable, >> let's change --chardev from the old QemuOpts-based parser to QAPI, so >> that it becomes a simple mapping of chardev-add to the

Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-23 Thread Kevin Wolf
Am 23.10.2020 um 12:36 hat Daniel P. Berrangé geschrieben: > On Fri, Oct 23, 2020 at 12:12:16PM +0200, Kevin Wolf wrote: > > While the qemu-storage-daemon command line is still considered unstable, > > let's change --chardev from the old QemuOpts-based parser to QAPI, so > > that it becomes a

Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-23 Thread Paolo Bonzini
On 23/10/20 12:36, Daniel P. Berrangé wrote: > IOW, if we're considering our ideal long term configuration goals > for QEMU, then I think we should be looking at a solution to bridge > the gap, so that we can get the best of both worlds, and have a single > way todo things. > > IMHO, ideally none

Re: [PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-23 Thread Daniel P . Berrangé
On Fri, Oct 23, 2020 at 12:12:16PM +0200, Kevin Wolf wrote: > While the qemu-storage-daemon command line is still considered unstable, > let's change --chardev from the old QemuOpts-based parser to QAPI, so > that it becomes a simple mapping of chardev-add to the command line and > will fit in a

[PATCH 0/6] qemu-storage-daemon: QAPIfy --chardev

2020-10-23 Thread Kevin Wolf
While the qemu-storage-daemon command line is still considered unstable, let's change --chardev from the old QemuOpts-based parser to QAPI, so that it becomes a simple mapping of chardev-add to the command line and will fit in a future fully QAPIfied command line without later incompatible changes