On Tue, Oct 09, 2018 at 21:35:20 +0200, Kevin Wolf wrote: > Peter, would this provide what libvirt urgently needs for backing files > vs. the commit block job?
This looks fine for us with one exception. I'd prefer if the curl driver implemented this too so that we don't have to hardcode the readonly flag for that as well. With CURL based backend backing an image I get: -blockdev {"driver":"http","url":"example.com","node-name":"libvirt-2-storage","discard":"unmap","auto-read-only":true}: curl block device does not support writes I've tested it with libvirt's usage of -blockdev together with block-commit and also with read-only-NBD backed images and everything seems to work well for me. I'll report back once I give it more testing.
signature.asc
Description: PGP signature