Re: [Qemu-devel] [PATCH 0/8] machine: query machine properties rather than qemu opts

[Marcel Apfelbaum <marcel.apfelbaum@gmail.com>]

On 02/25/2015 01:55 PM, Marcel Apfelbaum wrote:
> On 02/04/2015 05:43 PM, Marcel Apfelbaum wrote:
>> Commit e79d5a6 ("machine: remove qemu_machine_opts global list") removed option
>> descriptions from the -machine QemuOptsList to avoid repeating MachineState's QOM properties.
>>
>> This results in a Qemu crash if a non string option is queried using qemu opts.
>> Fix this by querying machine properties through designated wrappers.
>>
>> I hope I didn't miss anything.
>> Comments are appreciated as always.
> Ping
Ping ping :)

Thanks,
Marcel
>
> Hi, who can take this series?
> It solves some command line bugs and people were pretty annoyed about it.
>
> Thanks,
> Marcel
>
>>
>> Thanks,
>> Marcel
>>
>> Marcel Apfelbaum (8):
>>    machine: query iommu machine property rather than qemu opts
>>    hw/machine: kernel-irqchip property support for allowed/required
>>    machine: query kernel-irqchip machine property rather than qemu opts
>>    kvm: add machine state to kvm_arch_init
>>    machine: query kvm-shadow-mem machine property rather than qemu opts
>>    machine: query phandle-start machine property rather than qemu opts
>>    machine: query dump-guest-core machine property rather than qemu opts
>>    machine: query mem-merge machine property rather than qemu opts
>>
>>   device_tree.c        |  5 ++---
>>   exec.c               |  6 +++---
>>   hw/core/machine.c    | 52 +++++++++++++++++++++++++++++++++++++++++++---------
>>   hw/pci-host/q35.c    |  2 +-
>>   hw/ppc/e500.c        | 16 +++++-----------
>>   hw/ppc/spapr.c       | 16 ++++++----------
>>   include/hw/boards.h  | 10 +++++++++-
>>   include/sysemu/kvm.h |  2 +-
>>   kvm-all.c            |  8 ++++----
>>   target-arm/kvm.c     |  2 +-
>>   target-i386/kvm.c    |  5 ++---
>>   target-mips/kvm.c    |  2 +-
>>   target-ppc/kvm.c     |  2 +-
>>   target-s390x/kvm.c   |  2 +-
>>   14 files changed, 80 insertions(+), 50 deletions(-)
>>
>