All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Tokarev <mjt@tls.msk.ru>
To: Gleb Natapov <gleb@redhat.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>, qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] ahci drive: how to make it non-bootable?
Date: Wed, 09 May 2012 20:10:07 +0400	[thread overview]
Message-ID: <4FAA96DF.3090006@msgid.tls.msk.ru> (raw)
In-Reply-To: <20120509080242.GN15960@redhat.com>

On 09.05.2012 12:02, Gleb Natapov wrote:
> On Tue, May 08, 2012 at 09:56:10PM +0400, Michael Tokarev wrote:
[]
>>> It's two only:
>>>   -drive if=none,id=<name>,...
>>>   -device virtio-blk-pci,drive=<name>
>>
>> Ok, at least it is not entirely insane :)

> That's the way you suppose to do that. Does it work if you specify
> bootindex for virtio this way? If it does then use it. Restoring
> deprecated hacks is not the way to deal with it. If it does not, then
> it is a bug that should be fixed.

Using the "too verbose" version is, well, difficult to type, when
you have to start many guests - eg, testing etc.  That's why I think
that 3 options for ahci drives is somewhat insane.

Technically there is no problem at all at combining, eg, -netdev and
-device rtl8139, in the same way as current -drive file=foo,if=bar
works, and it is MUCH easier to type and hence is MUCH user-friendly.
And much less error-prone, too.

I think it'd be a good solution to allow bootindex and some other
options to be recognizable in context of, eg -drive with if!=none,
by combining options recognizable from if= with the ones recognizable
by -drive.  Ditto for -netdev..-device: let -netdev to recognize
model=, and allow model-specific (e1000|8130|virtio-net-pci|whatever)
to be recognized as well.  This way there will be no need to specify
id=foo and netdev=foo, the command line will be shorter and also more
understandable.

Please don't say that the only sane option to start qemu guests is
to use libvirt.  Qemu has its great value exactly because it allows
starting guests from command line, which is well-scriptable.

The old hacks which were prematurely removed from qemu-kvm makes the
life easier for the _user_, which is the main target of the software.
I'd love to stop using them but sometimes it is not possible.  With
this extboot thing, qemu-kvm dropped ability to boot from SCSI for
example, and it turns out there are quite some users of this interface
exists - despite the fact SCSI is broken, there is a proprietary
bootrom exists etc -- we just broke users setups without providing
viable alternative, which, to my view, is unacceptable.

Thanks,

/mjt

  reply	other threads:[~2012-05-09 16:10 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-07 18:48 [Qemu-devel] ahci drive: how to make it non-bootable? Michael Tokarev
2012-05-08  5:20 ` Gerd Hoffmann
2012-05-08  5:45   ` Michael Tokarev
2012-05-08  6:43     ` Gerd Hoffmann
2012-05-08 17:56       ` Michael Tokarev
2012-05-09  8:02         ` Gleb Natapov
2012-05-09 16:10           ` Michael Tokarev [this message]
2012-05-10  7:33             ` Paolo Bonzini
2012-05-10 12:27             ` Gleb Natapov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FAA96DF.3090006@msgid.tls.msk.ru \
    --to=mjt@tls.msk.ru \
    --cc=gleb@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.