[coreboot-gerrit] Patch set updated for coreboot: ca46f8b program loading: add prog_run() function
Aaron Durbin (adurbin@google.com)
gerrit at coreboot.org
Mon Mar 23 15:38:21 CET 2015
Aaron Durbin (adurbin at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8846
-gerrit
commit ca46f8bc387392188f3ea5d419a48ecb010eb33d
Author: Aaron Durbin <adurbin at chromium.org>
Date: Fri Mar 20 15:55:08 2015 -0500
program loading: add prog_run() function
The prog_run() function abstracts away what is required
for running a given program. Within it, there are 2
calls: 1. platform_prog_run() and 2. arch_prog_run().
The platform_prog_run() allows for a chipset to intercept
a program that will be run. This allows for CPU switching
as currently needed in t124 and t132.
Change-Id: I22a5dd5bfb1018e7e46475e47ac993a0941e2a8c
Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
src/arch/arm/Makefile.inc | 2 ++
src/arch/arm/boot.c | 11 +++++++---
src/arch/arm64/Makefile.inc | 2 ++
src/arch/arm64/boot.c | 11 +++++++---
src/arch/riscv/Makefile.inc | 2 ++
src/arch/riscv/boot.c | 14 ++++++++----
src/arch/x86/boot/Makefile.inc | 1 +
src/arch/x86/boot/boot.c | 10 ++++++++-
src/include/program_loading.h | 12 +++++++++-
src/lib/Makefile.inc | 8 +++----
src/lib/arch_ops.c | 28 -----------------------
src/lib/loaders/load_and_run_payload.c | 2 +-
src/lib/loaders/load_and_run_ramstage.c | 4 ++--
src/lib/loaders/load_and_run_romstage.c | 2 +-
src/lib/prog_ops.c | 39 +++++++++++++++++++++++++++++++++
15 files changed, 100 insertions(+), 48 deletions(-)
diff --git a/src/arch/arm/Makefile.inc b/src/arch/arm/Makefile.inc
index e339be1..177992e 100644
--- a/src/arch/arm/Makefile.inc
+++ b/src/arch/arm/Makefile.inc
@@ -54,6 +54,7 @@ ifeq ($(CONFIG_ARCH_BOOTBLOCK_ARM),y)
bootblock-y += id.S
$(obj)/arch/arm/id.bootblock.o: $(obj)/build.h
+bootblock-y += boot.c
bootblock-y += stages.c
bootblock-y += eabi_compat.c
bootblock-y += memset.S
@@ -86,6 +87,7 @@ verstage-y += stages.c
ifeq ($(CONFIG_ARCH_ROMSTAGE_ARM),y)
+romstage-y += boot.c
romstage-y += stages.c
romstage-y += div0.c
romstage-y += eabi_compat.c
diff --git a/src/arch/arm/boot.c b/src/arch/arm/boot.c
index 9dc6dbd..6a2934b 100644
--- a/src/arch/arm/boot.c
+++ b/src/arch/arm/boot.c
@@ -20,12 +20,17 @@
#include <arch/cache.h>
#include <program_loading.h>
-void arch_payload_run(const struct payload *payload)
+void arch_prog_run(struct prog *prog)
{
void (*doit)(void *);
cache_sync_instructions();
- doit = prog_entry(&payload->prog);
- doit(prog_entry_arg(&payload->prog));
+ doit = prog_entry(prog);
+ doit(prog_entry_arg(prog));
+}
+
+void arch_payload_run(struct payload *payload)
+{
+ arch_prog_run(&payload->prog);
}
diff --git a/src/arch/arm64/Makefile.inc b/src/arch/arm64/Makefile.inc
index b50499e..9c97c4d 100644
--- a/src/arch/arm64/Makefile.inc
+++ b/src/arch/arm64/Makefile.inc
@@ -57,6 +57,7 @@ bootblock-y += div0.c
bootblock-y += id.S
$(obj)/arch/arm64/id.bootblock.o: $(obj)/build.h
+bootblock-y += boot.c
bootblock-y += c_entry.c
bootblock-y += stage_entry.S
bootblock-y += stages.c
@@ -79,6 +80,7 @@ endif # CONFIG_ARCH_BOOTBLOCK_ARM64
ifeq ($(CONFIG_ARCH_ROMSTAGE_ARM64),y)
+romstage-y += boot.c
romstage-y += c_entry.c
romstage-y += stage_entry.S
romstage-y += stages.c
diff --git a/src/arch/arm64/boot.c b/src/arch/arm64/boot.c
index 9dc6dbd..6a2934b 100644
--- a/src/arch/arm64/boot.c
+++ b/src/arch/arm64/boot.c
@@ -20,12 +20,17 @@
#include <arch/cache.h>
#include <program_loading.h>
-void arch_payload_run(const struct payload *payload)
+void arch_prog_run(struct prog *prog)
{
void (*doit)(void *);
cache_sync_instructions();
- doit = prog_entry(&payload->prog);
- doit(prog_entry_arg(&payload->prog));
+ doit = prog_entry(prog);
+ doit(prog_entry_arg(prog));
+}
+
+void arch_payload_run(struct payload *payload)
+{
+ arch_prog_run(&payload->prog);
}
diff --git a/src/arch/riscv/Makefile.inc b/src/arch/riscv/Makefile.inc
index 242fd40..30c84a9 100644
--- a/src/arch/riscv/Makefile.inc
+++ b/src/arch/riscv/Makefile.inc
@@ -31,6 +31,7 @@ ifeq ($(CONFIG_ARCH_BOOTBLOCK_RISCV),y)
bootblock_lds = $(src)/arch/riscv/bootblock.ld
bootblock-y = bootblock.S stages.c
+bootblock-y += boot.c
bootblock-y += rom_media.c
bootblock-y += \
$(top)/src/lib/memchr.c \
@@ -52,6 +53,7 @@ endif
################################################################################
ifeq ($(CONFIG_ARCH_ROMSTAGE_RISCV),y)
+romstage-y += boot.c
romstage-y += stages.c
romstage-y += rom_media.c
romstage-y += \
diff --git a/src/arch/riscv/boot.c b/src/arch/riscv/boot.c
index 4c2be03..f66c02f 100644
--- a/src/arch/riscv/boot.c
+++ b/src/arch/riscv/boot.c
@@ -17,11 +17,17 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
-#include <arch/stages.h>
#include <program_loading.h>
-void arch_payload_run(const struct payload *payload)
+void arch_prog_run(struct prog *prog)
{
-// uart_rx_byte(0);
- stage_exit(prog_entry(&payload->prog));
+ void (*doit)(void *);
+
+ doit = prog_entry(prog);
+ doit(prog_entry_arg(prog));
+}
+
+void arch_payload_run(struct payload *payload)
+{
+ arch_prog_run(&payload->prog);
}
diff --git a/src/arch/x86/boot/Makefile.inc b/src/arch/x86/boot/Makefile.inc
index 9f06b4f..7810ba8 100644
--- a/src/arch/x86/boot/Makefile.inc
+++ b/src/arch/x86/boot/Makefile.inc
@@ -2,6 +2,7 @@
ifeq ($(CONFIG_ARCH_ROMSTAGE_X86_32),y)
romstage-y += cbmem.c
+romstage-y += boot.c
endif # CONFIG_ARCH_ROMSTAGE_X86_32
diff --git a/src/arch/x86/boot/boot.c b/src/arch/x86/boot/boot.c
index 1fb36e2..d536584 100644
--- a/src/arch/x86/boot/boot.c
+++ b/src/arch/x86/boot/boot.c
@@ -123,7 +123,7 @@ static void jmp_payload(void *entry, unsigned long buffer, unsigned long size)
);
}
-void arch_payload_run(const struct payload *payload)
+void arch_payload_run(struct payload *payload)
{
if (IS_ENABLED(CONFIG_RELOCATABLE_RAMSTAGE))
jmp_payload_no_bounce_buffer(prog_entry(&payload->prog));
@@ -132,3 +132,11 @@ void arch_payload_run(const struct payload *payload)
(uintptr_t)payload->bounce.data,
payload->bounce.size);
}
+
+void arch_prog_run(struct prog *prog)
+{
+ __asm__ volatile (
+ "jmp *%%edi\n"
+ :: "D"(prog_entry(prog))
+ );
+}
diff --git a/src/include/program_loading.h b/src/include/program_loading.h
index 38047c7..1df1825 100644
--- a/src/include/program_loading.h
+++ b/src/include/program_loading.h
@@ -90,6 +90,16 @@ static inline void prog_set_entry(struct prog *prog, void *e, void *arg)
prog->arg = arg;
}
+/* Run the program described by prog. */
+void prog_run(struct prog *prog);
+/* Per architecture implementation running a program. */
+void arch_prog_run(struct prog *prog);
+/* Platform (SoC/chipset) specific overrides for running a program. This is
+ * called prior to calling the arch_prog_run. Thus, if there is anything
+ * special that needs to be done by the platform similar to the architecture
+ * code it needs to that as well. */
+void platform_prog_run(struct prog *prog);
+
/************************
* ROMSTAGE LOADING *
************************/
@@ -143,7 +153,7 @@ void payload_run(void);
void mirror_payload(struct payload *payload);
/* architecture specific function to run payload. */
-void arch_payload_run(const struct payload *payload);
+void arch_payload_run(struct payload *payload);
/* Payload loading operations. */
struct payload_loader_ops {
diff --git a/src/lib/Makefile.inc b/src/lib/Makefile.inc
index 3b15537..d6470f7 100644
--- a/src/lib/Makefile.inc
+++ b/src/lib/Makefile.inc
@@ -18,7 +18,7 @@
#
subdirs-y += loaders
-bootblock-y += arch_ops.c
+bootblock-y += prog_ops.c
bootblock-y += cbfs.c cbfs_core.c
bootblock-$(CONFIG_COMMON_CBFS_SPI_WRAPPER) += cbfs_spi.c
@@ -26,7 +26,7 @@ bootblock-$(CONFIG_CONSOLE_CBMEM) += cbmem_console.c
bootblock-y += memchr.c
bootblock-y += memcmp.c
-verstage-y += arch_ops.c
+verstage-y += prog_ops.c
verstage-y += delay.c
verstage-y += cbfs.c
verstage-y += memcmp.c
@@ -36,7 +36,7 @@ verstage-y += tlcl.c
verstage-$(CONFIG_GENERIC_UDELAY) += timer.c
-romstage-y += arch_ops.c
+romstage-y += prog_ops.c
romstage-y += memchr.c
romstage-y += memcmp.c
$(foreach arch,$(ARCH_SUPPORTED),\
@@ -61,7 +61,7 @@ romstage-$(CONFIG_ARCH_ROMSTAGE_X86_32) += gcc.c
ramstage-$(CONFIG_ARCH_RAMSTAGE_X86_32) += gcc.c
endif
-ramstage-y += arch_ops.c
+ramstage-y += prog_ops.c
ramstage-y += hardwaremain.c
ramstage-y += selfboot.c
ramstage-y += coreboot_table.c
diff --git a/src/lib/arch_ops.c b/src/lib/arch_ops.c
deleted file mode 100644
index ac1be77..0000000
--- a/src/lib/arch_ops.c
+++ /dev/null
@@ -1,28 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2014 Imagination Technologies
- * Copyright 2015 Google Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#include <program_loading.h>
-
-/* For each segment of a program loaded this function is called*/
-void __attribute__ ((weak)) arch_segment_loaded(uintptr_t start, size_t size,
- int flags)
-{
- /* do nothing */
-}
diff --git a/src/lib/loaders/load_and_run_payload.c b/src/lib/loaders/load_and_run_payload.c
index 8e745c9..e3208a9 100644
--- a/src/lib/loaders/load_and_run_payload.c
+++ b/src/lib/loaders/load_and_run_payload.c
@@ -83,7 +83,7 @@ out:
void payload_run(void)
{
- const struct payload *payload = &global_payload;
+ struct payload *payload = &global_payload;
/* Reset to booting from this image as late as possible */
boot_successful();
diff --git a/src/lib/loaders/load_and_run_ramstage.c b/src/lib/loaders/load_and_run_ramstage.c
index 82bc1e0..9067a28 100644
--- a/src/lib/loaders/load_and_run_ramstage.c
+++ b/src/lib/loaders/load_and_run_ramstage.c
@@ -48,7 +48,7 @@ load_ramstage(const struct ramstage_loader_ops *ops,
timestamp_add_now(TS_END_COPYRAM);
- stage_exit(prog_entry(ramstage));
+ prog_run(ramstage);
}
static void run_ramstage_from_resume(struct romstage_handoff *handoff,
@@ -60,7 +60,7 @@ static void run_ramstage_from_resume(struct romstage_handoff *handoff,
if (prog_entry(ramstage) != NULL) {
printk(BIOS_DEBUG, "Jumping to image.\n");
- stage_exit(prog_entry(ramstage));
+ prog_run(ramstage);
}
}
}
diff --git a/src/lib/loaders/load_and_run_romstage.c b/src/lib/loaders/load_and_run_romstage.c
index 3ad9176..8467e90 100644
--- a/src/lib/loaders/load_and_run_romstage.c
+++ b/src/lib/loaders/load_and_run_romstage.c
@@ -38,5 +38,5 @@ void run_romstage(void)
halt();
}
- stage_exit(prog_entry(&romstage));
+ prog_run(&romstage);
}
diff --git a/src/lib/prog_ops.c b/src/lib/prog_ops.c
new file mode 100644
index 0000000..755b135
--- /dev/null
+++ b/src/lib/prog_ops.c
@@ -0,0 +1,39 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2014 Imagination Technologies
+ * Copyright 2015 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#include <program_loading.h>
+
+/* For each segment of a program loaded this function is called*/
+void __attribute__ ((weak)) arch_segment_loaded(uintptr_t start, size_t size,
+ int flags)
+{
+ /* do nothing */
+}
+
+void prog_run(struct prog *prog)
+{
+ platform_prog_run(prog);
+ arch_prog_run(prog);
+}
+
+void __attribute__ ((weak)) platform_prog_run(struct prog *prog)
+{
+ /* do nothing */
+}
More information about the coreboot-gerrit
mailing list