efi: disentangle the generic EFI stub from FDT

LoongArch does not use FDT or DT natively [yet], and the only reason it currently uses it is so that it can reuse the existing EFI stub code.

Overloading the DT with data passed between the EFI stub and the core kernel has been a source of problems: there is the overlap between information provided by EFI which DT can also provide (initrd base/size, command line, memory descriptions), requiring us to reason about which is which and what to prioritize. It has also resulted in ABI leaks, i.e., internal ABI being promoted to external ABI inadvertently because the bootloader can set the EFI stub's DT properties as well (e.g., "kaslr-seed"). This has become especially problematic with boot environments that want to pretend that EFI boot is being done (to access ACPI and SMBIOS tables, for instance) but have no ability to execute the EFI stub, and so the environment that the EFI stub creates is emulated [poorly, in some cases].

Another downside of treating DT like this is that the DT binary that the kernel receives is different from the one created by the firmware, which is undesirable in the context of secure and measured boot.

Given that LoongArch support in Linux is brand new, we can avoid these pitfalls, and treat the DT strictly as a hardware description, and use a separate handover method between the EFI stub and the kernel. Now that initrd loading and passing the EFI memory map have been refactored into pure EFI routines that use EFI configuration tables, the only thing we need to pass directly is the kernel command line (even if we could pass this via a config table as well, it is used extremely early, so passing it directly is preferred in this case.)

Signed-off-by Ard Biesheuvel <[email protected]>

---
 arch/loongarch/Kconfig                        |  3 --
 arch/loongarch/include/asm/bootinfo.h         |  2 +-
 arch/loongarch/kernel/efi.c                   | 30 ++++++++++-
 arch/loongarch/kernel/env.c                   | 13 ++---
 arch/loongarch/kernel/head.S                  |  2 +
 arch/loongarch/kernel/setup.c                 |  4 +-
 drivers/firmware/efi/libstub/Makefile         | 13 +++--
 drivers/firmware/efi/libstub/loongarch-stub.c | 56 +++++++++++++++++---
 8 files changed, 95 insertions(+), 28 deletions(-)

arch/loongarch/Kconfig +0 -3

index fca106a8b8af..14a2a1ec8561 100644
@@ -104,8 +104,6 @@ config LOONGARCH
        select MODULES_USE_ELF_RELA if MODULES
        select NEED_PER_CPU_EMBED_FIRST_CHUNK
        select NEED_PER_CPU_PAGE_FIRST_CHUNK
-       select OF
-       select OF_EARLY_FLATTREE
        select PCI
        select PCI_DOMAINS_GENERIC
        select PCI_ECAM if ACPI

@@ -311,7 +309,6 @@ config DMI
 config EFI
        bool "EFI runtime service support"
        select UCS2_STRING
-       select EFI_PARAMS_FROM_FDT
        select EFI_RUNTIME_WRAPPERS
        help
          This enables the kernel to use EFI runtime services that are

arch/loongarch/include/asm/bootinfo.h +1 -1

index e02ac4af7f6e..8e5881bc5ad1 100644
@@ -36,7 +36,7 @@ struct loongson_system_configuration {
 };
 
 extern u64 efi_system_table;
-extern unsigned long fw_arg0, fw_arg1;
+extern unsigned long fw_arg0, fw_arg1, fw_arg2;
 extern struct loongson_board_info b_info;
 extern struct loongson_system_configuration loongson_sysconf;
 

arch/loongarch/kernel/efi.c +29 -1

index 1f1f755fb425..a31329971133 100644
@@ -27,8 +27,13 @@
 static unsigned long efi_nr_tables;
 static unsigned long efi_config_table;
 
+static unsigned long __initdata boot_memmap = EFI_INVALID_TABLE_ADDR;
+
 static efi_system_table_t *efi_systab;
-static efi_config_table_type_t arch_tables[] __initdata = {{},};
+static efi_config_table_type_t arch_tables[] __initdata = {
+       {LINUX_EFI_BOOT_MEMMAP_GUID,    &boot_memmap,   "MEMMAP" },
+       {},
+};
 
 void __init efi_runtime_init(void)
 {

@@ -51,6 +56,7 @@ void __init efi_init(void)
 {
        int size;
        void *config_tables;
+       struct efi_boot_memmap *tbl;
 
        if (!efi_system_table)
                return;

@@ -61,6 +67,8 @@ void __init efi_init(void)
                return;
        }
 
+       efi_systab_report_header(&efi_systab->hdr, efi_systab->fw_vendor);
+
        set_bit(EFI_64BIT, &efi.flags);
        efi_nr_tables    = efi_systab->nr_tables;
        efi_config_table = (unsigned long)efi_systab->tables;

@@ -70,6 +78,26 @@ void __init efi_init(void)
        efi_config_parse_tables(config_tables, efi_systab->nr_tables, arch_tables);
        early_memunmap(config_tables, efi_nr_tables * size);
 
+       set_bit(EFI_CONFIG_TABLES, &efi.flags);
+
        if (screen_info.orig_video_isVGA == VIDEO_TYPE_EFI)
                memblock_reserve(screen_info.lfb_base, screen_info.lfb_size);
+
+       if (boot_memmap == EFI_INVALID_TABLE_ADDR)
+               return;
+
+       tbl = early_memremap_ro(boot_memmap, sizeof(*tbl));
+       if (tbl) {
+               struct efi_memory_map_data data;
+
+               data.phys_map           = boot_memmap + sizeof(*tbl);
+               data.size               = tbl->map_size;
+               data.desc_size          = tbl->desc_size;
+               data.desc_version       = tbl->desc_ver;
+
+               if (efi_memmap_init_early(&data) < 0)
+                       panic("Unable to map EFI memory map.\n");
+
+               early_memunmap(tbl, sizeof(*tbl));
+       }
 }

arch/loongarch/kernel/env.c +4 -9

index 82b478a5c665..6d56a463b091 100644
@@ -8,7 +8,6 @@
 #include <linux/efi.h>
 #include <linux/export.h>
 #include <linux/memblock.h>
-#include <linux/of_fdt.h>
 #include <asm/early_ioremap.h>
 #include <asm/bootinfo.h>
 #include <asm/loongson.h>

@@ -20,21 +19,17 @@ EXPORT_SYMBOL(loongson_sysconf);
 void __init init_environ(void)
 {
        int efi_boot = fw_arg0;
-       struct efi_memory_map_data data;
-       void *fdt_ptr = early_memremap_ro(fw_arg1, SZ_64K);
+       char *cmdline = early_memremap_ro(fw_arg1, COMMAND_LINE_SIZE);
 
        if (efi_boot)
                set_bit(EFI_BOOT, &efi.flags);
        else
                clear_bit(EFI_BOOT, &efi.flags);
 
-       early_init_dt_scan(fdt_ptr);
-       early_init_fdt_reserve_self();
-       efi_system_table = efi_get_fdt_params(&data);
+       strscpy(boot_command_line, cmdline, COMMAND_LINE_SIZE);
+       early_memunmap(cmdline, COMMAND_LINE_SIZE);
 
-       efi_memmap_init_early(&data);
-       memblock_reserve(data.phys_map & PAGE_MASK,
-                        PAGE_ALIGN(data.size + (data.phys_map & ~PAGE_MASK)));
+       efi_system_table = fw_arg2;
 }
 
 static int __init init_cpu_fullname(void)

arch/loongarch/kernel/head.S +2 -0

index 01bac62a6442..8f89f39fd31b 100644
@@ -67,6 +67,8 @@ SYM_CODE_START(kernel_entry)                  # kernel entry point
        st.d            a0, t0, 0               # firmware arguments
        la              t0, fw_arg1
        st.d            a1, t0, 0
