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

[Xen-devel] Re: [PATCH] libxlu: correctly parse disk "backendtype" field



On 10/07/2011 05:41 PM, Ian Campbell wrote:
> On Fri, 2011-10-07 at 21:50 +0100, Daniel De Graaf wrote:
>> The expression for backendtype was missing a comma.
> 
> This appears correct even though things seem to work without, at least:
> xl -N block-attach 0 
> backendtype=phy,vdev=xvdb,access=w,target=/dev/vg/guest-volume
> works as expected. Do you have a test case which fails or did you just
> find by code inspection?
> 
> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> 
> (we need a signed-off-by I suspect)
> 

Signed-off-by: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>

Found when xl was parsing a configuration file with the disk line:

disk = [ 'backendtype=phy,backend=3,vdev=xvda,access=w,target=/dev/clam/vm1' ]

This is after adding another patch allowing backend domain ID for disk/vif to
be changed in the config, so it may or may not show up on its own; I didn't
test the new syntax until I needed it.

>>
>> --- a/tools/libxl/libxlu_disk_l.l
>> +++ b/tools/libxl/libxlu_disk_l.l
>> @@ -139,7 +139,7 @@ devtype=disk,?   { DPC->disk->is_cdrom = 0; }
>>  devtype=[^,]*,?     { xlu__disk_err(DPC,yytext,"unknown value for type"); }
>>  
>>  access=[^,]*,?      { STRIP(','); setaccess(DPC, FROMEQUALS); }
>> -backendtype=[^,]*? { STRIP(','); setbackendtype(DPC,FROMEQUALS); }
>> +backendtype=[^,]*,? { STRIP(','); setbackendtype(DPC,FROMEQUALS); }
>>  
>>  vdev=[^,]*,?        { STRIP(','); SAVESTRING("vdev", vdev, FROMEQUALS); }
>>  script=[^,]*,?      { STRIP(','); SAVESTRING("script", script, FROMEQUALS); 
>> }
> 

_______________________________________________
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®.