Add a new 'upl exec' command which loads a payload, sets up the handoff information and jumps to it.
This collects image information as the FIT's images are loaded.
With this, the size of the FIT is not known so is reported through UPL as zero. Future work will address this.
Signed-off-by: Simon Glass sjg@chromium.org ---
boot/Kconfig | 8 +++ boot/Makefile | 1 + boot/upl_exec.c | 130 +++++++++++++++++++++++++++++++++++++++++++++++ boot/upl_write.c | 2 +- cmd/upl.c | 25 ++++++++- include/upl.h | 29 ++++++++++- 6 files changed, 191 insertions(+), 4 deletions(-) create mode 100644 boot/upl_exec.c
diff --git a/boot/Kconfig b/boot/Kconfig index f9263513268..983bd9cdbf6 100644 --- a/boot/Kconfig +++ b/boot/Kconfig @@ -765,6 +765,7 @@ config BOOTMETH_SCRIPT config UPL bool "upl - Universal Payload Specification" imply CMD_UPL + imply UPL_EXEC imply UPL_READ imply UPL_WRITE imply SPL_UPL if SPL @@ -794,6 +795,13 @@ config UPL_WRITE for how to tell U-Boot SPL to actually write it before jumping to the next phase.
+config UPL_EXEC + bool "upl - Support for executing a UPL image" + help + Provides support for executing a loaded UPL image, passing it a + suitable handoff block. This can be used to pass control to a UPL from + another firmware project. + config UPL_IN bool "upl - Read the UPL handoff on startup" select UPL_READ diff --git a/boot/Makefile b/boot/Makefile index 9446c6b82a9..bc4ff91a934 100644 --- a/boot/Makefile +++ b/boot/Makefile @@ -42,6 +42,7 @@ obj-$(CONFIG_$(PHASE_)OF_LIBFDT) += fdt_support.o obj-$(CONFIG_$(PHASE_)FDT_SIMPLEFB) += fdt_simplefb.o
obj-$(CONFIG_$(PHASE_)UPL) += upl_common.o +obj-$(CONFIG_$(PHASE_)UPL_EXEC) += upl_exec.o obj-$(CONFIG_$(PHASE_)UPL_READ) += upl_read.o obj-$(CONFIG_$(PHASE_)UPL_WRITE) += upl_write.o
diff --git a/boot/upl_exec.c b/boot/upl_exec.c new file mode 100644 index 00000000000..8eb72a4cb16 --- /dev/null +++ b/boot/upl_exec.c @@ -0,0 +1,130 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * UPL handoff parsing + * + * Copyright 2024 Google LLC + * Written by Simon Glass sjg@chromium.org + */ + +#define LOG_CATEGORY UCLASS_BOOTSTD + +#include <bootstage.h> +#include <cpu.h> +#include <display_options.h> +#include <dm.h> +#include <image.h> +#include <log.h> +#include <mapmem.h> +#include <serial.h> +#include <upl.h> +#include <video.h> +#include <asm/global_data.h> +#include <dm/root.h> + +DECLARE_GLOBAL_DATA_PTR; + +struct upl *cur_upl; + +static void upl_prepare(const struct upl_image *img, const struct abuf *buf) +{ + printf("\nUPL: handoff at %lx size %zx\n", abuf_addr(buf), buf->size); + printf("Starting at %lx ...\n\n", img->entry); + + bootstage_mark_name(BOOTSTAGE_ID_BOOTM_HANDOFF, "upl_prepare"); + if (IS_ENABLED(CONFIG_BOOTSTAGE_REPORT)) + bootstage_report(); + + /* + * Call remove function of all devices with a removal flag set. + * This may be useful for last-stage operations, like cancelling + * of DMA operation or releasing device internal buffers. + */ + dm_remove_devices_active(); +} + +int _upl_add_image(int node, ulong load_addr, ulong size, const char *desc) +{ + struct upl_image img; + + if (!cur_upl) + return 0; + + memset(&img, '\0', sizeof(img)); + img.reg.base = load_addr; + img.reg.size = size; + img.offset = node; + img.description = desc; + if (!alist_add(&cur_upl->image, img)) + return -ENOMEM; + log_debug("upl: add image %s at %lx size %lx\n", desc, load_addr, size); + + return 0; +} + +int upl_exec(ulong addr) +{ + struct bootm_headers images; + struct upl s_upl, *upl = &s_upl; + struct upl_image *img; + ulong img_data, img_len; + const char *uname; + struct abuf buf; + int ret; + + upl_init(upl); + + cur_upl = upl; + uname = NULL; + memset(&images, '\0', sizeof(images)); + images.fit_uname_cfg = "conf-1"; + + /* a side-effect of this function is to call _upl_add_image() above */ + ret = fit_image_load(&images, addr, &uname, &images.fit_uname_cfg, + IH_ARCH_DEFAULT, IH_TYPE_FIRMWARE, + BOOTSTAGE_ID_FIT_KERNEL_START, + FIT_LOAD_OPTIONAL_NON_ZERO, &img_data, &img_len); + if (ret < 0) { + cur_upl = NULL; + return log_msg_retz("ufi", ret); + } + + /* add the FIT */ + upl->fit.base = addr; + upl->fit.size = 0; /* TODO(sjg@chromium.org): Add size */ + + images.fit_hdr_os = map_sysmem(addr, 0); + img = alist_getw(&upl->image, 0, struct upl_image); + if (!img) + return log_msg_ret("uim", ret); + + if (fit_image_get_entry(images.fit_hdr_os, ret, &img->entry)) + return log_msg_ret("uae", -ENOENT); + log_debug("entry %lx\n", img->entry); + + /* this calls _upl_add_image() with each loaded image */ + ret = boot_get_loadable(&images); + cur_upl = NULL; + if (ret) + return log_msg_ret("ulo", ret); + + /* we now have a full handoff state, including the images */ + ret = upl_create_handoff(upl, &buf); + if (ret) + return log_msg_ret("uec", ret); + + if (_DEBUG) { + set_working_fdt_addr(abuf_addr(&buf)); + run_command("fdt print", 0); + } + + upl_prepare(img, &buf); + + if (IS_ENABLED(CONFIG_X86)) { + ret = arch_upl_jump(img->entry, &buf); + } else { + printf("UPL is not supported on this architecture\n"); + ret = -ENOSYS; + } + + return ret; +} diff --git a/boot/upl_write.c b/boot/upl_write.c index c43a588dc40..ce9f733b450 100644 --- a/boot/upl_write.c +++ b/boot/upl_write.c @@ -685,7 +685,7 @@ int upl_create_handoff_tree(const struct upl *upl, oftree *treep) return 0; }
-int upl_create_handoff(struct upl *upl, ulong addr, struct abuf *buf) +int upl_create_handoff(struct upl *upl, struct abuf *buf) { oftree tree; int ret; diff --git a/cmd/upl.c b/cmd/upl.c index 461f8f4f0aa..e64ac7bbd20 100644 --- a/cmd/upl.c +++ b/cmd/upl.c @@ -112,12 +112,33 @@ static int do_upl_read(struct cmd_tbl *cmdtp, int flag, int argc, return 0; }
+static int do_upl_exec(struct cmd_tbl *cmdtp, int flag, int argc, + char *const argv[]) +{ + ulong addr; + int ret; + + if (argc < 2) + return CMD_RET_USAGE; + + addr = hextoul(argv[1], NULL); + ret = upl_exec(addr); + if (ret) { + printf("Failed (err=%dE)\n", ret); + return CMD_RET_FAILURE; + } + + return 0; +} + U_BOOT_LONGHELP(upl, "info [-v] - Check UPL status\n" "upl read <addr> - Read handoff information\n" - "upl write - Write handoff information"); + "upl write - Write handoff information\n" + "upl exec - Execute a loaded UPL");
U_BOOT_CMD_WITH_SUBCMDS(upl, "Universal Payload support", upl_help_text, U_BOOT_SUBCMD_MKENT(info, 2, 1, do_upl_info), U_BOOT_SUBCMD_MKENT(read, 2, 1, do_upl_read), - U_BOOT_SUBCMD_MKENT(write, 1, 1, do_upl_write)); + U_BOOT_SUBCMD_MKENT(write, 1, 1, do_upl_write), + U_BOOT_SUBCMD_MKENT(exec, 2, 1, do_upl_exec)); diff --git a/include/upl.h b/include/upl.h index b9bcdf60089..f210a39df0c 100644 --- a/include/upl.h +++ b/include/upl.h @@ -347,6 +347,14 @@ int upl_create_handoff_tree(const struct upl *upl, oftree *treep); */ int upl_read_handoff(struct upl *upl, oftree tree);
+/** + * upl_exec() - Execulated a loaded UPL image + * + * @addr: Address of image + * Return: 0 on success, or -ve error + */ +int upl_exec(ulong addr); + /** * upl_get_test_data() - Fill a UPL with some test data * @@ -404,7 +412,7 @@ int _upl_add_image(int node, ulong load_addr, ulong size, const char *desc); static inline int upl_add_image(const void *fit, int node, ulong load_addr, ulong size) { - if (CONFIG_IS_ENABLED(UPL) && IS_ENABLED(CONFIG_XPL_BUILD)) { + if (CONFIG_IS_ENABLED(UPL)) { const char *desc = fdt_getprop(fit, node, FIT_DESC_PROP, NULL);
return _upl_add_image(node, load_addr, size, desc); @@ -469,6 +477,25 @@ int upl_write_to_buf(struct upl *upl, ofnode root, struct abuf *buf); int upl_add_region(struct alist *lst, u64 base, ulong size); #endif
+/** + * upl_create_handoff() - Create a UPL handoff + * + * @upl: Returns the handoff structure that was created + * @buf: Returns buffer containing the final handoff info + * Return: 0 if OK, -ve on error + */ +int upl_create_handoff(struct upl *upl, struct abuf *buf); + +/** + * arch_upl_jump() - Jump to the UPL payload + * + * Jump to + * + * @entry: Address to jump to + * @buf: Buffer containing UPL-handoff information + */ +int arch_upl_jump(ulong entry, const struct abuf *buf); + /** upl_init() - Set up a UPL struct */ void upl_init(struct upl *upl);