All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Smith" <dpsmith@apertussolutions.com>
To: xen-devel@lists.xenproject.org
Cc: "Daniel P. Smith" <dpsmith@apertussolutions.com>,
	jason.andryuk@amd.com, christopher.w.clark@gmail.com,
	stefano.stabellini@amd.com
Subject: [PATCH v5 00/44] Boot modules for Hyperlaunch
Date: Sun,  6 Oct 2024 17:49:11 -0400	[thread overview]
Message-ID: <20241006214956.24339-1-dpsmith@apertussolutions.com> (raw)

The Boot Modules for Hyperlaunch series is an effort to split out preliminary
changes necessary for the introduction of the Hyperlaunch domain builder
logic. These preliminary changes revolve around introducing the struct
boot_module and struct boot_domain structures. This includes converting the
dom0 construction path to use these structures. These abstractions lay the
groundwork to transform and extend the dom0 construction logic into a limited,
but general domain builder.

The splitting of Hyperlaunch into a set of series are twofold, to reduce the
effort in reviewing a much larger series, and to reduce the effort in handling
the knock-on effects to the construction logic from requested review changes.

Much thanks to AMD for supporting this work.

Documentation on Hyperlaunch:
https://wiki.xenproject.org/wiki/Hyperlaunch

Original Hyperlaunch v1 patch series:
https://lists.xenproject.org/archives/html/xen-devel/2022-07/msg00345.html

V/r,
Daniel P. Smith

Changes since v4:
- added requested inline code comments
- moved instance of struct boot_info to unit level and extern'ed
- array of struct boot_module moved into struct boot_info
- renamed function to multiboot_fill_bootinfo, now returns *struct boot_info
- multiboot_fill_bootinfo changed to take multiboot_info_t addr as param
- added missing guard that checked there were multiboot1/2 modules passed
- renmaed struct elements per the review
- fixed errant commit messages per the review
- corrected coding style per review
- attempted to repalce all open codings of page/addr translations touched 
- unified use of `bi` as var name for pointer ref to struct boot_info
- when appropriate, ensure variables where typed, eg size_t, paddr_t, etc.
- dropped all uses of "a = b = c"

Changes since v3:
- reduced scope to x86 only
- broke changes into a smaller chunks with a linear progression
- concerns about deconflicting with Arm deferred
- conversion from mb1 to boot modules no longer attempted at entry points
- the temporary conversion function is now the permenant means to convert
- incorporated suggestion from Andy Cooper for handling bootstrap_map

Changes since v2:
- combined v2 patches 7 and 8 for common review
- rebased the v2 series onto the current tip of staging (sorry)
- fixed the placement of the patch changelogs
- provided the changes description in the cover letter

Changes since v1:
- the v2 and v3 series implement functionality from v1 patches 2-4
    - v2 series objective is to enable efficient patch review in support
      of merging the functionality into the hypervisor. It implements a
      subset of the v1 series, incorporating changes from community
      feedback.
- the bootstrap map is made accessible early in the v2 series via both
  multiboot and boot module arguments until later in the series where
  multiboot use is retired. This allows for incremental conversion across
  several patches from multiboot to boot modules.
- the 32-bit x86 boot environment header is removed, and changes are
  made to allow the new common bootinfo headers to be used instead.
- Arm and RISC-V architecture bootinfo headers are added to ensure that
  builds on those architectures can complete correctly.
- The KConfig patch to set the maximum number of boot modules allowed
  is not included in this series, replaced with a static maximum define.

Andrew Cooper (1):
  x86/boot: split bootstrap_map_addr() out of bootstrap_map()

Christopher Clark (1):
  x86/boot: move x86 boot module counting into a new boot_info struct

