opensbi.git/firmware, branch v1.4

firmware: fix section types

2023-12-09T08:56:58Z

These sections are only intended to hold data, and should not be executable. Signed-off-by: Matt Waltz Reviewed-by: Xiang W Reviewed-by: Anup Patel

firmware: fw_base.S: Fix boot hart status synchronization

2023-11-16T10:49:42Z

It's wrong to put the fence after setting the boot status flag because all relocation operations must be finished before setting the status flag. So, this fence must be put before the setting status flag, and there is no use in putting a fence between _start_warm and setting status flag. Also, nop can't delay other harts too much, so use div instead, just like Linux cpu_relax. Current opensbi force enables “M” Standard Extension, and mul instructions have been used in the fw_base.S. After the above two fixes, the boot hart index param of the fw_dynamic_info could be guaranteed properly for all platforms. Signed-off-by: Guo Ren Signed-off-by: Guo Ren Reviewed-by: Anup Patel

firmware: Remove handling of R_RISCV_{32,64}

2023-10-06T03:51:23Z

Since everything is statically linked, we won't actually have R_RISCV_{32,64} relocations. No need to handle these. Fixes: 0f20e8adcf42 ("firmware: Support position independent execution") Signed-off-by: Vivian Wang Reviewed-by: Anup Patel

firmware: Remove ALIGN in .rela.dyn in linker script

2023-10-06T03:51:21Z

The .rela.dyn section should be exactly the size of the relocations, without padding. On RV64, .rela* sections are already aligned and there's no need for padding. On RV32, this adds padding up to 4 bytes, which, if present, confuses the relocation loop into processing an extra entry past the end of .rela*, and it crashes with an invalid memory access. Fixes: 0f20e8adcf42 ("firmware: Support position independent execution") Signed-off-by: Vivian Wang Reviewed-by: Anup Patel

firmware: payload: test: Change to SBI v2.0 DBCN ecalls

2023-09-06T11:20:50Z

As the the "Console Putchar" extension is already legacy and may be removed in the furture. So replace it with the SBI v2.0 "DBCN" extension. Signed-off-by: Inochi Amaoto Reviewed-by: Anup Patel

fw_base.S: Fix assembler error with clang 16+

2023-08-06T05:36:02Z

Attempting to build OpenSBI with clang 16 and the following command: $ make LLVM=1 PLATFORM=generic Results in the following error: AS platform/generic/firmware/fw_dynamic.o /tmp/fw_dynamic-d000a6.s:429:9: error: symbol '_fw_start' can not be undefined in a subtraction expression .dword _fw_rw_start - _fw_start Work around this issue by eliminating the __fw_rw_offset variable and performing the offset calculation at run-time instead. This takes advantage of the fact that the a4 register contains the value of _fw_start. Signed-off-by: Mitchell Horne Reviewed-by: Anup Patel

firmware: Fix find hart index

2023-06-21T04:50:51Z

After the loop to find the hartid is launched, assigning -1 to index will fail in the subsequent compare instruction bge. Fix This. Signed-off-by: Xiang W Reviewed-by: Anup Patel

platform: Allow platforms to specify heap size

2023-06-05T10:15:33Z

We extend struct sbi_platform and struct sbi_scratch to allow platforms specify the heap size to the OpenSBI firmwares. The OpenSBI firmwares will use this information to determine the location of heap and provide heap base address in per-HART scratch space. Signed-off-by: Anup Patel Reviewed-by: Andrew Jones

firmware: Change to use positive offset to access relocation entries

2023-04-17T03:25:55Z

The codes currently skip the very first relocation entry, but later reference the elements in the relocation entry using minus offsets. Change to use positive offsets so that there is no need to skip the first relocation entry. Signed-off-by: Bin Meng Reviewed-by: Anup Patel Reviewed-by: Xiang W

firmware: Optimize loading relocation type

2023-04-17T03:25:49Z

't5' already contains relocation type so don't bother reloading it. Signed-off-by: Bin Meng Reviewed-by: Anup Patel Reviewed-by: Xiang W