kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: seabios@seabios.org, kvm@vger.kernel.org
Cc: Gerd Hoffmann <kraxel@redhat.com>
Subject: [PATCH seabios 0/3] add kvmtool support
Date: Thu,  2 Nov 2017 16:50:28 +0100	[thread overview]
Message-ID: <20171102155031.17454-1-kraxel@redhat.com> (raw)

  Hi,

The 1.11 seabios release which is just around the corner brings serial
console support to seabios.  Which is useful for kvmtool too.  So I
decided to undust my old kvmtool support patch series and check out
where we stand. 

Patch #1 adds the config option, wires up everything and adds ram
detection for kvmtool.  Patches #2 and #3 tweak the seabios virtio
drivers so they work with kvmtool.

With that in place seabios works nicely.  Loads grub from hard disk,
allows to edit menu entries, loads and runs the linux kernel.

But there are also some known issues:

 (1) ram detection is clumpsy (see patch #1) due to kvmtool not passing
     ram information to the firmware (on x86, seems other platforms use
     the device tree for that).

 (2) kvmtool virtio implementation seems to have some bugs.
     specifically handing over the virtio-blk devices from seabios to
     the linux kernel doesn't work (see patch #3).

 (3) when kernels are booted via seabios+grub all the command line args
     kvmtool adds to the kernel command line on direct kernel boot will
     not be passed to the kernel.  Not sure how much of a problem that
     is in practice, due to (2) I didn't yet manage to boot a linux
     guest to the login prompt.

cheers,
  Gerd

Gerd Hoffmann (3):
  kvmtool: initial support
  kvmtool: allow mmio for legacy bar 0
  kvmtool: support larger virtio queues

 Makefile             |  1 +
 src/fw/paravirt.h    |  3 +++
 src/hw/virtio-ring.h |  2 +-
 src/fw/paravirt.c    | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 src/hw/virtio-pci.c  | 16 ++++++++++++----
 src/post.c           |  4 +++-
 src/sercon.c         |  2 ++
 src/Kconfig          | 23 +++++++++++++++++++++--
 8 files changed, 91 insertions(+), 8 deletions(-)

-- 
2.9.3

             reply	other threads:[~2017-11-02 15:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-02 15:50 Gerd Hoffmann [this message]
2017-11-02 15:50 ` [PATCH seabios 1/3] kvmtool: initial support Gerd Hoffmann
2017-11-02 15:50 ` [PATCH seabios 2/3] kvmtool: allow mmio for legacy bar 0 Gerd Hoffmann
2017-11-02 15:50 ` [PATCH seabios 3/3] kvmtool: support larger virtio queues Gerd Hoffmann
2017-11-03 13:49   ` Jean-Philippe Brucker
2017-11-03 15:34     ` Gerd Hoffmann
2017-11-03 19:42       ` Jean-Philippe Brucker
2017-11-06 14:54         ` Jean-Philippe Brucker

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=20171102155031.17454-1-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=seabios@seabios.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).