[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 3/3] riscv: Add opensbi firmware dynamic support
From: |
Bin Meng |
Subject: |
Re: [PATCH v3 3/3] riscv: Add opensbi firmware dynamic support |
Date: |
Fri, 26 Jun 2020 20:17:22 +0800 |
Hi Atish,
On Fri, Jun 26, 2020 at 8:33 AM Atish Patra <atish.patra@wdc.com> wrote:
>
> OpenSBI is the default firmware in Qemu and has various firmware loading
> options. Currently, qemu loader uses fw_jump which has a compile time
> pre-defined address where fdt & kernel image must reside. This puts a
> constraint on image size of the Linux kernel depending on the fdt location
> and available memory. However, fw_dynamic allows the loader to specify
> the next stage location (i.e. Linux kernel/U-boot) in memory and other
nits: U-Boot
> configurable boot options available in OpenSBI.
>
> Add support for OpenSBI dynamic firmware loading support. This doesn't
> break existing setup and fw_jump will continue to work as it is. Any
> other firmware will continue to work without any issues as long as it
> doesn't expect anything specific from loader in "a2" register.
>
> Signed-off-by: Atish Patra <atish.patra@wdc.com>
> Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
> ---
> hw/riscv/boot.c | 39 ++++++++++++++++++++--
> hw/riscv/sifive_u.c | 15 +++++++--
> hw/riscv/spike.c | 11 +++++--
> hw/riscv/virt.c | 11 +++++--
> include/hw/riscv/boot.h | 5 ++-
> include/hw/riscv/boot_opensbi.h | 58 +++++++++++++++++++++++++++++++++
> 6 files changed, 130 insertions(+), 9 deletions(-)
> create mode 100644 include/hw/riscv/boot_opensbi.h
>
> diff --git a/hw/riscv/boot.c b/hw/riscv/boot.c
> index 02c4018a8105..c96249c73c9f 100644
> --- a/hw/riscv/boot.c
> +++ b/hw/riscv/boot.c
> @@ -25,6 +25,7 @@
> #include "hw/boards.h"
> #include "hw/loader.h"
> #include "hw/riscv/boot.h"
> +#include "hw/riscv/boot_opensbi.h"
> #include "elf.h"
> #include "sysemu/device_tree.h"
> #include "sysemu/qtest.h"
> @@ -33,8 +34,10 @@
>
> #if defined(TARGET_RISCV32)
> # define KERNEL_BOOT_ADDRESS 0x80400000
> +#define fw_dynamic_info_data(__val) cpu_to_le32(__val)
> #else
> # define KERNEL_BOOT_ADDRESS 0x80200000
> +#define fw_dynamic_info_data(__val) cpu_to_le64(__val)
> #endif
>
> void riscv_find_and_load_firmware(MachineState *machine,
> @@ -189,14 +192,45 @@ hwaddr riscv_load_fdt(hwaddr dram_base, uint64_t
> mem_size, void *fdt)
> return fdt_addr;
> }
>
> +void riscv_rom_copy_firmware_info(hwaddr rom_base, hwaddr rom_size,
> + uint32_t reset_vec_size, uint64_t kernel_entry)
> +{
> + struct fw_dynamic_info dinfo;
> + uint64_t dinfo_len;
size_t?
> +
> + dinfo.magic = fw_dynamic_info_data(FW_DYNAMIC_INFO_MAGIC_VALUE);
> + dinfo.version = fw_dynamic_info_data(FW_DYNAMIC_INFO_VERSION);
nits: two spaces after =
> + dinfo.next_mode = fw_dynamic_info_data(FW_DYNAMIC_INFO_NEXT_MODE_S);
> + dinfo.next_addr = fw_dynamic_info_data(kernel_entry);
> + dinfo.options = 0;
> + dinfo.boot_hart = 0;
> + dinfo_len = sizeof(dinfo);
> +
> + /**
> + * copy the dynamic firmware info. This information is specific to
> + * OpenSBI but doesn't break any other firmware as long as they don't
> + * expect any certain value in "a2" register.
> + */
> + if (dinfo_len > (rom_size - reset_vec_size)) {
> + error_report("not enough space to store dynamic firmware info");
> + exit(1);
> + }
> +
> + rom_add_blob_fixed_as("mrom.finfo", &dinfo, dinfo_len,
> + rom_base + reset_vec_size,
> + &address_space_memory);
> +}
> +
> void riscv_setup_rom_reset_vec(hwaddr start_addr, hwaddr rom_base,
> - hwaddr rom_size,
> + hwaddr rom_size, uint64_t kernel_entry,
> hwaddr fdt_load_addr, void *fdt)
> {
> int i;
> +
This change should be in the 1st patch
> /* reset vector */
> uint32_t reset_vec[10] = {
> 0x00000297, /* 1: auipc t0, %pcrel_hi(fw_dyn) */
> + 0x02828613, /* addi a2, t0, %pcrel_lo(1b) */
> 0xf1402573, /* csrr a0, mhartid */
> #if defined(TARGET_RISCV32)
> 0x0202a583, /* lw a1, 32(t0) */
> @@ -206,7 +240,6 @@ void riscv_setup_rom_reset_vec(hwaddr start_addr, hwaddr
> rom_base,
> 0x0182b283, /* ld t0, 24(t0) */
> #endif
> 0x00028067, /* jr t0 */
> - 0x00000000,
> start_addr, /* start: .dword */
> 0x00000000,
> fdt_load_addr, /* fdt_laddr: .dword */
> @@ -220,6 +253,8 @@ void riscv_setup_rom_reset_vec(hwaddr start_addr, hwaddr
> rom_base,
> }
> rom_add_blob_fixed_as("mrom.reset", reset_vec, sizeof(reset_vec),
> rom_base, &address_space_memory);
> + riscv_rom_copy_firmware_info(rom_base, rom_size, sizeof(reset_vec),
> + kernel_entry);
>
> return;
> }
> diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c
> index 7d39a4e4ec6d..de25e9ceb0e7 100644
> --- a/hw/riscv/sifive_u.c
> +++ b/hw/riscv/sifive_u.c
> @@ -380,6 +380,7 @@ static void sifive_u_machine_init(MachineState *machine)
> target_ulong start_addr = memmap[SIFIVE_U_DRAM].base;
> int i;
> hwaddr fdt_load_addr;
> + uint64_t kernel_entry;
>
> /* Initialize SoC */
> object_initialize_child(OBJECT(machine), "soc", &s->soc,
> TYPE_RISCV_U_SOC);
> @@ -436,7 +437,7 @@ static void sifive_u_machine_init(MachineState *machine)
> riscv_find_and_load_firmware(machine, BIOS_FILENAME, start_addr, NULL);
>
> if (machine->kernel_filename) {
> - uint64_t kernel_entry = riscv_load_kernel(machine->kernel_filename,
> + kernel_entry = riscv_load_kernel(machine->kernel_filename,
> NULL);
nits: NULL not indented correctly. Can be moved to the same line above
>
> if (machine->initrd_filename) {
> @@ -449,6 +450,12 @@ static void sifive_u_machine_init(MachineState *machine)
> qemu_fdt_setprop_cell(s->fdt, "/chosen", "linux,initrd-end",
> end);
> }
> + } else {
> + /*
> + * If dynamic firmware is used, it doesn't know where is the next mode
> + * if kernel argument is not set.
> + */
> + kernel_entry = 0;
> }
>
> /* Compute the fdt load address in dram */
> @@ -459,6 +466,7 @@ static void sifive_u_machine_init(MachineState *machine)
> uint32_t reset_vec[11] = {
> s->msel, /* MSEL pin state */
> 0x00000297, /* 1: auipc t0, %pcrel_hi(fw_dyn) */
> + 0x02828613, /* addi a2, t0, %pcrel_lo(1b) */
nits: /* is not indented correctly
> 0xf1402573, /* csrr a0, mhartid */
> #if defined(TARGET_RISCV32)
> 0x0202a583, /* lw a1, 32(t0) */
> @@ -468,7 +476,6 @@ static void sifive_u_machine_init(MachineState *machine)
> 0x0182b283, /* ld t0, 24(t0) */
> #endif
> 0x00028067, /* jr t0 */
> - 0x00000000,
> start_addr, /* start: .dword */
> 0x00000000,
> fdt_load_addr, /* fdt_laddr: .dword */
> @@ -482,6 +489,10 @@ static void sifive_u_machine_init(MachineState *machine)
> }
> rom_add_blob_fixed_as("mrom.reset", reset_vec, sizeof(reset_vec),
> memmap[SIFIVE_U_MROM].base, &address_space_memory);
> +
> + riscv_rom_copy_firmware_info(memmap[SIFIVE_U_MROM].base,
> + memmap[SIFIVE_U_MROM].size,
> + sizeof(reset_vec), kernel_entry);
> }
>
> static bool sifive_u_machine_get_start_in_flash(Object *obj, Error **errp)
> diff --git a/hw/riscv/spike.c b/hw/riscv/spike.c
> index 69f050c07e5a..1ede1a28e6d4 100644
> --- a/hw/riscv/spike.c
> +++ b/hw/riscv/spike.c
> @@ -164,6 +164,7 @@ static void spike_board_init(MachineState *machine)
> MemoryRegion *mask_rom = g_new(MemoryRegion, 1);
> unsigned int smp_cpus = machine->smp.cpus;
> hwaddr fdt_load_addr;
> + uint64_t kernel_entry;
>
> /* Initialize SOC */
> object_initialize_child(OBJECT(machine), "soc", &s->soc,
> @@ -194,7 +195,7 @@ static void spike_board_init(MachineState *machine)
> htif_symbol_callback);
>
> if (machine->kernel_filename) {
> - uint64_t kernel_entry = riscv_load_kernel(machine->kernel_filename,
> + kernel_entry = riscv_load_kernel(machine->kernel_filename,
> htif_symbol_callback);
nits: not indented to (
>
> if (machine->initrd_filename) {
> @@ -207,6 +208,12 @@ static void spike_board_init(MachineState *machine)
> qemu_fdt_setprop_cell(s->fdt, "/chosen", "linux,initrd-end",
> end);
> }
> + } else {
> + /*
> + * If dynamic firmware is used, it doesn't know where is the next mode
> + * if kernel argument is not set.
> + */
> + kernel_entry = 0;
> }
>
> /* Compute the fdt load address in dram */
> @@ -214,7 +221,7 @@ static void spike_board_init(MachineState *machine)
> machine->ram_size, s->fdt);
> /* load the reset vector */
> riscv_setup_rom_reset_vec(memmap[SPIKE_DRAM].base,
> memmap[SPIKE_MROM].base,
> - memmap[SPIKE_MROM].size,
> + memmap[SPIKE_MROM].size, kernel_entry,
> fdt_load_addr, s->fdt);
>
> /* initialize HTIF using symbols found in load_kernel */
> diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
> index 639e284fc2e3..25ac984e99f0 100644
> --- a/hw/riscv/virt.c
> +++ b/hw/riscv/virt.c
> @@ -479,6 +479,7 @@ static void virt_machine_init(MachineState *machine)
> size_t plic_hart_config_len;
> target_ulong start_addr = memmap[VIRT_DRAM].base;
> hwaddr fdt_load_addr;
> + uint64_t kernel_entry;
> int i;
> unsigned int smp_cpus = machine->smp.cpus;
>
> @@ -510,7 +511,7 @@ static void virt_machine_init(MachineState *machine)
> memmap[VIRT_DRAM].base, NULL);
>
> if (machine->kernel_filename) {
> - uint64_t kernel_entry = riscv_load_kernel(machine->kernel_filename,
> + kernel_entry = riscv_load_kernel(machine->kernel_filename,
> NULL);
nits: not indented correctly. Can be moved to above line
>
> if (machine->initrd_filename) {
> @@ -523,6 +524,12 @@ static void virt_machine_init(MachineState *machine)
> qemu_fdt_setprop_cell(s->fdt, "/chosen", "linux,initrd-end",
> end);
> }
> + } else {
> + /*
> + * If dynamic firmware is used, it doesn't know where is the next mode
> + * if kernel argument is not set.
> + */
> + kernel_entry = 0;
> }
>
> if (drive_get(IF_PFLASH, 0, 0)) {
> @@ -538,7 +545,7 @@ static void virt_machine_init(MachineState *machine)
> machine->ram_size, s->fdt);
> /* load the reset vector */
> riscv_setup_rom_reset_vec(start_addr, virt_memmap[VIRT_MROM].base,
> - virt_memmap[VIRT_MROM].size,
> + virt_memmap[VIRT_MROM].size, kernel_entry,
> fdt_load_addr, s->fdt);
>
> /* create PLIC hart topology configuration string */
> diff --git a/include/hw/riscv/boot.h b/include/hw/riscv/boot.h
> index f64fcadd2390..0ccfd2285888 100644
> --- a/include/hw/riscv/boot.h
> +++ b/include/hw/riscv/boot.h
> @@ -37,8 +37,11 @@ hwaddr riscv_load_initrd(const char *filename, uint64_t
> mem_size,
> uint64_t kernel_entry, hwaddr *start);
> hwaddr riscv_load_fdt(hwaddr dram_start, uint64_t dram_size,
> void *fdt);
> +void riscv_rom_copy_firmware_info(hwaddr rom_base, hwaddr rom_size,
> + uint32_t reset_vec_size,
> + uint64_t kernel_entry);
> void riscv_setup_rom_reset_vec(hwaddr saddr, hwaddr rom_base,
> - hwaddr rom_size,
> + hwaddr rom_size, uint64_t kernel_entry,
> hwaddr fdt_load_addr, void *fdt);
>
> #endif /* RISCV_BOOT_H */
> diff --git a/include/hw/riscv/boot_opensbi.h b/include/hw/riscv/boot_opensbi.h
> new file mode 100644
> index 000000000000..0d5ddd6c3daf
> --- /dev/null
> +++ b/include/hw/riscv/boot_opensbi.h
> @@ -0,0 +1,58 @@
> +/* SPDX-License-Identifier: BSD-2-Clause */
> +/*
> + * Copyright (c) 2019 Western Digital Corporation or its affiliates.
> + *
> + * Based on include/sbi/{fw_dynamic.h,sbi_scratch.h} from the OpenSBI
> project.
> + */
> +#ifndef OPENSBI_H
> +#define OPENSBI_H
> +
> +/** Expected value of info magic ('OSBI' ascii string in hex) */
> +#define FW_DYNAMIC_INFO_MAGIC_VALUE 0x4942534f
> +
> +/** Maximum supported info version */
> +#define FW_DYNAMIC_INFO_VERSION 0x2
> +
> +/** Possible next mode values */
> +#define FW_DYNAMIC_INFO_NEXT_MODE_U 0x0
> +#define FW_DYNAMIC_INFO_NEXT_MODE_S 0x1
> +#define FW_DYNAMIC_INFO_NEXT_MODE_M 0x3
> +
> +enum sbi_scratch_options {
> + /** Disable prints during boot */
> + SBI_SCRATCH_NO_BOOT_PRINTS = (1 << 0),
> + /** Enable runtime debug prints */
> + SBI_SCRATCH_DEBUG_PRINTS = (1 << 1),
> +};
> +
> +/** Representation dynamic info passed by previous booting stage */
> +struct fw_dynamic_info {
> + /** Info magic */
> + target_long magic;
> + /** Info version */
> + target_long version;
> + /** Next booting stage address */
> + target_long next_addr;
> + /** Next booting stage mode */
> + target_long next_mode;
> + /** Options for OpenSBI library */
> + target_long options;
> + /**
> + * Preferred boot HART id
> + *
> + * It is possible that the previous booting stage uses same link
> + * address as the FW_DYNAMIC firmware. In this case, the relocation
> + * lottery mechanism can potentially overwrite the previous booting
> + * stage while other HARTs are still running in the previous booting
> + * stage leading to boot-time crash. To avoid this boot-time crash,
> + * the previous booting stage can specify last HART that will jump
> + * to the FW_DYNAMIC firmware as the preferred boot HART.
> + *
> + * To avoid specifying a preferred boot HART, the previous booting
> + * stage can set it to -1UL which will force the FW_DYNAMIC firmware
> + * to use the relocation lottery mechanism.
> + */
> + target_long boot_hart;
> +};
> +
> +#endif
> --
Regards,
Bin
- [PATCH v3 0/3] Add OpenSBI dynamic firmware support, Atish Patra, 2020/06/25
- [PATCH v3 1/3] riscv: Unify Qemu's reset vector code path, Atish Patra, 2020/06/25
- [PATCH v3 3/3] riscv: Add opensbi firmware dynamic support, Atish Patra, 2020/06/25
- Re: [PATCH v3 3/3] riscv: Add opensbi firmware dynamic support,
Bin Meng <=
- [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Atish Patra, 2020/06/25
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Bin Meng, 2020/06/26
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Atish Patra, 2020/06/26
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Bin Meng, 2020/06/26
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Atish Patra, 2020/06/27
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Alistair Francis, 2020/06/27
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Bin Meng, 2020/06/27
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Bin Meng, 2020/06/27
- Re: [PATCH v3 2/3] RISC-V: Copy the fdt in dram instead of ROM, Atish Patra, 2020/06/29