mirror of https://github.com/xqemu/xqemu.git
pcspk: fix KVM
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: 873b4d3f05
Tested-by: Peter Lieven <pl@kamp.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 1467298657-6588-1-git-send-email-pbonzini@redhat.com
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
parent
297e8005f8
commit
8a0b4de048
|
@ -171,7 +171,7 @@ static void pcspk_initfn(Object *obj)
|
||||||
|
|
||||||
memory_region_init_io(&s->ioport, OBJECT(s), &pcspk_io_ops, s, "pcspk", 1);
|
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,
|
(Object **)&s->pit,
|
||||||
qdev_prop_allow_set_link_before_realize,
|
qdev_prop_allow_set_link_before_realize,
|
||||||
0, &error_abort);
|
0, &error_abort);
|
||||||
|
|
|
@ -37,6 +37,14 @@ typedef struct PITChannelInfo {
|
||||||
int out;
|
int out;
|
||||||
} PITChannelInfo;
|
} PITChannelInfo;
|
||||||
|
|
||||||
|
#define TYPE_PIT_COMMON "pit-common"
|
||||||
|
#define PIT_COMMON(obj) \
|
||||||
|
OBJECT_CHECK(PITCommonState, (obj), TYPE_PIT_COMMON)
|
||||||
|
#define PIT_COMMON_CLASS(klass) \
|
||||||
|
OBJECT_CLASS_CHECK(PITCommonClass, (klass), TYPE_PIT_COMMON)
|
||||||
|
#define PIT_COMMON_GET_CLASS(obj) \
|
||||||
|
OBJECT_GET_CLASS(PITCommonClass, (obj), TYPE_PIT_COMMON)
|
||||||
|
|
||||||
#define TYPE_I8254 "isa-pit"
|
#define TYPE_I8254 "isa-pit"
|
||||||
#define TYPE_KVM_I8254 "kvm-pit"
|
#define TYPE_KVM_I8254 "kvm-pit"
|
||||||
|
|
||||||
|
|
|
@ -57,14 +57,6 @@ typedef struct PITCommonState {
|
||||||
PITChannelState channels[3];
|
PITChannelState channels[3];
|
||||||
} PITCommonState;
|
} PITCommonState;
|
||||||
|
|
||||||
#define TYPE_PIT_COMMON "pit-common"
|
|
||||||
#define PIT_COMMON(obj) \
|
|
||||||
OBJECT_CHECK(PITCommonState, (obj), TYPE_PIT_COMMON)
|
|
||||||
#define PIT_COMMON_CLASS(klass) \
|
|
||||||
OBJECT_CLASS_CHECK(PITCommonClass, (klass), TYPE_PIT_COMMON)
|
|
||||||
#define PIT_COMMON_GET_CLASS(obj) \
|
|
||||||
OBJECT_GET_CLASS(PITCommonClass, (obj), TYPE_PIT_COMMON)
|
|
||||||
|
|
||||||
typedef struct PITCommonClass {
|
typedef struct PITCommonClass {
|
||||||
ISADeviceClass parent_class;
|
ISADeviceClass parent_class;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue