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][PATCH] Do FLR of assigned devices with VT-d

To: "Han, Weidong" <weidong.han@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [VTD][PATCH] Do FLR of assigned devices with VT-d
From: "Guy Zana" <guy@xxxxxxxxxxxx>
Date: Tue, 29 Jan 2008 13:39:06 -0500
Cc: "Xu, Anthony" <anthony.xu@xxxxxxxxx>, "Kay, Allen M" <allen.m.kay@xxxxxxxxx>
Delivery-date: Tue, 29 Jan 2008 10:40:32 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <08DF4D958216244799FC84F3514D70F0FDCB13@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/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: AcheMj+WzZYuIsWFQ72XRyo60xmE4QBU40UgAFZy1MAAcLn1QA==
Thread-topic: [Xen-devel] [VTD][PATCH] Do FLR of assigned devices with VT-d
Hi,

Oh, now I see why you did that. I'm eager to test the PCI hotplug stuff :-)

The only thing that is concerning me is that it becomes VT-d specific.
And I still think that the hypervisor is a bad place to put that code...

Thanks,
Guy.

> -----Original Message-----
> From: Han, Weidong [mailto:weidong.han@xxxxxxxxx] 
> Sent: Sunday, January 27, 2008 2:38 PM
> To: Guy Zana; xen-devel@xxxxxxxxxxxxxxxxxxx
> Cc: Xu, Anthony; Kay, Allen M
> Subject: RE: [Xen-devel] [VTD][PATCH] Do FLR of assigned 
> devices with VT-d
> 
> Hi Guy,
> 
> If you use "xm destroy" or click "x" on qemu window to 
> destroy guest, the assigned device may be stayed in a 
> incorrect status. We met that the irq of the assigned device 
> may be disabled by dom0 after destroy guest, then all the 
> devices using the same irq can't work any more. We also met 
> guest complained the device was not in a correct status 
> during its booting period.
> 
> We think there are three places where need invoking FLR 
> function. The first place is before device assignment, the 
> second one is after exit/destroy guest, the last one is 
> dynamic reassignment after support vtd hotplug. In the latter 
> two cases, it is hard to catch the point to do FLR in qemu.
> 
> Randy (Weidong)
> 
> Guy Zana wrote:
> > That's a good addition, why moving this functionality to the 
> > hypervisor? BTW, can you give real world example where this was 
> > needed?
> > 
> > Thanks,
> > Guy.
> > 
> >> -----Original Message-----
> >> From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
> >> [mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Han, 
> >> Weidong
> >> Sent: Thursday, January 24, 2008 4:39 AM
> >> To: xen-devel@xxxxxxxxxxxxxxxxxxx
> >> Cc: Xu, Anthony; Kay,Allen M
> >> Subject: [Xen-devel] [VTD][PATCH] Do FLR of assigned devices with 
> >> VT-d
> >> 
> >> Currently there is a pdev_flr() function to do FLR before device 
> >> assignment in qemu, but most of devices don't have FLR capability. 
> >> What's more, should do FLR before assignment and deassignment for 
> >> keeping correct device status. If the device doesn't have FLR 
> >> capablility, this patch implemented to enter D3hot and 
> return to D0 
> >> to do FLR. And exposed pdev_flr() in VT-d utils, then it can be 
> >> invoked by assignment and deassignment functions.
> >> 
> >> Signed-off-by: Weidong Han <weidong.han@xxxxxxxxx>
> >> Signed-off-by: Anthony Xu <anthony.xu@xxxxxxxxx
> 

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