All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: Janne Huttunen <janne.huttunen@nokia.com>,
	Gerd Hoffmann <kraxel@redhat.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RFC][PATCH 0/6] "bootonceindex" property
Date: Wed, 22 Mar 2017 06:42:12 +0100	[thread overview]
Message-ID: <87inn1eukr.fsf@dusky.pond.sub.org> (raw)
In-Reply-To: <33ee8842-e57d-c637-ae26-e4b89a7c1002@redhat.com> (Eric Blake's message of "Tue, 21 Mar 2017 12:48:58 -0500")

Eric Blake <eblake@redhat.com> writes:

> On 03/16/2017 04:46 AM, Janne Huttunen wrote:
>> On Wed, 2017-03-15 at 08:24 +0100, Gerd Hoffmann wrote:
>>>>
>>>> The short answer: emulating real hardware.
>>>
>>> Ok, that is reason enough.
>>>
>>> Adding bootonceindex everywhere doesn't look like the best plan to me
>>> though.  Possibly we can pimp up bootindex in a backward-compatible
>>> way?
>>> Something like bootindex=<normal>[.<once>] ?
>> 
>> That would (likely) avoid modifying all devices, but wouldn't
>> that make the 'bootindex' property a string (now: 'int32')
>> and thus change the QOM API?
>
> Is there any way to make use of an alternate that supports both int and
> string simultaneously?  But see also Markus' recent addendum on the
> difficulties of supporting multiple types on command-line vs. through QMP:
>
> https://lists.gnu.org/archive/html/qemu-devel/2017-03/msg04125.html

Making it a string that encodes two integers would be inappropriate for
QMP.  If you need two integers, QMP wants you to put two integers in
JSON.  That means either a new member next to bootindex, or turning
bootindex into an alternate of integer and object.

Not passing judgement on whether we actually need "once".

  reply	other threads:[~2017-03-22  5:42 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-14 12:50 [Qemu-devel] [RFC][PATCH 0/6] "bootonceindex" property Janne Huttunen
2017-03-14 12:50 ` [Qemu-devel] [RFC][PATCH 1/6] Re-factor bootdevice list handling, pt1 Janne Huttunen
2017-03-14 12:50 ` [Qemu-devel] [RFC][PATCH 2/6] Re-factor bootdevice list handling, pt2 Janne Huttunen
2017-03-14 12:50 ` [Qemu-devel] [RFC][PATCH 3/6] Add support for "bootonceindex" property Janne Huttunen
2017-03-14 12:50 ` [Qemu-devel] [RFC][PATCH 4/6] Clear the boot once list after it has been used Janne Huttunen
2017-03-14 12:50 ` [Qemu-devel] [RFC][PATCH 5/6] Support "bootonceindex" property for virtio-net interfaces Janne Huttunen
2017-03-14 12:50 ` [Qemu-devel] [RFC][PATCH 6/6] Support "bootonceindex" property for SCSI disks Janne Huttunen
2017-03-14 16:57 ` [Qemu-devel] [RFC][PATCH 0/6] "bootonceindex" property Gerd Hoffmann
2017-03-15  6:58   ` Janne Huttunen
2017-03-15  7:24     ` Gerd Hoffmann
2017-03-16  9:46       ` Janne Huttunen
2017-03-16  9:59         ` Gerd Hoffmann
2017-03-21 17:48         ` Eric Blake
2017-03-22  5:42           ` Markus Armbruster [this message]
2017-03-21 17:55     ` Paolo Bonzini
2017-03-22  6:36       ` Janne Huttunen
2017-03-22  8:43         ` Paolo Bonzini
2017-03-22  9:00           ` Huttunen, Janne (Nokia - FI/Espoo)
2017-03-22 10:51             ` Laszlo Ersek
2017-03-22 13:58               ` Janne Huttunen
2017-03-22 14:36                 ` Laszlo Ersek
2017-03-22 15:19                   ` Janne Huttunen
2017-03-22 15:29                     ` Laszlo Ersek
2017-03-23  6:53                       ` Janne Huttunen
2017-03-23  9:13                         ` Laszlo Ersek
     [not found]   ` <CACaajQsrkuZeC6WYXdEBJY=FEkTQ9iY_i8E-d5j7JqK5DDUuOw@mail.gmail.com>
2017-03-15  7:21     ` Vasiliy Tolstov

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=87inn1eukr.fsf@dusky.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=eblake@redhat.com \
    --cc=janne.huttunen@nokia.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.