[coreboot-gerrit] Change in coreboot[master]: arch/x86/boot.c: Pass arguments when running programs

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/36143 )

Change subject: arch/x86/boot.c: Pass arguments when running programs
......................................................................

arch/x86/boot.c: Pass arguments when running programs

Payloads can use coreboot tables passed on via arguments instead of
via a pointer in lower memory.

Stages can make use of the argument to pass on information.

Change-Id: Ie0f44e9e1992221e02c49d0492cdd2a3d9013560
Signed-off-by: Arthur Heymans 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36143
Reviewed-by: Nico Huber 
Reviewed-by: Aaron Durbin 
Tested-by: build bot (Jenkins) 
---
M src/arch/x86/boot.c
1 file changed, 5 insertions(+), 6 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved
  Aaron Durbin: Looks good to me, approved

diff --git a/src/arch/x86/boot.c b/src/arch/x86/boot.c
index 5f60f13..ada49d0 100644
--- a/src/arch/x86/boot.c
+++ b/src/arch/x86/boot.c
@@ -30,13 +30,12 @@

void arch_prog_run(struct prog *prog)
 {
-	__asm__ volatile (
 #ifdef __x86_64__
-		"jmp  *%%rdi\n"
+	void (*doit)(void *arg);
 #else
-		"jmp  *%%edi\n"
+	/* Ensure the argument is pushed on the stack. */
+	asmlinkage void (*doit)(void *arg);
 #endif
-
-		:: "D"(prog_entry(prog))
-	);
+	doit = prog_entry(prog);
+	doit(prog_entry_arg(prog));
 }