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] [PATCH] Support Integrated graphics passthrough

To: 'Ian Jackson' <Ian.Jackson@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Support Integrated graphics passthrough
From: "Han, Weidong" <weidong.han@xxxxxxxxx>
Date: Mon, 12 Oct 2009 15:19:18 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: 'Keir, "'xen-devel@xxxxxxxxxxxxxxxxxxx'" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "Kay, Allen M" <allen.m.kay@xxxxxxxxx>, Fraser' <Keir.Fraser@xxxxxxxxxxxxx>, 'Jean Guyader' <Jean.Guyader@xxxxxxxxxx>
Delivery-date: Mon, 12 Oct 2009 00:21:02 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19148.46038.692908.257685@xxxxxxxxxxxxxxxxxxxxxxxx>
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: <715D42877B251141A38726ABF5CABF2C05507CF993@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <19148.46038.692908.257685@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcpHYvtWr6lBX7GqSXaKj21MFRu9+gDp2gDw
Thread-topic: [Xen-devel] [PATCH] Support Integrated graphics passthrough
Hi Ian,

Thanks for pointing it out. 

pass-through.h references files of pciutils-dev package, so it shouldn't be 
included when pciutils-dev is not installed. Fixed it by checking 
CONFIG_PASSTHROUGH before including pass-through.h. Attached the updated patch. 
Thanks.

Regards,
Weidong

Ian Jackson wrote:
> Han, Weidong writes ("[Xen-devel] [PATCH] Support Integrated graphics
> passthrough"): 
>> Passthroughed IGD must use 00:02.0 in guest like in native, otherwise
>> the driver may fail to operate the IGD. This patch reserves the
>> 00:02.0 in guest and assigns it IGD automatically.
> 
> This breaks the build if passthrough is disabled due to pciutils-dev
> being missing:
> 
> Ian.
> 
> get-path: will use #!/usr/bin/python2.5 for python programs
> xen-hooks.mak:56: === pciutils-dev package not found - missing
> /usr/include/pci 
> xen-hooks.mak:57: === PCI passthrough capability has been disabled
> get-path: will use #!/usr/bin/python2.5 for python programs
> make[1]: Entering directory `/u/iwj/work/1/qemu-iwj.git/i386-dm'
> ../xen-hooks.mak:56: === pciutils-dev package not found - missing
> /usr/include/pci ../xen-hooks.mak:57: === PCI passthrough capability
> has been disabled ../xen-hooks.mak:56: === pciutils-dev package not
> found - missing /usr/include/pci ../xen-hooks.mak:57: === PCI
>   passthrough capability has been disabled CC    i386-dm/vl.o
>   CC    i386-dm/pci.o
>   CC    i386-dm/xenstore.o
> In file included from /u/iwj/work/1/qemu-iwj.git/hw/pci.c:33:
> /u/iwj/work/1/qemu-iwj.git/hw/pass-through.h:23:24: error:
> pci/header.h: No such file or directory
> /u/iwj/work/1/qemu-iwj.git/hw/pass-through.h:24:21: error: pci/pci.h:
> No such file or directory  
> In file included from /u/iwj/work/1/qemu-iwj.git/hw/pci.c:33:
> /u/iwj/work/1/qemu-iwj.git/hw/pass-through.h:397: error: expected
> '=', ',', ';', 'asm' or '__attribute__' before 'pt_pci_base_addr'
> /u/iwj/work/1/qemu-iwj.git/hw/pass-through.h:407: error: expected
> '=', ',', ';', 'asm' or '__attribute__' before
> 'pt_pci_host_read_byte'
> /u/iwj/work/1/qemu-iwj.git/hw/pass-through.h:408: error: expected
> '=', ',', ';', 'asm' or '__attribute__' before
> 'pt_pci_host_read_word' [etc]    

Attachment: 0001-IGD-passthrough-support-v2.patch
Description: 0001-IGD-passthrough-support-v2.patch

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