WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Qemu-devel] [Xen-devel] [PATCH RFC V1 00/11] Xen PCI Passthrough

To: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Subject: Re: [Qemu-devel] [Xen-devel] [PATCH RFC V1 00/11] Xen PCI Passthrough
From: Avi Kivity <avi@xxxxxxxxxx>
Date: Tue, 04 Oct 2011 20:24:48 +0200
Cc: Anthony Perard <anthony.perard@xxxxxxxxxx>, Alex Williamson <alex.williamson@xxxxxxxxxx>, Xen Devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, QEMU-devel <qemu-devel@xxxxxxxxxx>, Anthony Liguori <anthony@xxxxxxxxxxxxx>
Delivery-date: Tue, 04 Oct 2011 11:29:06 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <alpine.DEB.2.00.1110041853390.3519@kaball-desktop>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1317739882-4809-1-git-send-email-anthony.perard@xxxxxxxxxx> <4E8B1F0D.4080203@xxxxxxxxxx> <4E8B1FBC.2080904@xxxxxxxxxxxxx> <4E8B3C7F.4020300@xxxxxxxxxx> <alpine.DEB.2.00.1110041853390.3519@kaball-desktop>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:6.0.2) Gecko/20110906 Thunderbird/6.0.2
On 10/04/2011 08:19 PM, Stefano Stabellini wrote:
On Tue, 4 Oct 2011, Avi Kivity wrote:
>  On 10/04/2011 05:01 PM, Anthony Liguori wrote:
>  >>  We also have pci passthrough in qemu-kvm (I think based on the same
>  >>  Neocleus
>  >>  code). Rather than having two pci assignment implementations, I think
>  >>  we should
>  >>  have just one, with the differences (programming the hypervisor)
>  >>  abstracted at
>  >>  that level.
>  >
>  >
>  >  I agree in principle but how close is qemu-kvm pci passthrough to a
>  >  mergable state?  Would it make sense to merge the Xen code first and
>  >  then abstract it?
>
>  Merging either implementation and abstracting it would risk regressions
>  in the other.

Honestly the last time I looked at the kvm passthrough code (admittedly
a while ago), it looked very similar to the xen passthrough code, so I
don't think we would risk much merging either one first and then
abstracting it.

There were 59 commits in the past year to hw/device-assignment.c, so the risk is real IMO.

>  How about merging both, with the ABIs (command line and qmp) tagged as
>  experimental, and then doing a merge in the same style as
>  i386+x86_64->x86 or the two kvm implementations in qemu?  We can pick
>  one implementation as the merge target and port fixes from the other.

I am OK with this too: it is probably more work but it doesn't risk
loosing any bug fixes.
If you think that kvm passthrough might have several bug fixes that xen
passthrough does not have is probably the right way to go.

and the other way round.

--
error compiling committee.c: too many arguments to function


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel