Re: [Qemu-devel] [PATCH] pcspk: fix KVM

[Peter Maydell]

On 30 June 2016 at 15:41, Paolo Bonzini <address@hidden> wrote:
> The link property that was added to the pcspk device has the wrong type:
> it is only correct for TCG and for KVM's userspace or split irqchip
> options.  The default KVM option (fully in-kernel irqchip) breaks
> because it uses a PIT whose type is a sibling of TYPE_I8254.
>
> Fixes: 873b4d3f0571a1e415cf089a67a230ea8a12d059
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  hw/audio/pcspk.c                  | 2 +-
>  include/hw/timer/i8254.h          | 1 +
>  include/hw/timer/i8254_internal.h | 1 -
>  3 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/audio/pcspk.c b/hw/audio/pcspk.c
> index a7aa4e9..42a6f48 100644
> --- a/hw/audio/pcspk.c
> +++ b/hw/audio/pcspk.c
> @@ -171,7 +171,7 @@ static void pcspk_initfn(Object *obj)
>
>      memory_region_init_io(&s->ioport, OBJECT(s), &pcspk_io_ops, s, "pcspk", 
> 1);
>
> -    object_property_add_link(obj, "pit", TYPE_I8254,
> +    object_property_add_link(obj, "pit", TYPE_PIT_COMMON,
>                               (Object **)&s->pit,
>                               qdev_prop_allow_set_link_before_realize,
>                               0, &error_abort);
> diff --git a/include/hw/timer/i8254.h b/include/hw/timer/i8254.h
> index 4349033..28a3fdc 100644
> --- a/include/hw/timer/i8254.h
> +++ b/include/hw/timer/i8254.h
> @@ -37,6 +37,7 @@ typedef struct PITChannelInfo {
>      int out;
>  } PITChannelInfo;
>
> +#define TYPE_PIT_COMMON "pit-common"
>  #define TYPE_I8254 "isa-pit"
>  #define TYPE_KVM_I8254 "kvm-pit"
>
> diff --git a/include/hw/timer/i8254_internal.h 
> b/include/hw/timer/i8254_internal.h
> index 61a1bfb..5fefd84 100644
> --- a/include/hw/timer/i8254_internal.h
> +++ b/include/hw/timer/i8254_internal.h
> @@ -57,7 +57,6 @@ typedef struct PITCommonState {
>      PITChannelState channels[3];
>  } PITCommonState;
>
> -#define TYPE_PIT_COMMON "pit-common"
>  #define PIT_COMMON(obj) \
>       OBJECT_CHECK(PITCommonState, (obj), TYPE_PIT_COMMON)
>  #define PIT_COMMON_CLASS(klass) \

Rather nonstandard not to keep all the QOM type #defines together.

thanks
-- PMM