From: Stefan Hajnoczi <stefanha@gmail.com>
To: Stefano Garzarella <sgarzare@redhat.com>
Cc: qemu-devel@nongnu.org, Eduardo Habkost <ehabkost@redhat.com>,
Maran Wilson <maran.wilson@oracle.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
George Kennedy <george.kennedy@oracle.com>,
Stefan Hajnoczi <stefanha@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
Boris Ostrovsky <boris.ostrovsky@oracle.com>,
Richard Henderson <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH v5 0/5] pvh: add new PVH option rom
Date: Mon, 21 Jan 2019 10:17:47 +0000 [thread overview]
Message-ID: <20190121101747.GB30536@stefanha-x1.localdomain> (raw)
In-Reply-To: <20190118120143.21631-1-sgarzare@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 2801 bytes --]
On Fri, Jan 18, 2019 at 01:01:38PM +0100, Stefano Garzarella wrote:
> This patch series is based on "[PATCH v3 0/4] QEMU changes to do PVH boot" and
> provides a PVH option rom that can be used with SeaBIOS to boot uncompressed
> kernel using the x86/HVM direct boot ABI.
>
> Patches 1 and 2 are to prepare the PVH option rom, moving common functions in
> the new headers. Patch 3 adds the new PVH option rom. Patch 4 uses it when we
> are booting an uncompressed kernel using the x86/HVM direct boot ABI. Patch 5
> adds the support of loading the initrd in the PVH option rom.
>
> Based-on: <1547554687-12687-1-git-send-email-liam.merwick@oracle.com>
The linuxboot_dma.bin option ROM went via Paolo's tree, so I guess he
will merge this when he's happy, too.
> Changes in v5:
> - Patch 5: added R-b's
> - Patch 3: updated .gitignore (by Liam)
>
> Changes in v4:
> - addressed comments by Paolo:
> - Patch 2: linuxboot_dma.c: moved includes after the asm block to avoid
> compiler error
> - Patch 3: Makefile: fixed pvh.img rule
>
> Changes in v3:
> - added R-b's
> - added Patch 5 to support initrd
> - addressed comments by Eric and Liam:
> - Patch 3: comments by Liam
> - Patch 4: moved Based-on tag in the cover letter (by Eric)
> - Patch 3: added pc-bios/pvh.bin binary in the commit
>
> Changes in v2:
> - addressed comments by Stefan and Eric:
> - Patch 2: moved inludes on top of linuxboot_dma.c and add <stdint.h> in
> optrom.h
> - Patch 4: added check of pvh.bin in xen_load_linux()
> - modified commit message of patch 2 to explain better the patch
>
> Stefano Garzarella (5):
> linuxboot_dma: remove duplicate definitions of FW_CFG
> linuxboot_dma: move common functions in a new header
> optionrom: add new PVH option rom
> hw/i386/pc: use PVH option rom
> optionrom/pvh: load initrd from fw_cfg
>
> .gitignore | 4 +
> Makefile | 2 +-
> hw/i386/pc.c | 5 +
> pc-bios/optionrom/Makefile | 5 +-
> pc-bios/optionrom/linuxboot_dma.c | 114 +++--------------
> pc-bios/optionrom/optrom.h | 110 ++++++++++++++++
> pc-bios/optionrom/optrom_fw_cfg.h | 92 ++++++++++++++
> pc-bios/optionrom/pvh.S | 200 ++++++++++++++++++++++++++++++
> pc-bios/optionrom/pvh_main.c | 133 ++++++++++++++++++++
> pc-bios/pvh.bin | Bin 0 -> 1536 bytes
> 10 files changed, 568 insertions(+), 97 deletions(-)
> create mode 100644 pc-bios/optionrom/optrom.h
> create mode 100644 pc-bios/optionrom/optrom_fw_cfg.h
> create mode 100644 pc-bios/optionrom/pvh.S
> create mode 100644 pc-bios/optionrom/pvh_main.c
> create mode 100644 pc-bios/pvh.bin
>
> --
> 2.20.1
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]
prev parent reply other threads:[~2019-01-21 10:17 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-18 12:01 [Qemu-devel] [PATCH v5 0/5] pvh: add new PVH option rom Stefano Garzarella
2019-01-18 12:01 ` [Qemu-devel] [PATCH v5 1/5] linuxboot_dma: remove duplicate definitions of FW_CFG Stefano Garzarella
2019-01-18 12:01 ` [Qemu-devel] [PATCH v5 2/5] linuxboot_dma: move common functions in a new header Stefano Garzarella
2019-01-21 15:43 ` Paolo Bonzini
2019-01-21 16:20 ` Stefano Garzarella
2019-01-18 12:01 ` [Qemu-devel] [PATCH v5 3/5] optionrom: add new PVH option rom Stefano Garzarella
2019-01-21 17:37 ` Paolo Bonzini
2019-01-18 12:01 ` [Qemu-devel] [PATCH v5 4/5] hw/i386/pc: use " Stefano Garzarella
2019-01-21 17:05 ` Eduardo Habkost
2019-01-21 17:36 ` Stefano Garzarella
2019-01-21 18:33 ` Paolo Bonzini
2019-01-22 9:22 ` Stefano Garzarella
2019-01-22 9:29 ` Paolo Bonzini
2019-01-22 9:38 ` Stefano Garzarella
2019-01-18 12:01 ` [Qemu-devel] [PATCH v5 5/5] optionrom/pvh: load initrd from fw_cfg Stefano Garzarella
2019-01-21 10:17 ` Stefan Hajnoczi [this message]
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=20190121101747.GB30536@stefanha-x1.localdomain \
--to=stefanha@gmail.com \
--cc=boris.ostrovsky@oracle.com \
--cc=ehabkost@redhat.com \
--cc=george.kennedy@oracle.com \
--cc=maran.wilson@oracle.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
--cc=sgarzare@redhat.com \
--cc=stefanha@redhat.com \
/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).