[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v4 00/12] ISA reconfigurability v4
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [RFC v4 00/12] ISA reconfigurability v4 |
Date: |
Mon, 13 Jun 2011 23:08:52 +0300 |
On Wed, Jun 8, 2011 at 9:55 PM, Andreas Färber <address@hidden> wrote:
> Hey,
>
> I've refined the series to track the state in ISADevice and to expose it as
> VMState.
> Error handling has been improved, and setting the state multiple times is
> no-op now.
>
> To read the state, I'm introducing support for bool qdev properties.
> Some more qdev_prop_get_*() helpers are introduced, too.
>
> Still need to do some runtime testing, but I'd like to hear if this is getting
> mergeable now, especially wrt VMState.
>
> Andreas
>
>
> Andreas Färber (11):
> qdev: Add support for property type bool
> qdev: Add helpers for reading properties
> isa: Provide set_state callback
> isa: Allow to un-assign I/O ports
> isa: Allow to un-associate an IRQ
I like the patches above.
But I think the set_state() interface could be improved. For example,
cpu_register_io_memory() gives an index which is passed to
sysbus_register_mmio(). Then the board can instantiate the device at
desired location without caring about the device internals. With
set_state(), the device does everything.
> parallel: Implement ISA set_state callback
> serial: Implement ISA set_state() callback
> fdc: Implement ISA set_state() callback
> ide: Allow to discard I/O ports
> ide: Implement ISA set_state() callback
> prep: Add pc87312 Super I/O emulation
>
> Hervé Poussineau (1):
> fdc: Parametrize ISA base, IRQ and DMA
>
> Makefile.objs | 1 +
> default-configs/ppc-softmmu.mak | 2 +
> hw/fdc.c | 62 ++++--
> hw/hw.h | 15 ++
> hw/ide/core.c | 8 +
> hw/ide/internal.h | 1 +
> hw/ide/isa.c | 32 +++-
> hw/isa-bus.c | 57 +++++
> hw/isa.h | 6 +
> hw/parallel.c | 69 ++++--
> hw/pc87312.c | 470
> +++++++++++++++++++++++++++++++++++++++
> hw/qdev-properties.c | 88 ++++++++
> hw/qdev.h | 13 +
> hw/serial.c | 30 ++-
> 14 files changed, 803 insertions(+), 51 deletions(-)
> create mode 100644 hw/pc87312.c
>
> --
> 1.7.5.3
>
>
>
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, (continued)
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Andreas Färber, 2011/06/09
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Gerd Hoffmann, 2011/06/09
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Andreas Färber, 2011/06/09
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Gerd Hoffmann, 2011/06/09
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Markus Armbruster, 2011/06/09
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Andreas Färber, 2011/06/12
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Markus Armbruster, 2011/06/09
- Re: [Qemu-devel] [RFC v4 03/12] isa: Provide set_state callback, Andreas Färber, 2011/06/12
- Re: [Qemu-devel] [PATCH v4 01/12] qdev: Add support for property type bool, Markus Armbruster, 2011/06/09
- Re: [Qemu-devel] [PATCH v4 01/12] qdev: Add support for property type bool, Andreas Färber, 2011/06/12
- Re: [Qemu-devel] [RFC v4 00/12] ISA reconfigurability v4,
Blue Swirl <=
- Re: [Qemu-devel] [RFC v4 00/12] ISA reconfigurability v4, Andreas Färber, 2011/06/13
- Re: [Qemu-devel] [RFC v4 00/12] ISA reconfigurability v4, Blue Swirl, 2011/06/15
- [Qemu-devel] [RFC v3 10/11] qdev: Add helpers for reading properties, Andreas Färber, 2011/06/07
- [Qemu-devel] [RFC v3 11/11] prep: Add pc87312 Super I/O emulation, Andreas Färber, 2011/06/07