Re: [Qemu-devel] [Bug 726619] [NEW] loadvm does not load (offline) snapshot anymore

[Kevin Wolf]

Am 28.02.2011 17:04, schrieb Ralf Haferkamp:
> Public bug reported:
> 
> qemu Version: 0.14.0
> The problem is present in the current code from git master as well.
> 
> Loading a snapshot that was created while qemu was not running (using
> qemu-img) does not seem to work anymore.

Is there even a use case for this? While an OS is running, just silently
replacing the disk contents isn't going to work. And if you're rebooting
anyway, you can as well load the snapshot offline.

> Using "loadvm <snapshot-id>" in the qemu monitor does not have the
> desired effect. Not even an error message is displayed.
> 
> I was able to track down the problem (using git bisect) to this commit:
> http://git.qemu.org/qemu.git/commit/?id=f0aa7a8b2d518c54430e4382309281b93e51981a
> 
> After reverting that commit in my local git checkout everything is
> workin as expected again.

In fact, that commit only lets loadvm fail earlier. Before, loadvm would
have loaded all disk snapshots and then returned -EINVAL because there
was no VM state, now it fails immediately (the VM is stopped in both
cases because of the failure).

I think the correct fix is to add an error message.

Kevin