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: [Xen-devel] Merging PCI pass-through code

To: Guy Zana <guy@xxxxxxxxxxxx>, "Kay, Allen M" <allen.m.kay@xxxxxxxxx>, "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Subject: Re: [Xen-devel] Merging PCI pass-through code
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: Mon, 04 Jun 2007 10:44:37 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 04 Jun 2007 02:42:52 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <9392A06CB0FDC847B3A530B3DC174E7B02ADC595@xxxxxxxxxxxxxxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acel/4aMbeIqTGNjTdSDya+uK3cRjwAjXBGw
Thread-topic: [Xen-devel] Merging PCI pass-through code
User-agent: Microsoft-Entourage/11.3.3.061214
On 3/6/07 17:52, "Guy Zana" <guy@xxxxxxxxxxxx> wrote:

Since both Neocleus' solution & Intel's solution have overlapping implementations, we should determine what should stays and what goes.
There is a question on how should we do the merge?

I like the idea of a separate tree, just for the short term, as a shared workspace for merging and development and bug fixing. I can arrange to create a new repo called xen-hvm-directio.hg (or a better name if you can think of one)? From the list of points below it sounds like both sets of patches need some work, quite apart from the required merging. Would this method of development suit everyone?

 -- Keir

Here are a number of issues that we should address:

1. We should really create a separate tree and have the merging done outside of the main unstable tree.
2. Neocleus will use your configuration interface to assign pci devices.
3.1 The lpci library - I think it is best to merge Intel's code with what we have in our implementation of libpci, and you can add your functions to our library.
3.2 Your implementation doesn't read/write to the real PCI config space, I'm not sure that all devices would like that :)
4. Pass-through initialization should be done regardless of an iommu present.
5. What type of interrupt handling is the way to go? I can't compare the polarity-change with your method since I don't have an IOMMU machine...
6. Does the PIO/MMIO access functions in qemu-dm (Neocleus') are needed? (It's good for debugging)
7. The 1:1 mapping and specific-iommu code can be merged separately.
 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>