[Qemu-devel] Re: [PATCH v2 0/7] APIC/IOAPIC cleanup

[Jan Kiszka <jan.kiszka@web.de>]

[-- Attachment #1: Type: text/plain, Size: 1292 bytes --]

Blue Swirl wrote:
> Clean up APIC and IOAPIC. Convert both devices to qdev.

Wanted to give this a try, but patches are line-wrapped. Please fix.

Jan

> 
> v1->v2:
> Remove apic.h reorganization.
> Add IOAPIC and APIC qdev conversions.
> Use CPUState also in 5/7. However on 6/7 we have to again use void *
> because of VMState limitations. VMState gurus, please comment.
> 
> Blue Swirl (7):
>   ioapic: unexport ioapic_set_irq
>   ioapic: convert to qdev
>   apic: avoid passing CPUState from devices
>   apic: avoid passing CPUState from CPU code
>   apic: avoid using CPUState internals
>   apic: convert to qdev
>   apic: qdev conversion cleanup
> 
>  hw/apic.c               |  174 +++++++++++++++++++++++-----------------------
>  hw/apic.h               |   21 ++++--
>  hw/ioapic.c             |   47 ++++++++----
>  hw/pc.c                 |   74 ++++++++++++++++++--
>  hw/pc.h                 |    4 +-
>  hw/pc_piix.c            |   19 +++++-
>  qemu-common.h           |    2 +-
>  target-i386/cpu.h       |   28 +++++---
>  target-i386/cpuid.c     |    6 ++
>  target-i386/helper.c    |    4 +-
>  target-i386/kvm.c       |   14 ++--
>  target-i386/op_helper.c |    8 +-
>  12 files changed, 258 insertions(+), 143 deletions(-)
> 
> 



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 257 bytes --]