mbox series

[v5,0/5] pvh: add new PVH option rom

Message ID 20190118120143.21631-1-sgarzare@redhat.com (mailing list archive)
Headers show
Series pvh: add new PVH option rom | expand

Message

Stefano Garzarella Jan. 18, 2019, 12:01 p.m. UTC
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>

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

Comments

Stefan Hajnoczi Jan. 21, 2019, 10:17 a.m. UTC | #1
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
> 
>