On Wed, Jun 16, 2010 at 9:40 AM, Kevin Wolf <kw...@redhat.com> wrote: > > If the human monitor was exactly what its name says, I'd happily apply > this one (though I think it should be made clear from which image the VM > state would be loaded). However, it isn't and I'm not sure if this > wouldn't break libvirt. Dan, can you help? >
I didn't mention in the commit, but I've looked at libvirt's source and it is not using 'info snapshots' AFAIK. At the present time, the VM state is always saved in the first block device that supports snapshots. I could update the patch to make it clear on the output somehow. From savevm.c:get_bs_snapshot(): static BlockDriverState *get_bs_snapshots(void) { BlockDriverState *bs; if (bs_snapshots) return bs_snapshots; /* FIXME what if bs_snapshots gets hot-unplugged? */ bs = NULL; while ((bs = bdrv_next(bs))) { if (bdrv_can_snapshot(bs)) { goto ok; } } return NULL; ok: bs_snapshots = bs; return bs; } Regards, Miguel