Re: [MINI-OS PATCH 2/6] setup: drop parameter from map_shared_info()

[Jason Andryuk]

On 2025-06-19 07:38, Juergen Gross wrote:

The parameter of map_shared_info() is evaluated in PV mode only, and it
is always the pointer to the initial struct start_info from boot time.

By setting start_info_ptr a little bit earlier the PV variant of
map_shared_info() can just use start_info_ptr instead of a parameter.

Signed-off-by: Juergen Gross <jgr...@suse.com>
---

diff --git a/arch/x86/setup.c b/arch/x86/setup.c
index e30eb065..1bdf9509 100644
--- a/arch/x86/setup.c
+++ b/arch/x86/setup.c

@@ -185,6 +184,8 @@ arch_init(void *par)
  {
        static char hello[] = "Bootstrapping...\n";

+ start_info_ptr = par;

Should start_info_ptr only be set for CONFIG_PARAVIRT?

Oh, patch 6 does that.  Strangely, patch 3 and 6 are not in my inbox.

Reviewed-by: Jason Andryuk <jason.andr...@amd.com>

Regards,
Jason

+
        hpc_init();
        (void)HYPERVISOR_console_io(CONSOLEIO_write, strlen(hello), hello);