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@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org