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] Device duplicate check fix

To: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>, Michal Novotny <minovotn@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Device duplicate check fix
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Tue, 02 Jun 2009 23:37:22 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 02 Jun 2009 15:37:47 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4A255EC5.3050400@xxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acnjpbl/IZxvKM9PRgGoR3ihf6naMgALPgpD
Thread-topic: [Xen-devel] [PATCH] Device duplicate check fix
User-agent: Microsoft-Entourage/12.17.0.090302
It's already reverted, since it broke automated localhost migration tests.

 -- Keir

On 02/06/2009 18:17, "Stefano Stabellini" <stefano.stabellini@xxxxxxxxxxxxx>
wrote:

> Sorry for the late reply, but I only now I realized that this patch
> breaks stubdoms: an exception is needed to allow a disk to be shared
> between the guest and its own stubdom.
> 
> Beside I do not see the need to add this check to xend since the same
> check is already present in the hotplug scripts.
> 
> Michal Novotny wrote:
> 
>> *Hello,
>> I've checked the duplicate-check code here and I found that's checked
>> only in the context of one domain but not cross-domain. The thing is
>> that we should check tap/vbd device cross-domain not to allow another
>> guest to use the same disk image in some circumstances to prevent VM's
>> disk corruption.
>> 
>> The patch included denies disk image addition under those circumstances:
>>  1. **We're adding read-only disk that's already used as write-exclusive
>>  2. **We're adding write-shared disk that's already used as write-exclusive
>>  3. **We're adding write-exclusive disk that's already used
>>  4. **We're adding read-only disk that's already used as write-shared*
>> (because of I/O caching issues etc.)
>> *
>> The vif device duplicate check remains the same it was and it's checked
>> in the context of current domain only so that behaviour has been preserved.
>> 
>> Michal
>> 
>> Signed-off-by: Michal Novotny <minovotn@xxxxxxxxxx>
>> *
>> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



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

<Prev in Thread] Current Thread [Next in Thread>