i8259: Convert PICCommonState to use QOM realizefn
Instead of having the parent provide PICCommonClass::init, let the children override DeviceClass::realize themselves. This pushes the responsibility of saving and calling the parent's realizefn to the children. Signed-off-by: Andreas Färber <afaerber@suse.de>
This commit is contained in:
parent
49fdb0c1c4
commit
d2628b7d18
|
@ -14,6 +14,20 @@
|
|||
#include "sysemu/kvm.h"
|
||||
|
||||
#define TYPE_KVM_I8259 "kvm-i8259"
|
||||
#define KVM_PIC_CLASS(class) \
|
||||
OBJECT_CLASS_CHECK(KVMPICClass, (class), TYPE_KVM_I8259)
|
||||
#define KVM_PIC_GET_CLASS(obj) \
|
||||
OBJECT_GET_CLASS(KVMPICClass, (obj), TYPE_KVM_I8259)
|
||||
|
||||
/**
|
||||
* KVMPICClass:
|
||||
* @parent_realize: The parent's realizefn.
|
||||
*/
|
||||
typedef struct KVMPICClass {
|
||||
PICCommonClass parent_class;
|
||||
|
||||
DeviceRealize parent_realize;
|
||||
} KVMPICClass;
|
||||
|
||||
static void kvm_pic_get(PICCommonState *s)
|
||||
{
|
||||
|
@ -100,10 +114,15 @@ static void kvm_pic_set_irq(void *opaque, int irq, int level)
|
|||
apic_report_irq_delivered(delivered);
|
||||
}
|
||||
|
||||
static void kvm_pic_init(PICCommonState *s)
|
||||
static void kvm_pic_realize(DeviceState *dev, Error **errp)
|
||||
{
|
||||
PICCommonState *s = PIC_COMMON(dev);
|
||||
KVMPICClass *kpc = KVM_PIC_GET_CLASS(dev);
|
||||
|
||||
memory_region_init_reservation(&s->base_io, "kvm-pic", 2);
|
||||
memory_region_init_reservation(&s->elcr_io, "kvm-elcr", 1);
|
||||
|
||||
kpc->parent_realize(dev, errp);
|
||||
}
|
||||
|
||||
qemu_irq *kvm_i8259_init(ISABus *bus)
|
||||
|
@ -116,11 +135,13 @@ qemu_irq *kvm_i8259_init(ISABus *bus)
|
|||
|
||||
static void kvm_i8259_class_init(ObjectClass *klass, void *data)
|
||||
{
|
||||
KVMPICClass *kpc = KVM_PIC_CLASS(klass);
|
||||
PICCommonClass *k = PIC_COMMON_CLASS(klass);
|
||||
DeviceClass *dc = DEVICE_CLASS(klass);
|
||||
|
||||
dc->reset = kvm_pic_reset;
|
||||
k->init = kvm_pic_init;
|
||||
kpc->parent_realize = dc->realize;
|
||||
dc->realize = kvm_pic_realize;
|
||||
k->pre_save = kvm_pic_get;
|
||||
k->post_load = kvm_pic_put;
|
||||
}
|
||||
|
@ -130,6 +151,7 @@ static const TypeInfo kvm_i8259_info = {
|
|||
.parent = TYPE_PIC_COMMON,
|
||||
.instance_size = sizeof(PICCommonState),
|
||||
.class_init = kvm_i8259_class_init,
|
||||
.class_size = sizeof(KVMPICClass),
|
||||
};
|
||||
|
||||
static void kvm_pic_register_types(void)
|
||||
|
|
|
@ -42,6 +42,18 @@
|
|||
//#define DEBUG_IRQ_COUNT
|
||||
|
||||
#define TYPE_I8259 "isa-i8259"
|
||||
#define PIC_CLASS(class) OBJECT_CLASS_CHECK(PICClass, (class), TYPE_I8259)
|
||||
#define PIC_GET_CLASS(obj) OBJECT_GET_CLASS(PICClass, (obj), TYPE_I8259)
|
||||
|
||||
/**
|
||||
* PICClass:
|
||||
* @parent_realize: The parent's realizefn.
|
||||
*/
|
||||
typedef struct PICClass {
|
||||
PICCommonClass parent_class;
|
||||
|
||||
DeviceRealize parent_realize;
|
||||
} PICClass;
|
||||
|
||||
#if defined(DEBUG_PIC) || defined(DEBUG_IRQ_COUNT)
|
||||
static int irq_level[16];
|
||||
|
@ -400,15 +412,18 @@ static const MemoryRegionOps pic_elcr_ioport_ops = {
|
|||
},
|
||||
};
|
||||
|
||||
static void pic_init(PICCommonState *s)
|
||||
static void pic_realize(DeviceState *dev, Error **err)
|
||||
{
|
||||
DeviceState *dev = DEVICE(s);
|
||||
PICCommonState *s = PIC_COMMON(dev);
|
||||
PICClass *pc = PIC_GET_CLASS(dev);
|
||||
|
||||
memory_region_init_io(&s->base_io, &pic_base_ioport_ops, s, "pic", 2);
|
||||
memory_region_init_io(&s->elcr_io, &pic_elcr_ioport_ops, s, "elcr", 1);
|
||||
|
||||
qdev_init_gpio_out(dev, s->int_out, ARRAY_SIZE(s->int_out));
|
||||
qdev_init_gpio_in(dev, pic_set_irq, 8);
|
||||
|
||||
pc->parent_realize(dev, err);
|
||||
}
|
||||
|
||||
void pic_info(Monitor *mon, const QDict *qdict)
|
||||
|
@ -481,10 +496,11 @@ qemu_irq *i8259_init(ISABus *bus, qemu_irq parent_irq)
|
|||
|
||||
static void i8259_class_init(ObjectClass *klass, void *data)
|
||||
{
|
||||
PICCommonClass *k = PIC_COMMON_CLASS(klass);
|
||||
PICClass *k = PIC_CLASS(klass);
|
||||
DeviceClass *dc = DEVICE_CLASS(klass);
|
||||
|
||||
k->init = pic_init;
|
||||
k->parent_realize = dc->realize;
|
||||
dc->realize = pic_realize;
|
||||
dc->reset = pic_reset;
|
||||
}
|
||||
|
||||
|
@ -493,6 +509,7 @@ static const TypeInfo i8259_info = {
|
|||
.instance_size = sizeof(PICCommonState),
|
||||
.parent = TYPE_PIC_COMMON,
|
||||
.class_init = i8259_class_init,
|
||||
.class_size = sizeof(PICClass),
|
||||
};
|
||||
|
||||
static void pic_register_types(void)
|
||||
|
|
|
@ -69,9 +69,6 @@ static int pic_dispatch_post_load(void *opaque, int version_id)
|
|||
static void pic_common_realize(DeviceState *dev, Error **errp)
|
||||
{
|
||||
PICCommonState *s = PIC_COMMON(dev);
|
||||
PICCommonClass *info = PIC_COMMON_GET_CLASS(s);
|
||||
|
||||
info->init(s);
|
||||
|
||||
isa_register_ioport(NULL, &s->base_io, s->iobase);
|
||||
if (s->elcr_addr != -1) {
|
||||
|
|
|
@ -42,7 +42,7 @@ typedef struct PICCommonState PICCommonState;
|
|||
typedef struct PICCommonClass
|
||||
{
|
||||
ISADeviceClass parent_class;
|
||||
void (*init)(PICCommonState *s);
|
||||
|
||||
void (*pre_save)(PICCommonState *s);
|
||||
void (*post_load)(PICCommonState *s);
|
||||
} PICCommonClass;
|
||||
|
|
Loading…
Reference in a new issue