[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [xen-devel][PATCH] xl - Special case vhd images validation


  • To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
  • From: Kamala Narasimhan <kamala.narasimhan@xxxxxxxxx>
  • Date: Thu, 10 Feb 2011 14:55:34 -0500
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Thu, 10 Feb 2011 11:56:22 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; b=hztDpCKrRYQNJ0fzybfatvc1u4cEU+VudmXBb6cbCk0IUcij8aHXhrs6gpOO56RgA7 h/k4CJcMBRQmDO5G8vfStHyTouAhDWq5bu6AKW8FHUUPRBegOcVYDj9WGeNeBTJceSO3 2Trqk/NLy1cKQQa1k0KnqrtSDpTeHp6bSieeY=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Ian Jackson wrote:
> Kamala Narasimhan writes ("[xen-devel][PATCH] xl - Special case vhd images 
> validation"):
>> Description - Special case how we validate vhd image files.  Without this 
>> patch when tap:aio:vhd prefixed image files are specified in the config 
>> file, disk validation and thus vm creation will fail.
> 
>> +    if (disk->format == DISK_FORMAT_RAW) {
>> +        delimiter = strchr(file_name, ':');
>> +        if (delimiter) { 
>> +            if (!strncmp(file_name, "vhd:", sizeof("vhd:")-1)) {
>> +                disk->format = DISK_FORMAT_VHD; 
>> +                file_name = ++delimiter;
>> +            }
>> +        }
>> +    }
> 
> This seems to suggest that if you say "tap:raw:" with a filename
> starting "vhd:" you get vhd instead.  This is a bit perverse but I
> guess acceptable for 4.1 at least.
>
We won't need this change if we have gone with patch 3 and further but since we
chose to move those post 4.1, we will need this.

> 
>>          LIBXL__LOG_ERRNO(ctx, LIBXL__LOG_ERROR, "failed to stat %s", 
>> file_name);
>>          return ERROR_INVAL;
>>      }
>> -    if (backend_type == DISK_BACKEND_PHY) {
>> +    if (disk->backend == DISK_BACKEND_PHY) {
> 
> Is this change stray in this patch, or should it be included in one of
> the others ?  I'm not sure I follow it, anyway.
> 

I switched the input param for the method as I had to, so we need this as well.

Kamala


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.