Re: [Qemu-ppc] [Qemu-devel] [PATCH 00/10] Fix machine class names to use the "-machine" suffix

[Eduardo Habkost]

Ping?

Andreas, should this go through your QOM tree?

On Thu, Aug 20, 2015 at 02:54:26PM -0700, Eduardo Habkost wrote:
> The machine class names in QEMU are not consistent today. Some of them use the
> "-machine" suffix, but some use completely different naming conventions. This
> makes it difficult to implement machine class lookup without iterating through
> the whole list of machine classes.
> 
> This series changes all machines to use "<machine name>-machine" as the class
> name, so we can simply use object_class_by_name() in the future to lookup
> machine classes. (This was not done yet because it will also require some
> changes in the way aliases are handled.)
> 
> Cc: Alexander Graf <address@hidden>
> Cc: Christian Borntraeger <address@hidden>
> Cc: Cornelia Huck <address@hidden>
> Cc: Peter Maydell <address@hidden>
> Cc: address@hidden
> Cc: Richard Henderson <address@hidden>
> 
> Eduardo Habkost (10):
>   machine: MACHINE_TYPE_NAME macro
>   vexpress: Don't set name on abstract class
>   vexpress: Rename machine classes to use MACHINE_TYPE_NAME
>   arm: Rename virt machine class to use MACHINE_TYPE_NAME
>   pseries: Rename machine class names to use MACHINE_TYPE_NAME
>   s390-virtio: Rename machine class name to use MACHINE_TYPE_NAME
>   s390: Rename s390-ccw-virtio-2.4 class name to use MACHINE_TYPE_NAME
>   mac99: Use MACHINE_TYPE_NAME to encode class name
>   machine: Ensure all TYPE_MACHINE subclasses have the right suffix
>   machine: Set MachineClass::name automatically
> 
>  hw/arm/vexpress.c          |  7 ++-----
>  hw/arm/virt.c              |  4 ++--
>  hw/core/machine.c          | 12 ++++++++++++
>  hw/ppc/mac_newworld.c      |  3 +--
>  hw/ppc/spapr.c             | 12 ++++--------
>  hw/s390x/s390-virtio-ccw.c |  2 +-
>  hw/s390x/s390-virtio.c     |  4 ++--
>  include/hw/boards.h        |  6 ++++++
>  vl.c                       |  3 +--
>  9 files changed, 31 insertions(+), 22 deletions(-)
> 
> -- 
> 2.1.0
> 
> 

-- 
Eduardo