+       la              t0, fw_arg2
+       st.d            a2, t0, 0
 
        /* KSave3 used for percpu base, initialized as 0 */
        csrwr           zero, PERCPU_BASE_KS

arch/loongarch/kernel/setup.c +2 -2

index e8714b1d94c8..7fabf2306e80 100644
@@ -51,7 +51,7 @@
 
 struct screen_info screen_info __section(".data");
 
-unsigned long fw_arg0, fw_arg1;
+unsigned long fw_arg0, fw_arg1, fw_arg2;
 DEFINE_PER_CPU(unsigned long, kernelsp);
 struct cpuinfo_loongarch cpu_data[NR_CPUS] __read_mostly;
 

@@ -187,7 +187,6 @@ early_param("mem", early_parse_mem);
 
 void __init platform_init(void)
 {
-       efi_init();
 #ifdef CONFIG_ACPI_TABLE_UPGRADE
        acpi_table_upgrade();
 #endif

@@ -347,6 +346,7 @@ void __init setup_arch(char **cmdline_p)
        *cmdline_p = boot_command_line;
 
        init_environ();
+       efi_init();
        memblock_init();
        parse_early_param();
 

drivers/firmware/efi/libstub/Makefile +8 -5

index ec2a7ba9364f..6234edf3d827 100644
@@ -29,7 +29,7 @@ cflags-$(CONFIG_RISCV)                := $(subst $(CC_FLAGS_FTRACE),,$(KBUILD_CFLAGS)) \
 cflags-$(CONFIG_LOONGARCH)     := $(subst $(CC_FLAGS_FTRACE),,$(KBUILD_CFLAGS)) \
                                   -fpie
 
-cflags-$(CONFIG_EFI_GENERIC_STUB) += -I$(srctree)/scripts/dtc/libfdt
+cflags-$(CONFIG_EFI_PARAMS_FROM_FDT)   += -I$(srctree)/scripts/dtc/libfdt
 
 KBUILD_CFLAGS                  := $(cflags-y) -Os -DDISABLE_BRANCH_PROFILING \
                                   -include $(srctree)/include/linux/hidden.h \

@@ -59,14 +59,17 @@ lib-y                               := efi-stub-helper.o gop.o secureboot.o tpm.o \
                                   skip_spaces.o lib-cmdline.o lib-ctype.o \
                                   alignedmem.o relocate.o vsprintf.o
 
-# include the stub's generic dependencies from lib/ when building for ARM/arm64
-efi-deps-y := fdt_rw.c fdt_ro.c fdt_wip.c fdt.c fdt_empty_tree.c fdt_sw.c
+# include the stub's libfdt dependencies from lib/ when needed
+libfdt-deps                    := fdt_rw.c fdt_ro.c fdt_wip.c fdt.c \
+                                  fdt_empty_tree.c fdt_sw.c
+
+lib-$(CONFIG_EFI_PARAMS_FROM_FDT) += fdt.o \
+                                    $(patsubst %.c,lib-%.o,$(libfdt-deps))
 
 $(obj)/lib-%.o: $(srctree)/lib/%.c FORCE
        $(call if_changed_rule,cc_o_c)
 
-lib-$(CONFIG_EFI_GENERIC_STUB) += efi-stub.o fdt.o string.o \
-                                  $(patsubst %.c,lib-%.o,$(efi-deps-y))
+lib-$(CONFIG_EFI_GENERIC_STUB) += efi-stub.o string.o
 
 lib-$(CONFIG_ARM)              += arm32-stub.o
 lib-$(CONFIG_ARM64)            += arm64-stub.o

drivers/firmware/efi/libstub/loongarch-stub.c +49 -8

index b7ef8d2df59e..32329f2a92f9 100644
@@ -9,7 +9,8 @@
 #include <asm/addrspace.h>
 #include "efistub.h"
 
-typedef void __noreturn (*kernel_entry_t)(bool efi, unsigned long fdt);
+typedef void __noreturn (*kernel_entry_t)(bool efi, unsigned long cmdline,
+                                         unsigned long systab);
 
 extern int kernel_asize;
 extern int kernel_fsize;

@@ -42,19 +43,60 @@ efi_status_t handle_kernel_image(unsigned long *image_addr,
        return status;
 }
 
-void __noreturn efi_enter_kernel(unsigned long entrypoint, unsigned long fdt, unsigned long fdt_size)
+struct exit_boot_struct {
+       efi_memory_desc_t       *runtime_map;
+       int                     runtime_entry_count;
+};
+
+static efi_status_t exit_boot_func(struct efi_boot_memmap *map, void *priv)
+{
+       struct exit_boot_struct *p = priv;
+
+       /*
+        * Update the memory map with virtual addresses. The function will also
+        * populate @runtime_map with copies of just the EFI_MEMORY_RUNTIME
+        * entries so that we can pass it straight to SetVirtualAddressMap()
+        */
+       efi_get_virtmap(map->map, map->map_size, map->desc_size,
+                       p->runtime_map, &p->runtime_entry_count);
+
+       return EFI_SUCCESS;
+}
+
+efi_status_t efi_boot_kernel(void *handle, efi_loaded_image_t *image,
+                            unsigned long kernel_addr, char *cmdline_ptr)
 {
        kernel_entry_t real_kernel_entry;
+       struct exit_boot_struct priv;
+       unsigned long desc_size;
+       efi_status_t status;
+       u32 desc_ver;
+
+       status = efi_alloc_virtmap(&priv.runtime_map, &desc_size, &desc_ver);
+       if (status != EFI_SUCCESS) {
+               efi_err("Unable to retrieve UEFI memory map.\n");
+               return status;
+       }
+
+       efi_info("Exiting boot services\n");
+
+       efi_novamap = false;
+       status = efi_exit_boot_services(handle, &priv, exit_boot_func);
+       if (status != EFI_SUCCESS)
+               return status;
+
+       /* Install the new virtual address map */
+       efi_rt_call(set_virtual_address_map,
+                   priv.runtime_entry_count * desc_size, desc_size,
+                   desc_ver, priv.runtime_map);
 
        /* Config Direct Mapping */
        csr_write64(CSR_DMW0_INIT, LOONGARCH_CSR_DMWIN0);
        csr_write64(CSR_DMW1_INIT, LOONGARCH_CSR_DMWIN1);
 
        real_kernel_entry = (kernel_entry_t)
-               ((unsigned long)&kernel_entry - entrypoint + VMLINUX_LOAD_ADDRESS);
+               ((unsigned long)&kernel_entry - kernel_addr + VMLINUX_LOAD_ADDRESS);
 
-       if (!efi_novamap)
-               real_kernel_entry(true, fdt);
-       else
-               real_kernel_entry(false, fdt);
+       real_kernel_entry(true, (unsigned long)cmdline_ptr,
+                         (unsigned long)efi_system_table);
 }
--

2.35.1

Acked-by Huacai Chen <[email protected]>

Quoted 339 lines

LoongArch does not use FDT or DT natively [yet], and the only reason it currently uses it is so that it can reuse the existing EFI stub code.

Overloading the DT with data passed between the EFI stub and the core kernel has been a source of problems: there is the overlap between information provided by EFI which DT can also provide (initrd base/size, command line, memory descriptions), requiring us to reason about which is which and what to prioritize. It has also resulted in ABI leaks, i.e., internal ABI being promoted to external ABI inadvertently because the bootloader can set the EFI stub's DT properties as well (e.g., "kaslr-seed"). This has become especially problematic with boot environments that want to pretend that EFI boot is being done (to access ACPI and SMBIOS tables, for instance) but have no ability to execute the EFI stub, and so the environment that the EFI stub creates is emulated [poorly, in some cases].

Another downside of treating DT like this is that the DT binary that the kernel receives is different from the one created by the firmware, which is undesirable in the context of secure and measured boot.

Given that LoongArch support in Linux is brand new, we can avoid these pitfalls, and treat the DT strictly as a hardware description, and use a separate handover method between the EFI stub and the kernel. Now that initrd loading and passing the EFI memory map have been refactored into pure EFI routines that use EFI configuration tables, the only thing we need to pass directly is the kernel command line (even if we could pass this via a config table as well, it is used extremely early, so passing it directly is preferred in this case.)

Signed-off-by Ard Biesheuvel <[email protected]>

---
 arch/loongarch/Kconfig                        |  3 --
 arch/loongarch/include/asm/bootinfo.h         |  2 +-
 arch/loongarch/kernel/efi.c                   | 30 ++++++++++-
 arch/loongarch/kernel/env.c                   | 13 ++---
 arch/loongarch/kernel/head.S                  |  2 +
 arch/loongarch/kernel/setup.c                 |  4 +-
 drivers/firmware/efi/libstub/Makefile         | 13 +++--
 drivers/firmware/efi/libstub/loongarch-stub.c | 56 +++++++++++++++++---
 8 files changed, 95 insertions(+), 28 deletions(-)

arch/loongarch/Kconfig +0 -3

index fca106a8b8af..14a2a1ec8561 100644
@@ -104,8 +104,6 @@ config LOONGARCH
        select MODULES_USE_ELF_RELA if MODULES
        select NEED_PER_CPU_EMBED_FIRST_CHUNK
        select NEED_PER_CPU_PAGE_FIRST_CHUNK
-       select OF
-       select OF_EARLY_FLATTREE
        select PCI
        select PCI_DOMAINS_GENERIC
        select PCI_ECAM if ACPI

@@ -311,7 +309,6 @@ config DMI
 config EFI
        bool "EFI runtime service support"
        select UCS2_STRING
-       select EFI_PARAMS_FROM_FDT
        select EFI_RUNTIME_WRAPPERS
        help
          This enables the kernel to use EFI runtime services that are

arch/loongarch/include/asm/bootinfo.h +1 -1

index e02ac4af7f6e..8e5881bc5ad1 100644
@@ -36,7 +36,7 @@ struct loongson_system_configuration {
 };
 
 extern u64 efi_system_table;
-extern unsigned long fw_arg0, fw_arg1;
+extern unsigned long fw_arg0, fw_arg1, fw_arg2;
 extern struct loongson_board_info b_info;
 extern struct loongson_system_configuration loongson_sysconf;
 

arch/loongarch/kernel/efi.c +29 -1

index 1f1f755fb425..a31329971133 100644
@@ -27,8 +27,13 @@
 static unsigned long efi_nr_tables;
 static unsigned long efi_config_table;
 
+static unsigned long __initdata boot_memmap = EFI_INVALID_TABLE_ADDR;
+
 static efi_system_table_t *efi_systab;
-static efi_config_table_type_t arch_tables[] __initdata = {{},};
+static efi_config_table_type_t arch_tables[] __initdata = {
+       {LINUX_EFI_BOOT_MEMMAP_GUID,    &boot_memmap,   "MEMMAP" },
+       {},
+};
 
 void __init efi_runtime_init(void)
 {

@@ -51,6 +56,7 @@ void __init efi_init(void)
 {
        int size;
        void *config_tables;
+       struct efi_boot_memmap *tbl;
 
        if (!efi_system_table)
                return;

@@ -61,6 +67,8 @@ void __init efi_init(void)
                return;
        }
 
+       efi_systab_report_header(&efi_systab->hdr, efi_systab->fw_vendor);
+
        set_bit(EFI_64BIT, &efi.flags);
        efi_nr_tables    = efi_systab->nr_tables;
        efi_config_table = (unsigned long)efi_systab->tables;

@@ -70,6 +78,26 @@ void __init efi_init(void)
        efi_config_parse_tables(config_tables, efi_systab->nr_tables, arch_tables);
        early_memunmap(config_tables, efi_nr_tables * size);
 
+       set_bit(EFI_CONFIG_TABLES, &efi.flags);
+
        if (screen_info.orig_video_isVGA == VIDEO_TYPE_EFI)
                memblock_reserve(screen_info.lfb_base, screen_info.lfb_size);
+
+       if (boot_memmap == EFI_INVALID_TABLE_ADDR)
+               return;
+
+       tbl = early_memremap_ro(boot_memmap, sizeof(*tbl));
+       if (tbl) {
+               struct efi_memory_map_data data;
+
+               data.phys_map           = boot_memmap + sizeof(*tbl);
+               data.size               = tbl->map_size;
+               data.desc_size          = tbl->desc_size;
+               data.desc_version       = tbl->desc_ver;
+
+               if (efi_memmap_init_early(&data) < 0)
+                       panic("Unable to map EFI memory map.\n");
+
+               early_memunmap(tbl, sizeof(*tbl));
+       }
 }

arch/loongarch/kernel/env.c +4 -9

index 82b478a5c665..6d56a463b091 100644
@@ -8,7 +8,6 @@
 #include <linux/efi.h>
 #include <linux/export.h>
 #include <linux/memblock.h>
-#include <linux/of_fdt.h>
 #include <asm/early_ioremap.h>
 #include <asm/bootinfo.h>
 #include <asm/loongson.h>

@@ -20,21 +19,17 @@ EXPORT_SYMBOL(loongson_sysconf);
 void __init init_environ(void)
 {
        int efi_boot = fw_arg0;
-       struct efi_memory_map_data data;
-       void *fdt_ptr = early_memremap_ro(fw_arg1, SZ_64K);
+       char *cmdline = early_memremap_ro(fw_arg1, COMMAND_LINE_SIZE);
 
        if (efi_boot)
                set_bit(EFI_BOOT, &efi.flags);
        else
                clear_bit(EFI_BOOT, &efi.flags);
 
-       early_init_dt_scan(fdt_ptr);
-       early_init_fdt_reserve_self();
-       efi_system_table = efi_get_fdt_params(&data);
+       strscpy(boot_command_line, cmdline, COMMAND_LINE_SIZE);
+       early_memunmap(cmdline, COMMAND_LINE_SIZE);
 
-       efi_memmap_init_early(&data);
-       memblock_reserve(data.phys_map & PAGE_MASK,
-                        PAGE_ALIGN(data.size + (data.phys_map & ~PAGE_MASK)));
+       efi_system_table = fw_arg2;
 }
 
 static int __init init_cpu_fullname(void)

arch/loongarch/kernel/head.S +2 -0

index 01bac62a6442..8f89f39fd31b 100644
@@ -67,6 +67,8 @@ SYM_CODE_START(kernel_entry)                  # kernel entry point
        st.d            a0, t0, 0               # firmware arguments
        la              t0, fw_arg1
        st.d            a1, t0, 0
+       la              t0, fw_arg2
+       st.d            a2, t0, 0
 
        /* KSave3 used for percpu base, initialized as 0 */
        csrwr           zero, PERCPU_BASE_KS

arch/loongarch/kernel/setup.c +2 -2

index e8714b1d94c8..7fabf2306e80 100644
@@ -51,7 +51,7 @@
 
 struct screen_info screen_info __section(".data");
 
-unsigned long fw_arg0, fw_arg1;
+unsigned long fw_arg0, fw_arg1, fw_arg2;
 DEFINE_PER_CPU(unsigned long, kernelsp);
 struct cpuinfo_loongarch cpu_data[NR_CPUS] __read_mostly;
 

@@ -187,7 +187,6 @@ early_param("mem", early_parse_mem);
 
 void __init platform_init(void)
 {
-       efi_init();
 #ifdef CONFIG_ACPI_TABLE_UPGRADE
        acpi_table_upgrade();
 #endif

@@ -347,6 +346,7 @@ void __init setup_arch(char **cmdline_p)
        *cmdline_p = boot_command_line;
 
        init_environ();
+       efi_init();
        memblock_init();
        parse_early_param();
 

drivers/firmware/efi/libstub/Makefile +8 -5

index ec2a7ba9364f..6234edf3d827 100644
@@ -29,7 +29,7 @@ cflags-$(CONFIG_RISCV)                := $(subst $(CC_FLAGS_FTRACE),,$(KBUILD_CFLAGS)) \
 cflags-$(CONFIG_LOONGARCH)     := $(subst $(CC_FLAGS_FTRACE),,$(KBUILD_CFLAGS)) \
                                   -fpie
 
-cflags-$(CONFIG_EFI_GENERIC_STUB) += -I$(srctree)/scripts/dtc/libfdt
+cflags-$(CONFIG_EFI_PARAMS_FROM_FDT)   += -I$(srctree)/scripts/dtc/libfdt
 
 KBUILD_CFLAGS                  := $(cflags-y) -Os -DDISABLE_BRANCH_PROFILING \
                                   -include $(srctree)/include/linux/hidden.h \

@@ -59,14 +59,17 @@ lib-y                               := efi-stub-helper.o gop.o secureboot.o tpm.o \
                                   skip_spaces.o lib-cmdline.o lib-ctype.o \
                                   alignedmem.o relocate.o vsprintf.o
 
-# include the stub's generic dependencies from lib/ when building for ARM/arm64
-efi-deps-y := fdt_rw.c fdt_ro.c fdt_wip.c fdt.c fdt_empty_tree.c fdt_sw.c
+# include the stub's libfdt dependencies from lib/ when needed
+libfdt-deps                    := fdt_rw.c fdt_ro.c fdt_wip.c fdt.c \
+                                  fdt_empty_tree.c fdt_sw.c
+
+lib-$(CONFIG_EFI_PARAMS_FROM_FDT) += fdt.o \
+                                    $(patsubst %.c,lib-%.o,$(libfdt-deps))
 
 $(obj)/lib-%.o: $(srctree)/lib/%.c FORCE
        $(call if_changed_rule,cc_o_c)
 
-lib-$(CONFIG_EFI_GENERIC_STUB) += efi-stub.o fdt.o string.o \
-                                  $(patsubst %.c,lib-%.o,$(efi-deps-y))
+lib-$(CONFIG_EFI_GENERIC_STUB) += efi-stub.o string.o
 
 lib-$(CONFIG_ARM)              += arm32-stub.o
 lib-$(CONFIG_ARM64)            += arm64-stub.o

drivers/firmware/efi/libstub/loongarch-stub.c +49 -8

index b7ef8d2df59e..32329f2a92f9 100644
@@ -9,7 +9,8 @@
 #include <asm/addrspace.h>
 #include "efistub.h"
 
-typedef void __noreturn (*kernel_entry_t)(bool efi, unsigned long fdt);
+typedef void __noreturn (*kernel_entry_t)(bool efi, unsigned long cmdline,
+                                         unsigned long systab);
 
 extern int kernel_asize;
 extern int kernel_fsize;

@@ -42,19 +43,60 @@ efi_status_t handle_kernel_image(unsigned long *image_addr,
        return status;
 }
 
-void __noreturn efi_enter_kernel(unsigned long entrypoint, unsigned long fdt, unsigned long fdt_size)
+struct exit_boot_struct {
+       efi_memory_desc_t       *runtime_map;
+       int                     runtime_entry_count;
+};
+
+static efi_status_t exit_boot_func(struct efi_boot_memmap *map, void *priv)
+{
+       struct exit_boot_struct *p = priv;
+
+       /*
+        * Update the memory map with virtual addresses. The function will also
+        * populate @runtime_map with copies of just the EFI_MEMORY_RUNTIME
+        * entries so that we can pass it straight to SetVirtualAddressMap()
+        */
+       efi_get_virtmap(map->map, map->map_size, map->desc_size,
+                       p->runtime_map, &p->runtime_entry_count);
+
+       return EFI_SUCCESS;
+}
+
+efi_status_t efi_boot_kernel(void *handle, efi_loaded_image_t *image,
+                            unsigned long kernel_addr, char *cmdline_ptr)
 {
        kernel_entry_t real_kernel_entry;
+       struct exit_boot_struct priv;
+       unsigned long desc_size;
+       efi_status_t status;
+       u32 desc_ver;
+
+       status = efi_alloc_virtmap(&priv.runtime_map, &desc_size, &desc_ver);
+       if (status != EFI_SUCCESS) {
+               efi_err("Unable to retrieve UEFI memory map.\n");
+               return status;
+       }
+
+       efi_info("Exiting boot services\n");
+
+       efi_novamap = false;
+       status = efi_exit_boot_services(handle, &priv, exit_boot_func);
+       if (status != EFI_SUCCESS)
+               return status;
+
+       /* Install the new virtual address map */
+       efi_rt_call(set_virtual_address_map,
+                   priv.runtime_entry_count * desc_size, desc_size,
+                   desc_ver, priv.runtime_map);
 
        /* Config Direct Mapping */
        csr_write64(CSR_DMW0_INIT, LOONGARCH_CSR_DMWIN0);
        csr_write64(CSR_DMW1_INIT, LOONGARCH_CSR_DMWIN1);
 
        real_kernel_entry = (kernel_entry_t)
-               ((unsigned long)&kernel_entry - entrypoint + VMLINUX_LOAD_ADDRESS);
+               ((unsigned long)&kernel_entry - kernel_addr + VMLINUX_LOAD_ADDRESS);
 
-       if (!efi_novamap)
-               real_kernel_entry(true, fdt);
-       else
-               real_kernel_entry(false, fdt);
+       real_kernel_entry(true, (unsigned long)cmdline_ptr,
+                         (unsigned long)efi_system_table);
 }
--

2.35.1

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures 1add08ceb15b efi: libstub: remove pointless goto kludge d2130a9cf0f7 efi: libstub: simplify efi_get_memory_map() and struct efi_boot_memmap
bc5853aa56de efi: libstub: avoid efi_get_memory_map() for allocating the virt map
d80ca810f096 efi: libstub: drop pointless get_memory_map() call ead384d95634 (tag: efi-loongarch-for-v6.1) efi/loongarch: Add efistub booting support
568035b01cfb (tag: v6.0-rc1) Linux 6.0-rc1

Quoted 343 lines · Ard Biesheuvel

Quoted 339 lines

LoongArch does not use FDT or DT natively [yet], and the only reason it currently uses it is so that it can reuse the existing EFI stub code.

Overloading the DT with data passed between the EFI stub and the core kernel has been a source of problems: there is the overlap between information provided by EFI which DT can also provide (initrd base/size, command line, memory descriptions), requiring us to reason about which is which and what to prioritize. It has also resulted in ABI leaks, i.e., internal ABI being promoted to external ABI inadvertently because the bootloader can set the EFI stub's DT properties as well (e.g., "kaslr-seed"). This has become especially problematic with boot environments that want to pretend that EFI boot is being done (to access ACPI and SMBIOS tables, for instance) but have no ability to execute the EFI stub, and so the environment that the EFI stub creates is emulated [poorly, in some cases].

Another downside of treating DT like this is that the DT binary that the kernel receives is different from the one created by the firmware, which is undesirable in the context of secure and measured boot.

Given that LoongArch support in Linux is brand new, we can avoid these pitfalls, and treat the DT strictly as a hardware description, and use a separate handover method between the EFI stub and the kernel. Now that initrd loading and passing the EFI memory map have been refactored into pure EFI routines that use EFI configuration tables, the only thing we need to pass directly is the kernel command line (even if we could pass this via a config table as well, it is used extremely early, so passing it directly is preferred in this case.)

Signed-off-by Ard Biesheuvel <[email protected]>

---
 arch/loongarch/Kconfig                        |  3 --
 arch/loongarch/include/asm/bootinfo.h         |  2 +-
 arch/loongarch/kernel/efi.c                   | 30 ++++++++++-
 arch/loongarch/kernel/env.c                   | 13 ++---
 arch/loongarch/kernel/head.S                  |  2 +
 arch/loongarch/kernel/setup.c                 |  4 +-
 drivers/firmware/efi/libstub/Makefile         | 13 +++--
 drivers/firmware/efi/libstub/loongarch-stub.c | 56 +++++++++++++++++---
 8 files changed, 95 insertions(+), 28 deletions(-)

arch/loongarch/Kconfig +0 -3

index fca106a8b8af..14a2a1ec8561 100644
@@ -104,8 +104,6 @@ config LOONGARCH
        select MODULES_USE_ELF_RELA if MODULES
        select NEED_PER_CPU_EMBED_FIRST_CHUNK
        select NEED_PER_CPU_PAGE_FIRST_CHUNK
-       select OF
-       select OF_EARLY_FLATTREE
        select PCI
        select PCI_DOMAINS_GENERIC
        select PCI_ECAM if ACPI

@@ -311,7 +309,6 @@ config DMI
 config EFI
        bool "EFI runtime service support"
        select UCS2_STRING
-       select EFI_PARAMS_FROM_FDT
        select EFI_RUNTIME_WRAPPERS
        help
          This enables the kernel to use EFI runtime services that are

arch/loongarch/include/asm/bootinfo.h +1 -1

index e02ac4af7f6e..8e5881bc5ad1 100644
@@ -36,7 +36,7 @@ struct loongson_system_configuration {
 };
 
 extern u64 efi_system_table;
-extern unsigned long fw_arg0, fw_arg1;
+extern unsigned long fw_arg0, fw_arg1, fw_arg2;
 extern struct loongson_board_info b_info;
 extern struct loongson_system_configuration loongson_sysconf;
 

arch/loongarch/kernel/efi.c +29 -1

index 1f1f755fb425..a31329971133 100644
@@ -27,8 +27,13 @@
 static unsigned long efi_nr_tables;
 static unsigned long efi_config_table;
 
+static unsigned long __initdata boot_memmap = EFI_INVALID_TABLE_ADDR;
+
 static efi_system_table_t *efi_systab;
-static efi_config_table_type_t arch_tables[] __initdata = {{},};
+static efi_config_table_type_t arch_tables[] __initdata = {
+       {LINUX_EFI_BOOT_MEMMAP_GUID,    &boot_memmap,   "MEMMAP" },
+       {},
+};
 
 void __init efi_runtime_init(void)
 {

@@ -51,6 +56,7 @@ void __init efi_init(void)
 {
        int size;
        void *config_tables;
+       struct efi_boot_memmap *tbl;
 
        if (!efi_system_table)
                return;

@@ -61,6 +67,8 @@ void __init efi_init(void)
                return;
        }
 
+       efi_systab_report_header(&efi_systab->hdr, efi_systab->fw_vendor);
+
        set_bit(EFI_64BIT, &efi.flags);
        efi_nr_tables    = efi_systab->nr_tables;
        efi_config_table = (unsigned long)efi_systab->tables;

@@ -70,6 +78,26 @@ void __init efi_init(void)
        efi_config_parse_tables(config_tables, efi_systab->nr_tables, arch_tables);
        early_memunmap(config_tables, efi_nr_tables * size);
 
+       set_bit(EFI_CONFIG_TABLES, &efi.flags);
+
        if (screen_info.orig_video_isVGA == VIDEO_TYPE_EFI)
                memblock_reserve(screen_info.lfb_base, screen_info.lfb_size);
+
+       if (boot_memmap == EFI_INVALID_TABLE_ADDR)
+               return;
+
+       tbl = early_memremap_ro(boot_memmap, sizeof(*tbl));
+       if (tbl) {
+               struct efi_memory_map_data data;
+
+               data.phys_map           = boot_memmap + sizeof(*tbl);
+               data.size               = tbl->map_size;
+               data.desc_size          = tbl->desc_size;
+               data.desc_version       = tbl->desc_ver;
+
+               if (efi_memmap_init_early(&data) < 0)
+                       panic("Unable to map EFI memory map.\n");
+
+               early_memunmap(tbl, sizeof(*tbl));
+       }
 }

arch/loongarch/kernel/env.c +4 -9

index 82b478a5c665..6d56a463b091 100644
@@ -8,7 +8,6 @@
 #include <linux/efi.h>
 #include <linux/export.h>
 #include <linux/memblock.h>
-#include <linux/of_fdt.h>
 #include <asm/early_ioremap.h>
 #include <asm/bootinfo.h>
 #include <asm/loongson.h>

@@ -20,21 +19,17 @@ EXPORT_SYMBOL(loongson_sysconf);
 void __init init_environ(void)
 {
        int efi_boot = fw_arg0;
-       struct efi_memory_map_data data;
-       void *fdt_ptr = early_memremap_ro(fw_arg1, SZ_64K);
+       char *cmdline = early_memremap_ro(fw_arg1, COMMAND_LINE_SIZE);
 
        if (efi_boot)
                set_bit(EFI_BOOT, &efi.flags);
        else
                clear_bit(EFI_BOOT, &efi.flags);
 
-       early_init_dt_scan(fdt_ptr);
-       early_init_fdt_reserve_self();
-       efi_system_table = efi_get_fdt_params(&data);
+       strscpy(boot_command_line, cmdline, COMMAND_LINE_SIZE);
+       early_memunmap(cmdline, COMMAND_LINE_SIZE);
 
-       efi_memmap_init_early(&data);
-       memblock_reserve(data.phys_map & PAGE_MASK,
-                        PAGE_ALIGN(data.size + (data.phys_map & ~PAGE_MASK)));
+       efi_system_table = fw_arg2;
 }
 
 static int __init init_cpu_fullname(void)

arch/loongarch/kernel/head.S +2 -0

index 01bac62a6442..8f89f39fd31b 100644
@@ -67,6 +67,8 @@ SYM_CODE_START(kernel_entry)                  # kernel entry point
        st.d            a0, t0, 0               # firmware arguments
        la              t0, fw_arg1
        st.d            a1, t0, 0
+       la              t0, fw_arg2
+       st.d            a2, t0, 0
 
        /* KSave3 used for percpu base, initialized as 0 */
        csrwr           zero, PERCPU_BASE_KS

arch/loongarch/kernel/setup.c +2 -2

index e8714b1d94c8..7fabf2306e80 100644
@@ -51,7 +51,7 @@
 
 struct screen_info screen_info __section(".data");
 
-unsigned long fw_arg0, fw_arg1;
+unsigned long fw_arg0, fw_arg1, fw_arg2;
 DEFINE_PER_CPU(unsigned long, kernelsp);
 struct cpuinfo_loongarch cpu_data[NR_CPUS] __read_mostly;
 

@@ -187,7 +187,6 @@ early_param("mem", early_parse_mem);
 
 void __init platform_init(void)
 {
-       efi_init();
 #ifdef CONFIG_ACPI_TABLE_UPGRADE
        acpi_table_upgrade();
 #endif

@@ -347,6 +346,7 @@ void __init setup_arch(char **cmdline_p)
        *cmdline_p = boot_command_line;
 
        init_environ();
+       efi_init();
        memblock_init();
        parse_early_param();
 

drivers/firmware/efi/libstub/Makefile +8 -5

index ec2a7ba9364f..6234edf3d827 100644
@@ -29,7 +29,7 @@ cflags-$(CONFIG_RISCV)                := $(subst $(CC_FLAGS_FTRACE),,$(KBUILD_CFLAGS)) \
 cflags-$(CONFIG_LOONGARCH)     := $(subst $(CC_FLAGS_FTRACE),,$(KBUILD_CFLAGS)) \
                                   -fpie
 
-cflags-$(CONFIG_EFI_GENERIC_STUB) += -I$(srctree)/scripts/dtc/libfdt
+cflags-$(CONFIG_EFI_PARAMS_FROM_FDT)   += -I$(srctree)/scripts/dtc/libfdt
 
 KBUILD_CFLAGS                  := $(cflags-y) -Os -DDISABLE_BRANCH_PROFILING \
                                   -include $(srctree)/include/linux/hidden.h \

@@ -59,14 +59,17 @@ lib-y                               := efi-stub-helper.o gop.o secureboot.o tpm.o \
                                   skip_spaces.o lib-cmdline.o lib-ctype.o \
                                   alignedmem.o relocate.o vsprintf.o
 
-# include the stub's generic dependencies from lib/ when building for ARM/arm64
-efi-deps-y := fdt_rw.c fdt_ro.c fdt_wip.c fdt.c fdt_empty_tree.c fdt_sw.c
+# include the stub's libfdt dependencies from lib/ when needed
+libfdt-deps                    := fdt_rw.c fdt_ro.c fdt_wip.c fdt.c \
+                                  fdt_empty_tree.c fdt_sw.c
+
+lib-$(CONFIG_EFI_PARAMS_FROM_FDT) += fdt.o \
+                                    $(patsubst %.c,lib-%.o,$(libfdt-deps))
 
 $(obj)/lib-%.o: $(srctree)/lib/%.c FORCE
        $(call if_changed_rule,cc_o_c)
 
-lib-$(CONFIG_EFI_GENERIC_STUB) += efi-stub.o fdt.o string.o \
-                                  $(patsubst %.c,lib-%.o,$(efi-deps-y))
+lib-$(CONFIG_EFI_GENERIC_STUB) += efi-stub.o string.o
 
 lib-$(CONFIG_ARM)              += arm32-stub.o
 lib-$(CONFIG_ARM64)            += arm64-stub.o

drivers/firmware/efi/libstub/loongarch-stub.c +49 -8

index b7ef8d2df59e..32329f2a92f9 100644
@@ -9,7 +9,8 @@
 #include <asm/addrspace.h>
 #include "efistub.h"
 
-typedef void __noreturn (*kernel_entry_t)(bool efi, unsigned long fdt);
+typedef void __noreturn (*kernel_entry_t)(bool efi, unsigned long cmdline,
+                                         unsigned long systab);
 
 extern int kernel_asize;
 extern int kernel_fsize;

@@ -42,19 +43,60 @@ efi_status_t handle_kernel_image(unsigned long *image_addr,
        return status;
 }
 
-void __noreturn efi_enter_kernel(unsigned long entrypoint, unsigned long fdt, unsigned long fdt_size)
+struct exit_boot_struct {
+       efi_memory_desc_t       *runtime_map;
+       int                     runtime_entry_count;
+};
+
+static efi_status_t exit_boot_func(struct efi_boot_memmap *map, void *priv)
+{
+       struct exit_boot_struct *p = priv;
+
+       /*
+        * Update the memory map with virtual addresses. The function will also
+        * populate @runtime_map with copies of just the EFI_MEMORY_RUNTIME
+        * entries so that we can pass it straight to SetVirtualAddressMap()
+        */
+       efi_get_virtmap(map->map, map->map_size, map->desc_size,
+                       p->runtime_map, &p->runtime_entry_count);
+
+       return EFI_SUCCESS;
+}
+
+efi_status_t efi_boot_kernel(void *handle, efi_loaded_image_t *image,
+                            unsigned long kernel_addr, char *cmdline_ptr)
 {
        kernel_entry_t real_kernel_entry;
+       struct exit_boot_struct priv;
+       unsigned long desc_size;
+       efi_status_t status;
+       u32 desc_ver;
+
+       status = efi_alloc_virtmap(&priv.runtime_map, &desc_size, &desc_ver);
+       if (status != EFI_SUCCESS) {
+               efi_err("Unable to retrieve UEFI memory map.\n");
+               return status;
+       }
+
+       efi_info("Exiting boot services\n");
+
+       efi_novamap = false;
+       status = efi_exit_boot_services(handle, &priv, exit_boot_func);
+       if (status != EFI_SUCCESS)
+               return status;
+
+       /* Install the new virtual address map */
+       efi_rt_call(set_virtual_address_map,
+                   priv.runtime_entry_count * desc_size, desc_size,
+                   desc_ver, priv.runtime_map);
 
        /* Config Direct Mapping */
        csr_write64(CSR_DMW0_INIT, LOONGARCH_CSR_DMWIN0);
        csr_write64(CSR_DMW1_INIT, LOONGARCH_CSR_DMWIN1);
 
        real_kernel_entry = (kernel_entry_t)
-               ((unsigned long)&kernel_entry - entrypoint + VMLINUX_LOAD_ADDRESS);
+               ((unsigned long)&kernel_entry - kernel_addr + VMLINUX_LOAD_ADDRESS);
 
-       if (!efi_novamap)
-               real_kernel_entry(true, fdt);
-       else
-               real_kernel_entry(false, fdt);
+       real_kernel_entry(true, (unsigned long)cmdline_ptr,
+                         (unsigned long)efi_system_table);
 }
--

2.35.1

Quoted 28 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

Quoted 4 lines

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 9 lines

1add08ceb15b efi: libstub: remove pointless goto kludge d2130a9cf0f7 efi: libstub: simplify efi_get_memory_map() and struct efi_boot_memmap
bc5853aa56de efi: libstub: avoid efi_get_memory_map() for allocating the virt map
d80ca810f096 efi: libstub: drop pointless get_memory_map() call ead384d95634 (tag: efi-loongarch-for-v6.1) efi/loongarch: Add efistub booting support
568035b01cfb (tag: v6.0-rc1) Linux 6.0-rc1

Hmmm, I want to also merge zboot support in the loongarch-next's base, is that possible?

Quoted 30 lines · Huacai Chen

Quoted 28 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

Quoted 4 lines

Quoted 4 lines

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Quoted 10 lines

Quoted 10 lines

1add08ceb15b efi: libstub: remove pointless goto kludge d2130a9cf0f7 efi: libstub: simplify efi_get_memory_map() and struct efi_boot_memmap
bc5853aa56de efi: libstub: avoid efi_get_memory_map() for allocating the virt map
d80ca810f096 efi: libstub: drop pointless get_memory_map() call ead384d95634 (tag: efi-loongarch-for-v6.1) efi/loongarch: Add efistub booting support
568035b01cfb (tag: v6.0-rc1) Linux 6.0-rc1

Quoted 3 lines

Hmmm, I want to also merge zboot support in the loongarch-next's base, is that possible?

No, let's keep that separate. LoongArch boots fine without it, so we can wait until it arrives via the EFI tree, no?

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

Quoted 10 lines

Quoted 10 lines

Quoted 10 lines

1add08ceb15b efi: libstub: remove pointless goto kludge d2130a9cf0f7 efi: libstub: simplify efi_get_memory_map() and struct efi_boot_memmap
bc5853aa56de efi: libstub: avoid efi_get_memory_map() for allocating the virt map
d80ca810f096 efi: libstub: drop pointless get_memory_map() call ead384d95634 (tag: efi-loongarch-for-v6.1) efi/loongarch: Add efistub booting support
568035b01cfb (tag: v6.0-rc1) Linux 6.0-rc1

Quoted 6 lines

Quoted 3 lines

Hmmm, I want to also merge zboot support in the loongarch-next's base, is that possible?

No, let's keep that separate. LoongArch boots fine without it, so we can wait until it arrives via the EFI tree, no?

Both OK but I prefer to use a compressed kernel in daily work.

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 14 lines

Quoted 14 lines

Quoted 13 lines

Quoted 10 lines

1add08ceb15b efi: libstub: remove pointless goto kludge d2130a9cf0f7 efi: libstub: simplify efi_get_memory_map() and struct efi_boot_memmap
bc5853aa56de efi: libstub: avoid efi_get_memory_map() for allocating the virt map
d80ca810f096 efi: libstub: drop pointless get_memory_map() call ead384d95634 (tag: efi-loongarch-for-v6.1) efi/loongarch: Add efistub booting support
568035b01cfb (tag: v6.0-rc1) Linux 6.0-rc1

Hmmm, I want to also merge zboot support in the loongarch-next's base, is that possible?

Quoted 4 lines

Quoted 2 lines

No, let's keep that separate. LoongArch boots fine without it, so we can wait until it arrives via the EFI tree, no?

Both OK but I prefer to use a compressed kernel in daily work.

I understand that. But what you use for daily work and what you send to Linus are two different things, no?

Quoted 44 lines · Ard Biesheuvel

Quoted 42 lines · Huacai Chen

Quoted 40 lines · Ard Biesheuvel

Quoted 30 lines · Huacai Chen

Quoted 28 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

Quoted 4 lines

Quoted 4 lines

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Quoted 7 lines

Quoted 3 lines

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 50 lines · Huacai Chen

Quoted 48 lines · Ard Biesheuvel

Quoted 45 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 30 lines · Huacai Chen

Quoted 28 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

Quoted 8 lines

Quoted 4 lines

Quoted 4 lines

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

Quoted 3 lines

Quoted 3 lines

OK, I will investigate. This was working fine before so I'm not sure what happened there.

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

Hi, Ard,

Quoted 56 lines · Ard Biesheuvel

Quoted 53 lines · Huacai Chen

Quoted 51 lines · Ard Biesheuvel

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 2 lines

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Quoted 60 lines · Huacai Chen

Hi, Ard,

Quoted 56 lines · Ard Biesheuvel

Quoted 53 lines · Huacai Chen

Quoted 51 lines · Ard Biesheuvel

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 7 lines

Quoted 2 lines

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Those have been blocked in review for almost 2 years. I cc'ed you on the latest version. Without those patches, GRUB will generate a DT to pass the initrd information.

I understand this is annoying right now, but I strongly suggest to implement only a single method of loading the INITRD for EFI boot - it is much easier to reason about when things like secure boot and measured boot become relevant on your architecture.

In any case, you can build your UEFI shell with the following dynamic command from EDK2:

OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf

This add a 'initrd' command to the UEFI shell

(it will also work for bare metal platforms not only for virtual machines)

If this is a deal breaker for you, one workaround we might consider is to parse /chosen/linux,initrd-* in the LoongArch stub (but nothing else) if locating the LoadFile2 protocol fails and a DTB exists in the config table array. I'd prefer to avoid that if we can, but at least the initrd is always visible to the stub, for measurement etc.

Quoted 60 lines · Huacai Chen

Hi, Ard,

Quoted 56 lines · Ard Biesheuvel

Quoted 53 lines · Huacai Chen

Quoted 51 lines · Ard Biesheuvel

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 7 lines

Quoted 2 lines

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Those have been blocked in review for almost 2 years. I cc'ed you on the latest version. Without those patches, GRUB will generate a DT to pass the initrd information.

I understand this is annoying right now, but I strongly suggest to implement only a single method of loading the INITRD for EFI boot - it is much easier to reason about when things like secure boot and measured boot become relevant on your architecture.

In any case, you can build your UEFI shell with the following dynamic command from EDK2:

OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf

This add a 'initrd' command to the UEFI shell

(it will also work for bare metal platforms not only for virtual machines)

If this is a deal breaker for you, one workaround we might consider is to parse /chosen/linux,initrd-* in the LoongArch stub (but nothing else) if locating the LoadFile2 protocol fails and a DTB exists in the config table array. I'd prefer to avoid that if we can, but at least the initrd is always visible to the stub, for measurement etc.

Quoted 70 lines · Ard Biesheuvel

Quoted 67 lines · Huacai Chen

Hi, Ard,

Quoted 52 lines · Ard Biesheuvel

Quoted 50 lines · Huacai Chen

Quoted 48 lines · Ard Biesheuvel

Quoted 45 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 30 lines · Huacai Chen

Quoted 28 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

Quoted 8 lines

Quoted 4 lines

Quoted 4 lines

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

Quoted 6 lines

Quoted 3 lines

Quoted 3 lines

OK, I will investigate. This was working fine before so I'm not sure what happened there.

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Quoted 3 lines

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Maybe it is a GRUB problem (I'll discuss with GRUB team), but why without this series the GRUB can load initrd? Because in the old way UEFI BIOS fill initrd information to FDT?

Quoted 74 lines · Huacai Chen

Quoted 72 lines · Ard Biesheuvel

Quoted 60 lines · Huacai Chen

Hi, Ard,

Quoted 56 lines · Ard Biesheuvel

Quoted 53 lines · Huacai Chen

Quoted 51 lines · Ard Biesheuvel

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 7 lines

Quoted 2 lines

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

Quoted 5 lines

Quoted 1 line

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Maybe it is a GRUB problem (I'll discuss with GRUB team), but why without this series the GRUB can load initrd? Because in the old way UEFI BIOS fill initrd information to FDT?

No GRUB loads the initrd and creates a DT with a /chosen node to record the initrd base and size.

Quoted 82 lines · Ard Biesheuvel

Quoted 79 lines · Huacai Chen

Quoted 70 lines · Ard Biesheuvel

Quoted 60 lines · Huacai Chen

Hi, Ard,

Quoted 56 lines · Ard Biesheuvel

Quoted 53 lines · Huacai Chen

Quoted 51 lines · Ard Biesheuvel

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 3 lines · Huacai Chen

Acked-by: Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge window, but if needed to avoid massive conflicts, you could merge the tag into the loongarch tree and rebase your changes on top. But please take care not to rebase those patches themselves: the commit SHAs need to remain the same.

However, if the conflicts are only minor, you can just explain in your PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT 102faf0747c2 efi: libstub: install boot-time memory map as config table 7318926e7bc9 efi: libstub: remove DT dependency from generic stub 7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 7 lines

Quoted 2 lines

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Quoted 3 lines

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Maybe it is a GRUB problem (I'll discuss with GRUB team), but why without this series the GRUB can load initrd? Because in the old way UEFI BIOS fill initrd information to FDT?

Quoted 2 lines

No GRUB loads the initrd and creates a DT with a /chosen node to record the initrd base and size.

Without this series, efistub also fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), but initrd works well, this makes me puzzled.

Quoted 84 lines · Huacai Chen

Quoted 82 lines · Ard Biesheuvel

Quoted 79 lines · Huacai Chen

Quoted 70 lines · Ard Biesheuvel

Quoted 60 lines · Huacai Chen

Hi, Ard,

Quoted 56 lines · Ard Biesheuvel

Quoted 53 lines · Huacai Chen

Quoted 51 lines · Ard Biesheuvel

Quoted 39 lines · Huacai Chen

Quoted 37 lines · Ard Biesheuvel

Quoted 34 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

On Wed, 21 Sept 2022 at 06:02, Huacai Chen <[email protected]> wrote:

Acked-by: Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the
first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge
window, but if needed to avoid massive conflicts, you could merge the
tag into the loongarch tree and rebase your changes on top. But please
take care not to rebase those patches themselves: the commit SHAs need
to remain the same.

However, if the conflicts are only minor, you can just explain in your
PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it
should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT
102faf0747c2 efi: libstub: install boot-time memory map as config table
7318926e7bc9 efi: libstub: remove DT dependency from generic stub
7cb6671e3caa efi: libstub: unify initrd loading between architectures

Quoted 6 lines

Quoted 3 lines

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

OK, I will investigate. This was working fine before so I'm not sure what happened there.

Quoted 7 lines

Quoted 2 lines

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Quoted 3 lines

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Maybe it is a GRUB problem (I'll discuss with GRUB team), but why without this series the GRUB can load initrd? Because in the old way UEFI BIOS fill initrd information to FDT?

Quoted 6 lines

Quoted 2 lines

No GRUB loads the initrd and creates a DT with a /chosen node to record the initrd base and size.

Without this series, efistub also fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), but initrd works well, this makes me puzzled.

That is because you don't have DTB support in the core kernel yet right?

Quoted 88 lines · Ard Biesheuvel

Quoted 86 lines · Huacai Chen

Quoted 84 lines · Ard Biesheuvel

Quoted 72 lines · Huacai Chen

Quoted 70 lines · Ard Biesheuvel

Quoted 67 lines · Huacai Chen

Hi, Ard,

Quoted 52 lines · Ard Biesheuvel

Quoted 50 lines · Huacai Chen

Quoted 48 lines · Ard Biesheuvel

Quoted 45 lines · Huacai Chen

Quoted 32 lines · Ard Biesheuvel

Quoted 30 lines · Huacai Chen

On Wed, Sep 21, 2022 at 4:15 PM Ard Biesheuvel <[email protected]> wrote:

On Wed, 21 Sept 2022 at 06:02, Huacai Chen <[email protected]> wrote:

Acked-by: Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the
first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge
window, but if needed to avoid massive conflicts, you could merge the
tag into the loongarch tree and rebase your changes on top. But please
take care not to rebase those patches themselves: the commit SHAs need
to remain the same.

However, if the conflicts are only minor, you can just explain in your
PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it
should appear in -next tomorrow.

Quoted 8 lines

Quoted 4 lines

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT
102faf0747c2 efi: libstub: install boot-time memory map as config table
7318926e7bc9 efi: libstub: remove DT dependency from generic stub
7cb6671e3caa efi: libstub: unify initrd loading between architectures

I will have to respin the initrd patch, unfortunately, as it breaks x86. So the signed tag will be updated, and the SHAs above will change as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

Quoted 6 lines

Quoted 3 lines

Quoted 3 lines

OK, I will investigate. This was working fine before so I'm not sure what happened there.

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Quoted 7 lines

Quoted 3 lines

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Maybe it is a GRUB problem (I'll discuss with GRUB team), but why without this series the GRUB can load initrd? Because in the old way UEFI BIOS fill initrd information to FDT?

No GRUB loads the initrd and creates a DT with a /chosen node to record the initrd base and size.

Quoted 6 lines

Quoted 4 lines

Without this series, efistub also fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), but initrd works well, this makes me puzzled.

That is because you don't have DTB support in the core kernel yet right?

Never mind - what exact commit are you testing this with?

Quoted 90 lines · Ard Biesheuvel

Quoted 88 lines · Ard Biesheuvel

Quoted 86 lines · Huacai Chen

Quoted 84 lines · Ard Biesheuvel

Quoted 72 lines · Huacai Chen

Quoted 70 lines · Ard Biesheuvel

Quoted 67 lines · Huacai Chen

Hi, Ard,

Quoted 52 lines · Ard Biesheuvel

Quoted 50 lines · Huacai Chen

Quoted 48 lines · Ard Biesheuvel

Quoted 45 lines · Huacai Chen

Quoted 40 lines · Ard Biesheuvel

On Thu, 22 Sept 2022 at 04:15, Huacai Chen <[email protected]> wrote:

On Wed, Sep 21, 2022 at 4:15 PM Ard Biesheuvel <[email protected]> wrote:

On Wed, 21 Sept 2022 at 06:02, Huacai Chen <[email protected]> wrote:

Acked-by: Huacai Chen <[email protected]>

Thank you Huacai

I have created a separate tag with these patches applied onto the
first loongarch efi enablement patch (see below)

I would prefer to keep the loongarch trees separate for the next merge
window, but if needed to avoid massive conflicts, you could merge the
tag into the loongarch tree and rebase your changes on top. But please
take care not to rebase those patches themselves: the commit SHAs need
to remain the same.

However, if the conflicts are only minor, you can just explain in your
PR to Linus what the resolution should look like.

The tag can be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2

I have merged this into the EFI tree as well and pushed it out; it
should appear in -next tomorrow.

b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT
102faf0747c2 efi: libstub: install boot-time memory map as config table
7318926e7bc9 efi: libstub: remove DT dependency from generic stub
7cb6671e3caa efi: libstub: unify initrd loading between architectures

I will have to respin the initrd patch, unfortunately, as it breaks
x86. So the signed tag will be updated, and the SHAs above will change
as well.

Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address.

Quoted 6 lines

Quoted 3 lines

Quoted 3 lines

OK, I will investigate. This was working fine before so I'm not sure what happened there.

I cannot reproduce this. Can you reproduce it with qemu? If so, can you share your command line and .config etc?

In qemu we also OK, but on a real machine efistub fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), maybe our UEFI BIOS has some problems? On a real machine we use grub and there is no "initrd" command in UEFI shell.

Quoted 7 lines

Quoted 3 lines

Ah ok, so i guess your GRUB lacks the LoadFile2 patches?

https://lists.gnu.org/archive/html/grub-devel/2022-09/msg00057.html

Maybe it is a GRUB problem (I'll discuss with GRUB team), but why without this series the GRUB can load initrd? Because in the old way UEFI BIOS fill initrd information to FDT?

No GRUB loads the initrd and creates a DT with a /chosen node to record the initrd base and size.

Quoted 8 lines

Quoted 6 lines

Quoted 4 lines

Without this series, efistub also fails at efi_bs_call(locate_device_path, &lf2_proto_guid, &dp, &handle), but initrd works well, this makes me puzzled.

That is because you don't have DTB support in the core kernel yet right?

Never mind - what exact commit are you testing this with?

OK, so the existing code only does

early_init_dt_scan(fdt_ptr);
early_init_fdt_reserve_self();
efi_system_table = efi_get_fdt_params(&data);

so it never parses the /chosen node, and therefore does not discover the initrd.

I suppose you may have been running with the loongson-2k patches?

But actually, this describes exactly the problem that I am addressing with this series: you have been inadvertently relying on DT hardware support for initrd functionality ...