All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Parrott <thomas.parrott@canonical.com>
To: qemu-devel@nongnu.org
Cc: marcel@redhat.com, jusual@redhat.com
Subject: Adding devices via QMP's device_add don't have their bootindex setting respected
Date: Tue, 18 May 2021 11:51:40 +0100	[thread overview]
Message-ID: <CADNu6esx69UdccUHsFfinqcN=dPj0mceKX7WT755qWcPfDrwGQ@mail.gmail.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1176 bytes --]

Due to QEMU moving towards a QMP configuration mechanism and away from
config file support, the LXD team are currently in the process of migrating
to using QMP to add devices to VMs (so that we can support the use of QEMU
6.0).

Currently we are using the `-S` flag to freeze CPU at startup, then using
QMP to add NIC devices via the `device_add` command, and then using the
`cont` command to start the VM guest.

This is working mostly fine, but there is one issue; the provided
"bootindex" property is not respected.

E.g.

device_add
{"addr":"00.0","bootindex":"0","bus":"qemu_pcie4","driver":"virtio-net-pci","id":"dev-lxd_eth0","mac":"00:16:3e:0c:69:e7","mq":"on","multifunction":"off","netdev":"lxd_eth0","vectors":"6"}

The device is seen within the VM guest and the VM BIOS, but its boot order
is last rather than first.

We've also tried using a non-zero bootindex of 1 and that has the same
effect.

After discussions on #qemu IRC channel, we found that running
`system_reset` after adding the devices allowed the `bootindex` property to
be respected.

So this looks like bug. Perhaps we can discuss it in one of the forthcoming
community calls?

Thanks
Tom Parrott

[-- Attachment #2: Type: text/html, Size: 1665 bytes --]

             reply	other threads:[~2021-05-18 13:41 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-18 10:51 Thomas Parrott [this message]
     [not found] ` <CADNu6esNHqEPzpROiRu+PEFGZJhi3Sgs=duEqjAz53FJv-UMCQ@mail.gmail.com>
2021-05-18 12:55   ` Fwd: Adding devices via QMP's device_add don't have their bootindex setting respected Paolo Bonzini
2021-05-18 13:08     ` Thomas Parrott

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='CADNu6esx69UdccUHsFfinqcN=dPj0mceKX7WT755qWcPfDrwGQ@mail.gmail.com' \
    --to=thomas.parrott@canonical.com \
    --cc=jusual@redhat.com \
    --cc=marcel@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.