Re: [PATCH 1/3] virDomainCreateXML: support new flag

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

 



On 06/08/2010 02:42 PM, Jiri Denemark wrote:
>> diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in
>> index 19d5205..c806441 100644
>> --- a/include/libvirt/libvirt.h.in
>> +++ b/include/libvirt/libvirt.h.in
>> @@ -121,7 +121,8 @@ typedef virDomainInfo *virDomainInfoPtr;
>>   * Domain.
>>   */
>>  typedef enum {
>> -     VIR_DOMAIN_NONE = 0
>> +    VIR_DOMAIN_NONE         = 0,      /* Default behavior */
>> +    VIR_DOMAIN_START_PAUSED = 0 << 1, /* Launch guest in paused state */
>>  } virDomainCreateFlags;
> 
> I believe, you wanted to write 1 << 0 as a value of VIR_DOMAIN_START_PAUSED,
> didn't you?

Blame the fact that I was traveling yesterday?  Yes, a bone-headed bug
on my part; thanks for catching it, and thank goodness for the review
process.

> 
> The rest of this patch looks just fine to me, ACK.

Fixed, and will push as soon as I actually complete some testing on 2/3
and 3/3 (which would have prevented posting this bug in the first place).

> 
> Jirka
> 

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]