Re: [Qemu-devel] [PATCH v2] sheepdog: fix loadvm operation

2013-04-24 Thread Liu Yuan
On 04/25/2013 12:47 AM, Liu Yuan wrote: > - don't break old behavior if we boot up on the snapshot by using s->reverted >to indicate if we delete working VDI successfully If we implement 'boot from snapshot' == loadvm snapshot, we don't need s->reverted. What do you think, Kazutaka? With this

[Qemu-devel] [PATCH v2] sheepdog: fix loadvm operation

2013-04-24 Thread Liu Yuan
From: Liu Yuan Currently the 'loadvm' opertaion works as following: 1. switch to the snapshot 2. mark current working VDI as a snapshot 3. rely on sd_create_branch to create a new working VDI based on the snapshot This works not the same as other format as QCOW2. For e.g, qemu > savevm # get a