[libvirt] [PATCH v5] qemu: Remove the managed state file only if restoring succeeded

Eric Blake eblake at redhat.com
Fri Apr 8 14:20:53 UTC 2011


On 04/07/2011 07:09 PM, Osier Yang wrote:
> 于 2011年04月07日 22:56, Eric Blake 写道:
>> Sorry for not noticing sooner, but we could do some grammar cleanups.
>>
>> Does this look like a reasonable followup?
>>
>> diff --git i/src/qemu/qemu_driver.c w/src/qemu/qemu_driver.c
>> index a84780b..0734a76 100644
>> --- i/src/qemu/qemu_driver.c
>> +++ w/src/qemu/qemu_driver.c
>> @@ -3430,7 +3430,7 @@ static int qemudDomainObjStart(virConnectPtr conn,
>>       if (!managed_save)
>>           goto cleanup;
>>
>> -    if ((managed_save)&&  (virFileExists(managed_save))) {
>> +    if (virFileExists(managed_save)) {
>>           ret = qemuDomainObjRestore(conn, driver, vm, managed_save);
>>

>> +Restores a domain from a B<virsh save>  state file. See I<save>  for
>> more
>> info.
>>
>>   B<Note>: To avoid corrupting file system contents within the domain,
>> you
>> -should not reuse the saved state file to B<restore>  unless you are
>> convinced
>> -with reverting the domain to the previous state.
>> +should not reuse the saved state file for a second B<restore>  unless
>> +you have also reverted all storage volumes back to the same contents
>> +as when the state file was created.
> 
> Doesn't reuse implies the "second, third, ...." B <restore>? :)
> 
> But anyway, it looks more clear, good to me.

Thanks; pushed.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110408/6d7b7b7a/attachment-0001.sig>


More information about the libvir-list mailing list