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: Michal Novotny <minovotn@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Device duplicate check fix
From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Date: Tue, 2 Jun 2009 18:17:57 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 02 Jun 2009 10:15:22 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4A0D30F7.6010506@xxxxxxxxxx>
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: <4A0D30F7.6010506@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird 2.0.0.14 (X11/20080505)
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

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