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] [VTD-NEO][patch 0/6] Intel VT-d/Neocleus 1:1 mreged code

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [VTD-NEO][patch 0/6] Intel VT-d/Neocleus 1:1 mreged code for PCI passthrough
From: "Kay, Allen M" <allen.m.kay@xxxxxxxxx>
Date: Wed, 12 Sep 2007 15:37:47 -0700
Cc: Guy Zana <guy@xxxxxxxxxxxx>
Delivery-date: Wed, 12 Sep 2007 15:38:19 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C30D69FC.156C6%Keir.Fraser@xxxxxxxxxxxx>
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>
References: <13A2F7DE1BAEA345A61DD40F303ED7A6742320@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C30D69FC.156C6%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcfvUPA+fwPVGW6LTE+QIE4aSL5IWQFIU1UoAACFqk4AGZTrkAAQSe9xABwCZ6A=
Thread-topic: [Xen-devel] [VTD-NEO][patch 0/6] Intel VT-d/Neocleus 1:1 mreged code for PCI passthrough
I have added CONFIG_PASSTHROUGH in ioemu/Makefile.target and
ioemu/hw/pc.c in attached vtd_tools2.patch.  This should turn off libpci
usage by default until user specifically enables it.

This can be safely check-in without breaking builds for people who do
not care about pass-through devices.  I will try to think of a better
way to enable this.

Also, can you check-in vtd_dir.patch if you don't have any objections?
This is a separate vt-d specific directory in hvm/vmx/vtd.  It should
not interfere with anything else.

I'm current working on getting xen.patch part to work with latest
staging tree.  I will send it in as soon as I get it working.

Allen

Signed-off-by: Allen Kay <allen.m.kay@xxxxxxxxx>
Signed-off-by: Guy Zana <guy@xxxxxxxxxxxx>

>-----Original Message-----
>From: Keir Fraser [mailto:Keir.Fraser@xxxxxxxxxxxx] 
>Sent: Wednesday, September 12, 2007 2:04 AM
>To: Kay, Allen M; xen-devel@xxxxxxxxxxxxxxxxxxx
>Cc: Guy Zana
>Subject: Re: [Xen-devel] [VTD-NEO][patch 0/6] Intel 
>VT-d/Neocleus 1:1 mreged code for PCI passthrough
>
>Vtd_tools patch breaks the build on my box. Cannot find pci/header.h or
>pci/pci.h. Probably because libpci is not such a standard 
>install component
>after all? I'm trying to build on Debian 3.1.
>
>Either the dependency needs to be avoided, or we need to be 
>sure that the
>dependency is easy to resolve on most distros, and we need a 
>tools/check
>script to check for and warn/error on the dependency.
>
> -- Keir
>
>On 12/9/07 02:21, "Kay, Allen M" <allen.m.kay@xxxxxxxxx> wrote:
>
>> Attached patches splits vtd and neo changes to tools 
>directory.  Applies
>> cleanly to staging tree.
>> 
>> Vtd_tools.patch: vt-d and generic changes
>> Neo_tools.patch: neocleus specific changes
>> 
>> Signed-off-by: Allen Kay <allen.m.kay@xxxxxxxxx>
>> Signed-off-by: Guy Zana <guy@xxxxxxxxxxxx>
>> 
>>> -----Original Message-----
>>> From: Keir Fraser [mailto:Keir.Fraser@xxxxxxxxxxxx]
>>> Sent: Tuesday, September 11, 2007 6:05 AM
>>> To: Kay, Allen M; xen-devel@xxxxxxxxxxxxxxxxxxx
>>> Cc: Guy Zana
>>> Subject: Re: [Xen-devel] [VTD-NEO][patch 0/6] Intel
>>> VT-d/Neocleus 1:1 mreged code for PCI passthrough
>>> 
>>> On 11/9/07 13:49, "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx> wrote:
>>> 
>>>> Haven't looked any further at the Xen parts, but I'll take a
>>> look at the
>>>> tools patch...
>>> 
>>> Looking at the tools patch, I see strange interfaces like
>>> HVMOP_copy_nativedom_e820_map:
>>> 1. Why is this an hvm_op unlike other added domctls?
>>> 2. Why is it needed at all? Can't xc_hvm_build.c work out the
>>> memory map
>>> for itself? It seems like more than necessary is being done in Xen.
>>> 
>>> Looks like splitting solely pt and solely vtd code into
>>> separate patches
>>> would be a good idea, so the more acdeptable chunks can slide
>>> straight in
>>> without delay.
>>> 
>>> -- Keir
>>> 
>

Attachment: vtd_tools2.patch
Description: vtd_tools2.patch

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