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

[Marcel Apfelbaum]

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

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(-)