Daniel P. Smith (42):
  x86/boot: move boot loader name to boot info
  x86/boot: move cmdline to boot info
  x86/boot: move mmap info to boot info
  x86/boot: introduce struct boot_module
  x86/boot: convert consider_modules to struct boot_module
  x86/boot: move headroom to boot modules
  x86/boot: convert setup.c mod refs to early_mod
  x86/boot: introduce boot module types
  x86/boot: introduce boot module flags
  x86/boot: add start and size fields to struct boot_module
  x86/boot: update struct boot_module on module relocation
  x86/boot: transition relocation calculations to struct boot_module
  x86/boot: introduce boot module interator
  x86/boot: introduce consumed flag for struct boot_module
  x86/boot: convert microcode loading to consume struct boot_info
  x86/boot: convert late microcode loading to struct boot_module
  x86/boot: use consumed boot module flag for microcode
  x86/boot: convert xsm policy loading to struct boot_module
  x86/boot: convert ramdisk locating to struct boot_module
  x86/boot: remove module_map usage from microcode loading
  x86/boot: remove module_map usage from xsm policy loading
  x86/boot: remove module_map usage by ramdisk loading
  x86/boot: convert create_dom0 to use boot info
  x86/boot: convert construct_dom0 to use struct boot_module
  x86/boot: relocate kextra into boot info
  x86/boot: add cmdline to struct boot_module
  x86/boot: convert dom0_construct_pv image param to struct boot_module
  x86/boot: convert dom0_construct_pv initrd param to struct boot_module
  x86/boot: convert dom0_construct_pvh to struct boot_module
  x86/boot: convert pvh_load_kernel to struct boot_module
  x86/boot: convert initial_images to struct boot_module
  x86/boot: drop the use of initial_images unit global
  x86/boot: remove usage of mod_end by discard_initial_images
  x86/boot: remove remaining early_mod references
  x86/boot: remove mod from struct boot_module
  x86/boot: introduce boot domain
  x86/boot: introduce domid field to struct boot_domain
  x86/boot: add cmdline to struct boot_domain
  x86/boot: add struct domain to struct boot_domain
  x86/boot: convert construct_dom0 to struct boot_domain
  x86/boot: convert dom0_construct_pv to struct boot_domain
  x86/boot: convert dom0_construct_pvh to struct boot_domain

 xen/arch/x86/cpu/microcode/core.c     |  78 +++----
 xen/arch/x86/dom0_build.c             |  21 +-
 xen/arch/x86/hvm/dom0_build.c         |  55 +++--
 xen/arch/x86/include/asm/bootdomain.h |  37 +++
 xen/arch/x86/include/asm/bootinfo.h   |  91 ++++++++
 xen/arch/x86/include/asm/dom0_build.h |  11 +-
 xen/arch/x86/include/asm/microcode.h  |  12 +-
 xen/arch/x86/include/asm/setup.h      |  11 +-
 xen/arch/x86/pv/dom0_build.c          |  54 ++---
 xen/arch/x86/setup.c                  | 321 ++++++++++++++++----------
 xen/include/xsm/xsm.h                 |  14 +-
 xen/xsm/xsm_core.c                    |  15 +-
 xen/xsm/xsm_policy.c                  |  18 +-
 13 files changed, 461 insertions(+), 277 deletions(-)
 create mode 100644 xen/arch/x86/include/asm/bootdomain.h
 create mode 100644 xen/arch/x86/include/asm/bootinfo.h

-- 
2.30.2



             reply	other threads:[~2024-10-06 21:50 UTC|newest]

