qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 0/4] QOM'ify apic and ioapic


From: Stefano Stabellini
Subject: Re: [Qemu-devel] [PATCH v3 0/4] QOM'ify apic and ioapic
Date: Wed, 18 Dec 2013 19:29:18 +0000
User-agent: Alpine 2.02 (DEB 1266 2009-07-14)

On Wed, 18 Dec 2013, Andreas Färber wrote:
> Am 18.12.2013 20:20, schrieb Stefano Stabellini:
> > On Wed, 18 Dec 2013, Andreas Färber wrote:
> >> Am 18.12.2013 18:57, schrieb Andreas Färber:
> >>> Am 05.11.2013 11:16, schrieb xiaoqiang zhao:
> >>>> xiaoqiang zhao (4):
> >>>>   apic: Cleanup for QOM'ify
> >>>>   apic: QOM'ify apic & icc_bus
> >>>>   ioapic: Cleanup for QOM'ify
> >>>>   ioapic: QOM'ify ioapic
> >>>
> >>> Thanks, I have rebased and queued them with changes on qom-next:
> >>> https://github.com/afaerber/qemu-cpu/commits/qom-next
> >>>
> >>> In particular I have reverted TYPE_ introductions were unnecessary and
> >>> reverted usages of TYPE_ constants for MemoryRegions. Also two white
> >>> lines have been dropped. The ICC bus conversion has been split out as a
> >>> separate step, using NULL errp as intermediate step since ignored.
> >>> (CC'ing Peter C. and Luiz)
> >>
> >> Oh and you forgot to convert the Xen APIC, I trivially fixed that.
> >> (CC'ing Stefano)
> > 
> > Yes, it should be really trivial. Give a look at hw/xen/xen_apic.c.
> 
> https://github.com/afaerber/qemu-cpu/commit/40ae2368c4c9135971c69fa680949b5b1cfa4410#diff-3
> 
> Just wanted to give you a heads-up that we're messing in your code. :)

Ah OK, thank you :-)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]