[Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2022-05-31 Thread Michael Tokarev
Internal implementation details aside, from the user PoV it is a *very* serious issue. If -snapshot can't be applied automatically, maybe qemu should warn or better fail if -snapshot is used together with -blockdev. -- You received this bug notification because you are a member of qemu- devel-ml,

Re: [Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-02-02 Thread Ildar
this answers the whole question. Thanks a lot. closing ** Changed in: qemu Status: New => Invalid -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1860759 Title: [REGRESSION] option `-snapsho

[Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-01-31 Thread Max Reitz
Hi, The technical difference is that -blockdev requires you (the user or management software) to create all block graph nodes explicitly. -drive snapshot=on implicitly creates a qcow2 node above the actual disk image (and that node points to a temporary image in /tmp). So because it’s implicit a

Re: [Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-01-30 Thread Ildar
Thank a lot for the detailed answer. Surely it's worth discussing qemu here leaving libvirt for RH bugzilla. > But since modern qemu has declared -snapshot to be unsupported with -blockdev, and modern libvirt has switched to -blockdev, I claim that this is not a qemu bug, but a libvirt feature req

Re: [Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-01-24 Thread Eric Blake
On 1/24/20 4:41 AM, Ildar wrote: Max, thanks a lot for the explanation. Do you mean that snapshot-ing isn't possible totally for blockdev? Then I guess some libvirt users are in trouble :(( Actually I didn't quite caught the reason why a blockdev supports backing but not {backing to a file on /tm

Re: [Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-01-24 Thread Ildar
Max, thanks a lot for the explanation. Do you mean that snapshot-ing isn't possible totally for blockdev? Then I guess some libvirt users are in trouble :(( Actually I didn't quite caught the reason why a blockdev supports backing but not {backing to a file on /tmp then promptly deleted} ? What's t

[Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-01-24 Thread Max Reitz
Hi, I don’t know much about libvirt, but I would have thought that any manual modification of the qemu command line isn’t supported and might always break. Anyway, from a QEMU POV, -snapshot only works with -drive (this includes -hda, etc.). It doesn’t work with -blockdev. I can see that this i

[Bug 1860759] Re: [REGRESSION] option `-snapshot` ignored with blockdev

2020-01-24 Thread Ildar
JFYI, I know that snapshot=on option should be used. But `-snapshot` option exists and must work. Also libvirt doesn't yet support it: https://bugzilla.redhat.com/show_bug.cgi?id=508662 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU