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] block-nbd script and example config

To: Nicholas Lee <emptysands@xxxxxxxxx>
Subject: Re: [Xen-devel] block-nbd script and example config
From: Anthony Liguori <aliguori@xxxxxxxxxx>
Date: Wed, 19 Oct 2005 22:25:13 -0500
Cc: Ian Pratt <m+Ian.Pratt@xxxxxxxxxxxx>, Xen development list <xen-devel@xxxxxxxxxxxxxxxxxxx>, Kurt Garloff <garloff@xxxxxxx>
Delivery-date: Thu, 20 Oct 2005 03:22:39 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <2b6116b30510191752w41c8f88bq11937c9d1688f454@xxxxxxxxxxxxxx>
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: <A95E2296287EAD4EB592B5DEEFCE0E9D32E4B9@xxxxxxxxxxxxxxxxxxxxxxxxxxx> <4356C389.1080804@xxxxxxxxxx> <2b6116b30510191752w41c8f88bq11937c9d1688f454@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla Thunderbird 1.0.7 (X11/20051013)
Nicholas Lee wrote:

The only thing to think about is how do handle file devices with spaces
in the filename.  This is going to be common I think.

Since it's a python variable can't you just run a validator on config
import and throw an exception if the devive block doesn't exist. Noting that device paths including special characters need to be
enclosured in "...".

ie. export=phy:"/dev/vgname/bob your uncle; mate! device", ...
Double quotes are actually valid characters in a filename although rarely used.

A more bold way of approaching this would be to adopt a more qemu like syntax of:

hda1="/dev/vgname/bob you uncle; mate1"
hda2="/root/rockhopper.img"

We could use stat to figure out whether something was a file or a device and use the appropriate script (file or phy). For everything else, we could use explicit urls such as:

hda3="nbd://rockhopper.austin.ibm.com:1025"

I like the syntax a lot but I'm not sure that marking a device read-only works nicely anymore. We could have a single readonly attribute like:

readonly=['hda1', 'hda2']

Or perhaps just require url encoding:

hda1="phy:///dev/vgname/bob you uncle&readonly=1"

Kind of ugly but at least we're optimizing for the common case.

Just thinking out loud.

Regards,

Anthony Liguori

--
Nicholas Lee
http://stateless.geek.nz
gpg 8072 4F86 EDCD 4FC1 18EF  5BDD 07B0 9597 6D58 D70C



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