Thread overview: 153+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-06 21:49 Daniel P. Smith [this message]
2024-10-06 21:49 ` [PATCH v5 01/44] x86/boot: move x86 boot module counting into a new boot_info struct Daniel P. Smith
2024-10-07 17:57   ` Jason Andryuk
2024-10-08  6:41     ` Jan Beulich
2024-10-09 11:15       ` Daniel P. Smith
2024-10-09 15:02     ` Jan Beulich
2024-10-06 21:49 ` [PATCH v5 02/44] x86/boot: move boot loader name to boot info Daniel P. Smith
2024-10-07 17:58   ` Jason Andryuk
2024-10-09 15:07   ` Jan Beulich
2024-10-10  0:55     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 03/44] x86/boot: move cmdline " Daniel P. Smith
2024-10-07 18:09   ` Jason Andryuk
2024-10-08  6:42     ` Jan Beulich
2024-10-09 11:28       ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 04/44] x86/boot: move mmap info " Daniel P. Smith
2024-10-07 18:10   ` Jason Andryuk
2024-10-09 15:13   ` Jan Beulich
2024-10-10  0:59     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 05/44] x86/boot: introduce struct boot_module Daniel P. Smith
2024-10-07 18:29   ` Jason Andryuk
2024-10-09 11:31     ` Daniel P. Smith
2024-10-09 15:17   ` Jan Beulich
2024-10-10  1:01     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 06/44] x86/boot: convert consider_modules to " Daniel P. Smith
2024-10-07 18:36   ` Jason Andryuk
2024-10-09 15:22   ` Jan Beulich
2024-10-10  1:02     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 07/44] x86/boot: move headroom to boot modules Daniel P. Smith
2024-10-07 18:55   ` Jason Andryuk
2024-10-09 11:46     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 08/44] x86/boot: convert setup.c mod refs to early_mod Daniel P. Smith
2024-10-07 19:34   ` Jason Andryuk
2024-10-09 14:23     ` Daniel P. Smith
2024-10-09 14:29       ` Jan Beulich
2024-10-09 14:31         ` Daniel P. Smith
2024-10-09 15:29   ` Jan Beulich
2024-10-10  1:07     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 09/44] x86/boot: introduce boot module types Daniel P. Smith
2024-10-07 19:50   ` Jason Andryuk
2024-10-09 14:25     ` Daniel P. Smith
2024-10-09 15:30   ` Jan Beulich
2024-10-10  1:11     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 10/44] x86/boot: introduce boot module flags Daniel P. Smith
2024-10-07 20:02   ` Jason Andryuk
2024-10-09 14:27     ` Daniel P. Smith
2024-10-09 15:32     ` Jan Beulich
2024-10-06 21:49 ` [PATCH v5 11/44] x86/boot: split bootstrap_map_addr() out of bootstrap_map() Daniel P. Smith
2024-10-07 20:04   ` Jason Andryuk
2024-10-09 15:38   ` Jan Beulich
2024-10-10  1:16     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 12/44] x86/boot: add start and size fields to struct boot_module Daniel P. Smith
2024-10-07 20:06   ` Jason Andryuk
2024-10-09 15:39   ` Jan Beulich
2024-10-10  1:23     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 13/44] x86/boot: update struct boot_module on module relocation Daniel P. Smith
2024-10-07 20:31   ` Jason Andryuk
2024-10-09 14:36     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 14/44] x86/boot: transition relocation calculations to struct boot_module Daniel P. Smith
2024-10-07 20:44   ` Jason Andryuk
2024-10-09 14:44     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 15/44] x86/boot: introduce boot module interator Daniel P. Smith
2024-10-07 20:59   ` Jason Andryuk
2024-10-09 15:53   ` Jan Beulich
2024-10-10  1:45     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 16/44] x86/boot: introduce consumed flag for struct boot_module Daniel P. Smith
2024-10-07 21:06   ` Jason Andryuk
2024-10-09 14:49     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 17/44] x86/boot: convert microcode loading to consume struct boot_info Daniel P. Smith
2024-10-07 21:22   ` Jason Andryuk
2024-10-08 12:50     ` Jason Andryuk
2024-10-09 14:58       ` Daniel P. Smith
2024-10-09 14:57     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 18/44] x86/boot: convert late microcode loading to struct boot_module Daniel P. Smith
2024-10-08 12:50   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 19/44] x86/boot: use consumed boot module flag for microcode Daniel P. Smith
2024-10-08 15:56   ` Jason Andryuk
2024-10-09 16:29     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 20/44] x86/boot: convert xsm policy loading to struct boot_module Daniel P. Smith
2024-10-08 16:13   ` Jason Andryuk
2024-10-09 17:21     ` Daniel P. Smith
2024-10-10 17:23       ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 21/44] x86/boot: convert ramdisk locating " Daniel P. Smith
2024-10-08 16:26   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 22/44] x86/boot: remove module_map usage from microcode loading Daniel P. Smith
2024-10-08 16:30   ` Jason Andryuk
2024-10-09 17:24     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 23/44] x86/boot: remove module_map usage from xsm policy loading Daniel P. Smith
2024-10-08 16:36   ` Jason Andryuk
2024-10-09 17:25     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 24/44] x86/boot: remove module_map usage by ramdisk loading Daniel P. Smith
2024-10-08 16:46   ` Jason Andryuk
2024-10-09 18:36     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 25/44] x86/boot: convert create_dom0 to use boot info Daniel P. Smith
2024-10-08 16:52   ` Jason Andryuk
2024-10-09 23:02     ` Daniel P. Smith
2024-10-10  8:03       ` Jan Beulich
2024-10-10 10:41         ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 26/44] x86/boot: convert construct_dom0 to use struct boot_module Daniel P. Smith
2024-10-08 16:57   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 27/44] x86/boot: relocate kextra into boot info Daniel P. Smith
2024-10-08 17:01   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 28/44] x86/boot: add cmdline to struct boot_module Daniel P. Smith
2024-10-08 17:08   ` Jason Andryuk
2024-10-09 23:09     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 29/44] x86/boot: convert dom0_construct_pv image param " Daniel P. Smith
2024-10-08 18:03   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 30/44] x86/boot: convert dom0_construct_pv initrd " Daniel P. Smith
2024-10-08 18:30   ` Jason Andryuk
2024-10-09 23:12     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 31/44] x86/boot: convert dom0_construct_pvh " Daniel P. Smith
2024-10-08 18:33   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 32/44] x86/boot: convert pvh_load_kernel " Daniel P. Smith
2024-10-08 18:42   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 33/44] x86/boot: convert initial_images " Daniel P. Smith
2024-10-08 18:52   ` Jason Andryuk
2024-10-09 23:15     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 34/44] x86/boot: drop the use of initial_images unit global Daniel P. Smith
2024-10-08 19:04   ` Jason Andryuk
2024-10-09 23:22     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 35/44] x86/boot: remove usage of mod_end by discard_initial_images Daniel P. Smith
2024-10-08 19:05   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 36/44] x86/boot: remove remaining early_mod references Daniel P. Smith
2024-10-08 19:15   ` Jason Andryuk
2024-10-09  6:53     ` Jan Beulich
2024-10-09 23:42       ` Daniel P. Smith
2024-10-10  8:05         ` Jan Beulich
2024-10-09 23:40     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 37/44] x86/boot: remove mod from struct boot_module Daniel P. Smith
2024-10-08 19:16   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 38/44] x86/boot: introduce boot domain Daniel P. Smith
2024-10-08 19:30   ` Jason Andryuk
2024-10-06 21:49 ` [PATCH v5 39/44] x86/boot: introduce domid field to struct boot_domain Daniel P. Smith
2024-10-08 19:31   ` Jason Andryuk
2024-10-08 19:36   ` Jason Andryuk
2024-10-09  6:54     ` Jan Beulich
2024-10-10  0:34     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 40/44] x86/boot: add cmdline " Daniel P. Smith
2024-10-08 20:05   ` Jason Andryuk
2024-10-10  0:45     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 41/44] x86/boot: add struct domain " Daniel P. Smith
2024-10-08 19:48   ` Jason Andryuk
2024-10-10  0:47     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 42/44] x86/boot: convert construct_dom0 " Daniel P. Smith
2024-10-08 19:47   ` Jason Andryuk
2024-10-10  0:48     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 43/44] x86/boot: convert dom0_construct_pv " Daniel P. Smith
2024-10-08 19:54   ` Jason Andryuk
2024-10-10  0:49     ` Daniel P. Smith
2024-10-06 21:49 ` [PATCH v5 44/44] x86/boot: convert dom0_construct_pvh " Daniel P. Smith
2024-10-08 19:56   ` Jason Andryuk
2024-10-10  0:51     ` Daniel P. Smith
2024-10-08 20:07 ` [PATCH v5 00/44] Boot modules for Hyperlaunch Jason Andryuk
2024-10-08 21:21   ` Andrew Cooper

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=20241006214956.24339-1-dpsmith@apertussolutions.com \
    --to=dpsmith@apertussolutions.com \
    --cc=christopher.w.clark@gmail.com \
    --cc=jason.andryuk@amd.com \
    --cc=stefano.stabellini@amd.com \
    --cc=xen-devel@lists.xenproject.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.