ä 2011å04æ07æ 10:21, Eric Blake åé:
On 04/06/2011 08:01 PM, Osier Yang wrote:
managed_save = qemuDomainManagedSavePath(driver, vm);
if ((managed_save)&& (virFileExists(managed_save))) {
If managed_save is NULL, then we should be skipping to cleanup
(qemuDomainManagedSavePath already reported OOM), rather than silently
falling back to normal startup.
No, qemuDomainObjStart is also used by qemuDomainStartWithFlags,
skipping to cleanup when managed_save is NULL will break the
starting of all domains which don't have managed state file.
That's risky.
Ignoring OOM is risky. If managed_save is NULL, we are out of memory,
and we should fail the command; that's the only time that
qemuDomainManagedSavePath returns NULL. We don't know if there was a
save file or not. And even if there was not a file, we'd probably run
out of memory again if we attempt normal setup.
Oh, I misunderstand your meaning here, forget about it, v5 comes. :)
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list