Re: -drive if=none: can't we make this the default?

[Kevin Wolf]

Hi Michael,

I didn't see this thread when you posted it, sorry for being a bit late.

Am 16.10.2023 um 13:58 hat Michael Tokarev geschrieben:
> Almost everyone mentions -blockdev as a replacement for -drive.

More specifically for -drive if=none. I honestly don't know many common
use cases for that one.

For management tools, -blockdev is indeed what should be used, and that
things are more explicit there is actually a feature, not a bug, for
management tools.

As a human user, in the common case where I don't care about the
details, I don't want to type up an explicit -device. if=virtio gives me
more directly what I want.

So you only need it when you want to specify one of the more exotic
-device options, which shouldn't happen that often. Well, it doesn't for
me anyway, other people may have other use cases. Is that your case? If
so, which options do you usually want to give to -device?

> But I have to remind several issues with it:
> 
> 1. While documentation has improved a lot, -blockdev is still mostly unknown
>    to the masses.

And for manual human use, that's okay anyway - you probably don't want
to use it. But if you're writing scripts or even advanced management
software, then you should use it.

(Of course, in complex cases you may have to use it manually anyway
because -drive has some limitations, but that should be the absolute
exception.)

> 2. -blockdev is just too verbose, one have to specify a lot of parameters 
> just to
>    do a simple thing which is solved with an extra parameter with -drive.
>    Including various backing stores/chains for qcow2 files - this is terrible 
> for
>    using things manually from command line

You don't have to specify the backing chain explicitly if you're happy
with the default options with which the backing files are opened.

-blockdev options are typically a bit longer than -drive ones, but not
extremely. The separate -device that if=none gives you is already a
similar amount of extra typing.

-drive if=virtio,file=test.qcow2
-drive if=none,file=test.qcow2,id=disk -device virtio-blk,drive=disk
-blockdev qcow2,file.driver=file,file.filename=test.qcow2,node-name=disk 
-device virtio-blk,drive=disk

> 3. -blockdev does not work with -snapshot
> 
> 4. Something else I forgot while typing all the above :)
> 
> In my view, -blockdev is not a substitute for -drive, not at all, and it is
> very user-unfriendly.  This is why -drive seems to be a good trade-off between
> things like -hda (which is just too simplistic) and -blockdev which which is
> way too verbose and lacks some automatic sugar like -snapshot.

I would agree with that, but if=none already feels a bit unfriendly.

Honestly, I would like to throw away the existing -drive and replace it
with one that has a simpler implementation (as a wrapper around
-blockdev) and I would be happy if it gained some additional options for
passing through things to -device so that you don't have to bother with
if=none even in the more complex cases any more.

It would be pure syntactic sugar with a similar compatibility promise as
in HMP (we won't break it just for fun, but we'll also not stop making
sensible changes just because they make things look a bit different).

Kevin