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 2/2 v2] graphics passthrough with VT-d

To: 'Ian Jackson' <Ian.Jackson@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 2/2 v2] graphics passthrough with VT-d
From: "Han, Weidong" <weidong.han@xxxxxxxxx>
Date: Wed, 16 Sep 2009 13:23:33 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "'xen-devel@xxxxxxxxxxxxxxxxxxx'" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "Lin, Ben Y" <ben.y.lin@xxxxxxxxx>, "Kay, Allen M" <allen.m.kay@xxxxxxxxx>, Guyader' <jean.guyader@xxxxxxxxxxxxx>, 'Jean, "'Keir.Fraser@xxxxxxxxxxxxx'" <Keir.Fraser@xxxxxxxxxxxxx>, "'bengheng@xxxxxxxxxxxxxx'" <bengheng@xxxxxxxxxxxxxx>
Delivery-date: Tue, 15 Sep 2009 22:25:54 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <715D42877B251141A38726ABF5CABF2C054B5B45FF@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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: <715D42877B251141A38726ABF5CABF2C054B533C43@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <19105.18480.7844.905693@xxxxxxxxxxxxxxxxxxxxxxxx> <715D42877B251141A38726ABF5CABF2C054B5B45FF@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcotgYq6Wx2WNJWARf6/le86zJcViwBHbDyAAft73mA=
Thread-topic: [Xen-devel] [PATCH 2/2 v2] graphics passthrough with VT-d
Ian,

Did you have a look at this patch? What's your opinion on it? Thanks.


Regards,
Weidong

-----Original Message-----
From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx 
[mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Han, Weidong
Sent: 2009年9月6日 13:25
To: 'Ian Jackson'
Cc: 'xen-devel@xxxxxxxxxxxxxxxxxxx'; Lin, Ben Y; Kay, Allen M; 'Jean Guyader'; 
'Keir.Fraser@xxxxxxxxxxxxx'; 'bengheng@xxxxxxxxxxxxxx'
Subject: RE: [Xen-devel] [PATCH 2/2 v2] graphics passthrough with VT-d

Ian Jackson wrote:
> Han, Weidong writes ("[Xen-devel] [PATCH 2/2 v2] graphics passthrough
> with VT-d"): 
>> This patch supports basic gfx passthrough on QEMU:
>>   - disable emulated VGA adpater if there is passthroughed gfx
>>   - retrieve VGA bios from host 0xC0000, then load it to guest
>> 0xC0000 
>>   - register/unregister legacy VGA I/O ports and MMIOs for
>> passthroughed gfx 
> 
> Thanks for this.  However:
> 
>  * It breaks the build if passthrough is disabled (eg due to lack
>    of pciutils-dev).

It won't break the build when lack of pciutils-dev for me.

> 
>  * Reindenting all of the code in pc.c which initalises out the
>    various graphics cards is very merge-unfriendly.  Is that change
>    needed at all, or could you just disable the various graphics cards
>    on the qemu command line ?
> 
>  * Likewise, in vl.c.

Ok. I updated the code to add a new vga type for gfx passthrough. Then it 
needn't to add the check before various vga initialization in pc.c and vl.c. 
Attached the updated patch. Pls review it. Thanks.

Regards,
Weidong

> 
> Regards,
> Ian.

Attachment: qemu-gfx-passthrough-v3.patch
Description: qemu-gfx-passthrough-v3.patch

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