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/
Home Products Support Community News


Re: [Xen-devel] [PATCH] xl: Perform minimal validation of virtual disk f

To: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xl: Perform minimal validation of virtual disk file while parsing config file
From: Kamala Narasimhan <kamala.narasimhan@xxxxxxxxx>
Date: Fri, 14 Jan 2011 09:55:58 -0500
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 14 Jan 2011 06:56:34 -0800
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=ZypPA+IFlKyjFfpdt7BI9Cyu21YiLG/uENsOmFJ1EIk=; b=svST+ipKrWFbI2fXvosM3eZPa9wlOEz3ETrsJGpl6EM7+CfZHUl2jwMvny1HtSpudZ qPR+4B+vPx97eOOy2WW4Rk9PYPeliOy96b3lxZRv0ooxd3SewXEM+tUDdZpyrF7AgAAZ lignYHdKo7b+XZ7LWCbQjPNLXYf5yBdP60qhw=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=p3VEqmvs1IeYy4/V/vRsxObuh0qT77BhP1dOdCAnZl7gxo5AbuxkknrqOXL2Mk2wPa zimBDjN3/QCWg51i53I0tTOORC/985WuUk8ldGMvJV9XXjVhlcmTNef3IKjlyaUU5IBV jpYwI5RCuY7zdNWWbgPdYJE+RLrsHzGbDtuoM=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1294995912.8240.86.camel@xxxxxxxxxxxxxxxxxxxxxx>
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: <AANLkTimSUym0u+SNm0AvNp3AZQQFspetAaXmNShkPJd4@xxxxxxxxxxxxxx> <1294995912.8240.86.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Fri, Jan 14, 2011 at 4:05 AM, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> On Thu, 2011-01-13 at 15:35 +0000, Kamala Narasimhan wrote:
>> This patch performs some very basic validation on the virtual disk
>> file passed through the config file.  This validation ensures that we
>> don't go too far with the initialization like spawn qemu and more
>> while there could be some potentially fundamental issues.  Obviously,
>> there is a lot of room for improvement in the kind of validations we
>> could do but the below is a minimal first stab at it.  Please consider
>> this for inclusion or feel free to tweak it as necessary.
>> Signed-off-by: Kamala Narasimhan <kamala.narasimhan@xxxxxxxxxx>
> I wonder if the validation function should be part of libxl?

We are better off performing these checks early on as they are very
basic.  Getting far enough to spawn qemu and getting to its block
device initialization code and failing there is a bit of a chase when
it comes to troubleshooting these issues, the cause of which are
rather trivial.  That said, in the long run we might want to move
these validations to upstream qemu as qemu also must perform these
checks especially when run without an accelerator (as there wouldn't
be a toolstack to perform these checks for it in that case).  But,
till that is accomplished these checks need to be somewhere and libxl
seem like a reasonable place in my opinion.


Xen-devel mailing list

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