On Tue, Oct 15, 2024 at 09:25:11AM +0100, Frediano Ziglio wrote:
> This series came from part of the work of removing duplications between
> boot code and rewriting part of code from assembly to C.
Acked-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
> Changes since v1, more details in specific commits:
> - style updates;
> - comments and descriptions improvements;
> - other improvements.
>
> Changes since v2:
> - rebased on master, resolved conflicts;
> - add comment on trampoline section.
>
> Changes since v3:
> - changed new function name;
> - declare efi_multiboot2 in a separate header;
> - distinguish entry point from using magic number;
> - other minor changes (see commens in commits).
>
> Changes since v4:
> - rebase on staging;
> - set %fs and %gs as other segment registers;
> - style and other changes.
>
> Changes since v5:
> - fixed a typo.
>
> Changes since v6:
> - remove merged patch;
> - comment and style;
> - change some pointer checks to avoid overflows;
> - rename parse-mbi2.c to mbi2.c.
>
> Changes since v7:
> - removed merged parts;
> - add required stack alignment.
>
> Changes since v8:
> - added "Fixes:" line;
> - typo in commit message: Adler -> Alder;
> - add ".init" to mbi2.o;
> - reduce difference in Makefile.
>
> Changes since v9:
> - minor messages updates.
>
> Frediano Ziglio (2):
> x86/boot: Align mbi2.c stack to 16 bytes
> x86/boot: Improve MBI2 structure check
>
> xen/arch/x86/efi/Makefile | 4 ++--
> xen/arch/x86/efi/mbi2.c | 7 +++++--
> 2 files changed, 7 insertions(+), 4 deletions(-)
>
> --
> 2.34.1
>
--
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab