[PATCH 0/3] x86/trampoline: Header cleanup

Andrew Cooper posted 3 patches 2 months, 2 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20240905130657.4075906-1-andrew.cooper3@citrix.com
xen/arch/x86/acpi/power.c             |   4 +-
xen/arch/x86/cpu/intel.c              |   2 +
xen/arch/x86/dmi_scan.c               |  12 ---
xen/arch/x86/efi/efi-boot.h           |   4 +-
xen/arch/x86/guest/xen/pvh-boot.c     |   1 +
xen/arch/x86/include/asm/acpi.h       |   1 -
xen/arch/x86/include/asm/config.h     |  19 ----
xen/arch/x86/include/asm/processor.h  |   2 -
xen/arch/x86/include/asm/setup.h      |   2 -
xen/arch/x86/include/asm/trampoline.h | 120 ++++++++++++++++++++++++++
xen/arch/x86/mm.c                     |   1 +
xen/arch/x86/platform_hypercall.c     |   2 +
xen/arch/x86/setup.c                  |   1 +
xen/arch/x86/smpboot.c                |   1 +
xen/arch/x86/tboot.c                  |   2 +
xen/arch/x86/x86_64/mm.c              |   2 +
16 files changed, 136 insertions(+), 40 deletions(-)
create mode 100644 xen/arch/x86/include/asm/trampoline.h
[PATCH 0/3] x86/trampoline: Header cleanup
Posted by Andrew Cooper 2 months, 2 weeks ago
This started while inspecting a preprossed file for bitops, but it turns out
is relevant for Frediano's 32bit boot code changes too.

Its header file juggling, and documentation with observations relevant to both
the ASI and Host UEFI Secureboot work, hence the extended CC list.

Andrew Cooper (3):
  x86/acpi: Drop acpi_video_flags and use bootsym(video_flags) directly
  x86/trampoline: Move the trampoline declarations out of <asm/config.h>
  x86/trampoline: Collect other scattered trampoline symbols

 xen/arch/x86/acpi/power.c             |   4 +-
 xen/arch/x86/cpu/intel.c              |   2 +
 xen/arch/x86/dmi_scan.c               |  12 ---
 xen/arch/x86/efi/efi-boot.h           |   4 +-
 xen/arch/x86/guest/xen/pvh-boot.c     |   1 +
 xen/arch/x86/include/asm/acpi.h       |   1 -
 xen/arch/x86/include/asm/config.h     |  19 ----
 xen/arch/x86/include/asm/processor.h  |   2 -
 xen/arch/x86/include/asm/setup.h      |   2 -
 xen/arch/x86/include/asm/trampoline.h | 120 ++++++++++++++++++++++++++
 xen/arch/x86/mm.c                     |   1 +
 xen/arch/x86/platform_hypercall.c     |   2 +
 xen/arch/x86/setup.c                  |   1 +
 xen/arch/x86/smpboot.c                |   1 +
 xen/arch/x86/tboot.c                  |   2 +
 xen/arch/x86/x86_64/mm.c              |   2 +
 16 files changed, 136 insertions(+), 40 deletions(-)
 create mode 100644 xen/arch/x86/include/asm/trampoline.h


base-commit: a2de7dc4d845738e734b10fce6550c89c6b1092c
-- 
2.39.2
Re: [PATCH 0/3] x86/trampoline: Header cleanup
Posted by Frediano Ziglio 2 months, 2 weeks ago
On Thu, Sep 5, 2024 at 2:07 PM Andrew Cooper <andrew.cooper3@citrix.com>
wrote:

> This started while inspecting a preprossed file for bitops, but it turns
> out
> is relevant for Frediano's 32bit boot code changes too.
>
> Its header file juggling, and documentation with observations relevant to
> both
> the ASI and Host UEFI Secureboot work, hence the extended CC list.
>
> Andrew Cooper (3):
>   x86/acpi: Drop acpi_video_flags and use bootsym(video_flags) directly
>   x86/trampoline: Move the trampoline declarations out of <asm/config.h>
>   x86/trampoline: Collect other scattered trampoline symbols
>
>  xen/arch/x86/acpi/power.c             |   4 +-
>  xen/arch/x86/cpu/intel.c              |   2 +
>  xen/arch/x86/dmi_scan.c               |  12 ---
>  xen/arch/x86/efi/efi-boot.h           |   4 +-
>  xen/arch/x86/guest/xen/pvh-boot.c     |   1 +
>  xen/arch/x86/include/asm/acpi.h       |   1 -
>  xen/arch/x86/include/asm/config.h     |  19 ----
>  xen/arch/x86/include/asm/processor.h  |   2 -
>  xen/arch/x86/include/asm/setup.h      |   2 -
>  xen/arch/x86/include/asm/trampoline.h | 120 ++++++++++++++++++++++++++
>  xen/arch/x86/mm.c                     |   1 +
>  xen/arch/x86/platform_hypercall.c     |   2 +
>  xen/arch/x86/setup.c                  |   1 +
>  xen/arch/x86/smpboot.c                |   1 +
>  xen/arch/x86/tboot.c                  |   2 +
>  xen/arch/x86/x86_64/mm.c              |   2 +
>  16 files changed, 136 insertions(+), 40 deletions(-)
>  create mode 100644 xen/arch/x86/include/asm/trampoline.h
>
>
> base-commit: a2de7dc4d845738e734b10fce6550c89c6b1092c
>
>
Hi,
   the series looks good to me.

Frediano