qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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 --]

      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).