From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kamala Narasimhan Subject: Re: [PATCH] xl: Perform minimal validation of virtual disk file while parsing config file Date: Tue, 25 Jan 2011 22:07:19 -0500 Message-ID: References: <1294995912.8240.86.camel@zakaz.uk.xensource.com> <1295024348.12018.222.camel@qabil.uk.xensource.com> <19768.22912.878633.622270@mariner.uk.xensource.com> <19769.31094.274199.464586@mariner.uk.xensource.com> <1295616468.12018.352.camel@qabil.uk.xensource.com> <4D3A4203.2050706@gmail.com> <19775.4623.927492.247420@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: In-Reply-To: <19775.4623.927492.247420@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Jackson Cc: Ian Campbell , "xen-devel@lists.xensource.com" , Gianni Tedesco , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On Tue, Jan 25, 2011 at 1:10 PM, Ian Jackson wrote: > Thanks, I have applied your patch, with a minor tweak to make it work > with Stefano's PHYSTYPE_EMPTY patch. > Ian - I am afraid the validation in its current form is going to fail for vhds with prefix 'tap:aio:vhd:' as I just found out. I will send a patch to fix that. In the mean time, if there is a wiki/doc that lists the different formats that can be passed through the disk option, I can verify all of those so as not to introduce any further regression. Kamala