Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-16 Thread Markus Armbruster
Blue Swirl writes: > On Tue, Jun 15, 2010 at 8:17 AM, Markus Armbruster wrote: >> Blue Swirl writes: >> >>> On Mon, Jun 14, 2010 at 9:36 AM, Markus Armbruster >>> wrote: Blue Swirl writes: > Make APICState completely private to apic.c by using DeviceState > in external APIs

Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-16 Thread Blue Swirl
On Tue, Jun 15, 2010 at 8:17 AM, Markus Armbruster wrote: > Blue Swirl writes: > >> On Mon, Jun 14, 2010 at 9:36 AM, Markus Armbruster wrote: >>> Blue Swirl writes: >>> Make APICState completely private to apic.c by using DeviceState in external APIs. >>> >>> Could you explain why thi

Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-15 Thread Markus Armbruster
Blue Swirl writes: > On Mon, Jun 14, 2010 at 9:36 AM, Markus Armbruster wrote: >> Blue Swirl writes: >> >>> Make APICState completely private to apic.c by using DeviceState >>> in external APIs. >> >> Could you explain why this is an improvement? > > Outside of apic.c, there is no need to acces

Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-14 Thread Blue Swirl
On Mon, Jun 14, 2010 at 9:36 AM, Markus Armbruster wrote: > Blue Swirl writes: > >> Make APICState completely private to apic.c by using DeviceState >> in external APIs. > > Could you explain why this is an improvement? Outside of apic.c, there is no need to access APICState fields so we can rem

Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-14 Thread Markus Armbruster
Blue Swirl writes: > Make APICState completely private to apic.c by using DeviceState > in external APIs. Could you explain why this is an improvement?

[Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-12 Thread Blue Swirl
Make APICState completely private to apic.c by using DeviceState in external APIs. Move apic_init() to pc.c. Signed-off-by: Blue Swirl --- hw/apic.c | 86 +++- hw/apic.h | 24 +++--- hw/pc.c | 32 +++