[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way |
Date: |
Wed, 28 Oct 2020 08:01:39 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Paolo Bonzini <pbonzini@redhat.com> writes:
> On 26/10/20 11:10, Markus Armbruster wrote:
>> Kevin's "[PATCH v2 0/6] qemu-storage-daemon: QAPIfy --chardev"
>> involves surgery to the QAPI generator. Some (most?) of it should go
>> away if we deprecate the "data" wrappers due to simple unions in QMP.
>>
>> Do we really need to mess with the code generator to solve the problem
>> at hand?
>>
>>
>> Let's recapitulate the problem:
>>
>> * We want to QAPIfy --chardev, i.e. define its argument as a QAPI
>> type.
>
> Considering that this is not 5.2 stuff at this point, I would like to
> suggest again moving chardevs to -object, and ask you to evaluate that
> option with the agreement that I do the work instead of you. :)
Replacing -chardev with -object without regressing features would be
lovely. One feature in particular: introspection.
If we manage to fully QAPIfy -object, we should be good. I understand
Eduardo is cutting a path through the jungle.
I can offer assistance with bridging QAPI schema to QOM.
- [PATCH 2/4] char: Factor out qemu_chr_print_types(), (continued)
- [PATCH 2/4] char: Factor out qemu_chr_print_types(), Markus Armbruster, 2020/10/26
- [PATCH 1/4] char/stdio: Fix QMP default for 'signal', Markus Armbruster, 2020/10/26
- [PATCH 4/4] qemu-storage-daemon: QAPIfy --chardev, Markus Armbruster, 2020/10/26
- [PATCH 3/4] char: Flat alternative to overly nested chardev-add arguments, Markus Armbruster, 2020/10/26
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Paolo Bonzini, 2020/10/27
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way,
Markus Armbruster <=
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Kevin Wolf, 2020/10/28
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Paolo Bonzini, 2020/10/28
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Kevin Wolf, 2020/10/28
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Paolo Bonzini, 2020/10/28
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Kevin Wolf, 2020/10/28
- Re: [PATCH 0/4] qemu-storage-daemon: QAPIfy --chardev the stupid way, Paolo Bonzini, 2020/10/28