Am 24.11.2016 um 16:48 hat Pavel Butsykin geschrieben: > On 23.11.2016 18:15, Kevin Wolf wrote: > >Am 15.11.2016 um 07:36 hat Pavel Butsykin geschrieben: > >>+static QemuOptsList runtime_opts = { > >>+ .name = "pcache", > >>+ .head = QTAILQ_HEAD_INITIALIZER(runtime_opts.head), > >>+ .desc = { > >>+ { > >>+ .name = "x-image", > >>+ .type = QEMU_OPT_STRING, > >>+ .help = "[internal use only, will be removed]", > >>+ }, > > > >blkdebug/blkverify have this because they have to support legacy syntax > >like -drive file=blkdebug:blkdebug.conf:test.img, i.e. it has to deal > >with filenames.q > > > >Here we don't have to support a legacy syntax, so I would completely > >avoid this from the beginning. You already support the "image" option, > >which should be good enough. > > Then the command line would look like this: > > -drive > file=/img/harddisk.hdd,if=none,id=drive-scsi0-0-0-0,cache=none,aio=native > -drive driver=pcache,image=scsi0-0-0-0,if=virtio
Yes, either that or with an inline block node definition, the block layer supports both: -drive driver=pcache,image.drive=file,image.filename=/img/harddisk.hdd,if=virtio,cache=none,image.aio=native Kevin