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] Don't allow assigning the same device twice

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [VTD][PATCH] Don't allow assigning the same device twice without release the first assignment
From: "Han, Weidong" <weidong.han@xxxxxxxxx>
Date: Tue, 16 Oct 2007 14:53:28 +0800
Cc: "Kay, Allen M" <allen.m.kay@xxxxxxxxx>
Delivery-date: Mon, 15 Oct 2007 23:55:10 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C33A1B9C.EEFE%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: <08DF4D958216244799FC84F3514D70F00ABAD0@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C33A1B9C.EEFE%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcgPPrZDfEJfzCOtSJyuGbCE3/cnAQAAjKktABVSMsAAClbYKwAAQyzQ
Thread-topic: [Xen-devel] [VTD][PATCH] Don't allow assigning the same device twice without release the first assignment
Keir Fraser wrote:
> On 16/10/07 03:04, "Han, Weidong" <weidong.han@xxxxxxxxx> wrote:
> 
>>> The DOMCTL_assign_device should check whether the device is already
>>> assigned. This has the benefit that it can atomically
>>> check-and-allocate, under the domctl lock.
>>> 
>>> You'll have to work out how to propagate DOMCTL_assign_device
>>> failure to the user. Either you have to get the error out of ioemu,
>>> or perhaps you can assign the device in xend before starting ioemu.
>> 
>> Yes, adding the check on DOMCTL_assign_device is simple, but I didn't
>> find a good way to propagate DOMCTL_assign_device failure to user.
>> This patch adds the check in xend before starting ioemu. In addtion,
>> adding the check in Xend can prompt user on the screen when the
>> device has already been assigned. Thanks.
> 
> Well, that's too bad because I won't take the original patch. Why
> can't the device be assigned by xend?
> 
>  -- Keir

Ok, I will try your suggestion. 

-- Weidong

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