[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic
From: |
Jamie Lokier |
Subject: |
Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic |
Date: |
Thu, 8 Oct 2009 17:07:26 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
Avi Kivity wrote:
> On 10/08/2009 03:49 PM, Anthony Liguori wrote:
> >Glauber Costa wrote:
> >>This patch provides kvm with an in-kernel ioapic. We are currently
> >>not enabling it.
> >>The code is heavily based on what's in qemu-kvm.git.
> >
> >It really ought to be it's own file and own device model. Having the
> >code mixed in with ioapic.c is confusing because it's unclear what
> >code is in use when the in-kernel model is used.
>
> I disagree. It's the same device with the same guest-visible interface
> and the same host-visible interface (save/restore, 'info ioapic' if we
> write one). Splitting it into two files will only result in code
> duplication.
>
> Think of it as an ioapic accelerator.
Haven't we already confirmed that it *isn't* just an ioapic accelerator
because you can't migrate between in-kernel iopic and qemu's ioapic?
Imho, if they cannot be swapped transparently, they are different
device emulations.
OF course there's nothing wrong with sharing lots of code.
Maybe ioapic.c and ioapic-kvm.c, with shared code in ioapic-common.c?
-- Jamie
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, (continued)
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, Anthony Liguori, 2009/10/12
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, Anthony Liguori, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, Avi Kivity, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, Anthony Liguori, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, Glauber Costa, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 4/9] provide in-kernel apic, Glauber Costa, 2009/10/08
- [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Jan Kiszka, 2009/10/08
- [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Anthony Liguori, 2009/10/08
- [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Avi Kivity, 2009/10/08
- [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Jan Kiszka, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic,
Jamie Lokier <=
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Anthony Liguori, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Avi Kivity, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Gleb Natapov, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Avi Kivity, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Gleb Natapov, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Avi Kivity, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Gleb Natapov, 2009/10/08
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Jamie Lokier, 2009/10/09
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Gleb Natapov, 2009/10/09
- Re: [Qemu-devel] Re: [PATCH v2 3/9] provide in-kernel ioapic, Glauber Costa, 2009/10/09