Re: [PATCH] roms: opensbi: Upgrade from v0.6 to v0.7

[Alistair Francis]

On Mon, Apr 20, 2020 at 6:34 PM Bin Meng <address@hidden> wrote:
>
> Hi Alistair,
>
> On Tue, Apr 21, 2020 at 2:41 AM Alistair Francis <address@hidden> wrote:
> >
> > On Mon, Apr 20, 2020 at 6:25 AM Bin Meng <address@hidden> wrote:
> > >
> > > Upgrade OpenSBI from v0.6 to v0.7 and the pre-built bios images.
> > >
> > > The v0.7 release includes the following commits:
> > >
> > > f64f4b9 lib: Add a new platform feature to bringup secondary harts
> > > b677a9b lib: Implement hart hotplug
> > > 5b48240 lib: Add possible hart status values
> > > e3f69fc lib: Implement Hart State Management (HSM) SBI extension
> > > 6704216 lib: Check MSIP bit after returning from WFI
> > > 82ae8e8 makefile: Do setup of the install target more flexible
> > > e1a5b73 platform: sifive: fu540: allow sv32 as an mmu-type
> > > 8c83fb2 lib: Fix return type of sbi_hsm_hart_started()
> > > 00d332b include: Move bits related defines and macros to sbi_bitops.h
> > > a148996 include: sbi_bitops: More useful bit operations
> > > 4a603eb platform: kendryte/k210: Set per-HART stack size to 8KB
> > > 678c3c3 include: sbi_scratch: Set per-HART scratch size to 4KB
> > > 2abc55b lib: Sort build objects in alphabetical order
> > > 6e87507 platform: ae350: Sort build objects in alphabetical order
> > > 650c0e5 lib: sbi: Fix coding style issues
> > > 078686d lib: serial: Fix coding style issues
> > > 3226bd9 lib: Simple bitmap library
> > > c741abc include: Simple hartmask library
> > > d6d7e18 lib: sbi_init: Don't allow HARTID greater than 
> > > SBI_HARTMASK_MAX_BITS
> > > a4a6a81 lib: Introduce SBI_TLB_INFO_INIT() helper macro
> > > d963164 lib: sbi_tlb: Use sbi_hartmask in sbi_tlb_info
> > > 71d2b83 lib: Move all coldboot wait APIs to sbi_init.c
> > > 2b945fc lib: sbi_init: Use hartmask for coldboot wait
> > > 44ce5b9 include: Remove disabled_hart_mask from sbi_platform
> > > 2db381f lib: Introduce sbi_hsm_hart_started_mask() API
> > > 61f7768 lib: sbi_ecall_legacy: Use sbi_hsm_hart_started_mask() API
> > > 466fecb lib: sbi_system: Use sbi_hsm_hart_started_mask() API
> > > 9aad831 lib: sbi_ipi: Use sbi_hsm_hart_started_mask() API
> > > eede1aa lib: sbi_hart: Remove HART available mask and related APIs
> > > 757bb44 docs: Remove out-of-date documentation
> > > 86d37bb lib: sbi: Fix misaligned trap handling
> > > ffdc858 platform: ariane-fpga: Change license for ariane-fpga from 
> > > GPL-2.0 to BSD-2
> > > 4b2f594 sbi: Add definitions for true/false
> > > 0cfe49a libfdt: Add INT32_MAX and UINT32_MAX in libfdt_env.h
> > > baac7e0 libfdt: Upgrade to v1.5.1 release
> > > f92147c include: Make sbi_hart_id_to_scratch() as macro
> > > eeae3d9 firmware: fw_base: Optimize _hartid_to_scratch() implementation
> > > 16e7071 lib: sbi_hsm: Optimize sbi_hsm_hart_get_state() implementation
> > > 823345e include: Make sbi_current_hartid() as macro in riscv_asm.h
> > > 9aabba2 Makefile: Fix distclean make target
> > > 9275ed3 platform: ariane-fpga: Set per-HART stack size to 8KB
> > > 2343efd platform: Set per-HART stack size to 8KB in the template platform 
> > > codes
> > > 72a0628 platform: Use one unified per-HART stack size macro for all 
> > > platforms
> > > 327ba36 scripts: Cover sifive/fu540 in the 32-bit build
> > > 5fbcd62 lib: sbi: Update pmp_get() to return decoded size directly
> > > dce8846 libfdt: Compile fdt_addresses.c
> > > fcb1ded lib: utils: Add a fdt_reserved_memory_fixup() helper
> > > 666be6d platform: Clean up include header files
> > > 6af5576 lib: utils: Move PLIC DT fix up codes to fdt_helper.c
> > > e846ce1 platform: andes/ae350: Fix up DT for reserved memory
> > > 8135520 platform: ariane-fpga: Fix up DT for reserved memory
> > > c9a5268 platform: qemu/virt: Fix up DT for reserved memory
> > > 6f9bb83 platform: sifive/fu540: Fix up DT for reserved memory
> > > 1071f05 platform: sifive/fu540: Remove "stdout-path" fix-up
> > > dd9439f lib: utils: Add a fdt_cpu_fixup() helper
> > > 3f1c847 platform: sifive/fu540: Replace cpu0 node fix-up with the new 
> > > helper
> > > db6a2b5 lib: utils: Add a general device tree fix-up helper
> > > 3f8d754 platform: Update to call general DT fix-up helper
> > > 87a7ef7 lib: sbi_scratch: Introduce HART id to scratch table
> > > e23d3ba include: Simplify HART id to scratch macro
> > > 19bd531 lib: sbi_hsm: Simplify hart_get_state() and hart_started() APIs
> > > 3ebfe0e lib: sbi_tlb: Simplify sbi_tlb_entry_process() function
> > > 209134d lib: Handle failure of sbi_hartid_to_scratch() API
> > > bd6ef02 include: sbi_platform: Improve sbi_platform_hart_disabled() API
> > > c9f60fc lib: sbi_scratch: Don't set hartid_to_scratch table for disabled 
> > > HART
> > > 680b098 lib: sbi_hsm: Don't use sbi_platform_hart_count() API
> > > db187d6 lib: sbi_hsm: Remove scratch parameter from hart_started_mask() 
> > > API
> > > 814f38d lib: sbi_hsm: Don't use sbi_platform_hart_disabled() API
> > > 75eec9d lib: Don't use sbi_platform_hart_count() API
> > > c51f02c include: sbi_platform: Introduce HART index to HART id table
> > > 315a877 platform: sifive/fu540: Remove FU540_ENABLED_HART_MASK option
> > > a0c88dd lib: Fix sbi_ecall_register_extension to prevent extension IDs 
> > > overlap
> > > 9a74a64 lib: Check MSIP bit after returning from WFI
> > > 5968894 platform: Move ariane standalone fpga project to its own project
> > > ed265b4 platform: fpga/ariane: Remove redundant plic address macros
> > > fb84879 platform: Add OpenPiton platform support
> > > d1d6560 platform: fpga/common: Add a fdt parsing helper functions
> > > 040e4e2 lib: utils: Move fdt fixup helper routines to a different file
> > > 4c37451 platform: openpiton: Read the device configurations from device 
> > > tree
> > > 4d93586 lib: prevent coldboot_lottery from overflowing
> > > 550ba88 scripts: Extend create-binary-archive.sh for unified binary tar 
> > > ball
> > > 160c885 lib: utils: Improve fdt_cpu_fixup() implementation
> > > 1de66d1 lib: Optimize unpriv load/store implementation
> > > 626467c lib: Remove scratch parameter from unpriv load/store functions
> > > cb78a48 lib: sbi_trap: Remove scratch parameter from sbi_trap_redirect()
> > > d11c79c lib: sbi_emulate_csr: Remove scratch and hartid parameter
> > > 5a7bd0c lib: sbi_illegal_insn: Remove mcause, scratch and hartid 
> > > parameters
> > > fe37d7d lib: sbi_misaligned_ldst: Remove mcause, scratch and hartid 
> > > parameters
> > > 7487116 lib: sbi_ecall: Remove mcause, scratch and hartid parameters
> > > 40b221b lib: sbi_trap: Simplify sbi_trap_handler() API
> > > 7b211ff include: sbi_platform: Remove priv parameter from hart_start() 
> > > callback
> > > 5b6957e include: Use more consistent name for atomic xchg() and cmpxchg()
> > > dd0f21c lib: sbi_scratch: Introduce sbi_scratch_last_hartid() API
> > > 54b2779 include: sbi_tlb: Remove scratch parameter from sbi_tlb_request()
> > > 9e52a45 include: sbi_ipi: Remove scratch parameter from most functions
> > > ec0d80f include: sbi_system: Remove scratch parameter and redundant 
> > > functions
> > > 0a28ea5 include: sbi_timer: Remove scratch parameter from most funcitons
> > > 648507a include: sbi_console: Remove scratch parameter from sbi_dprintf()
> > > e5a7f55 platform: thead/c910: Use HSM extension to boot secondary cores
> > > f281de8 lib: irqchip/plic: Fix maximum priority threshold value
> > > 6c7922e lib: Support vector extension
> > > 615587c docs: Update README about supported SBI versions
> > > 66d0184 lib: Allow overriding SBI implementation ID
> > > 9f1b72c include: Bump-up version to 0.7
> > >
> > > Signed-off-by: Bin Meng <address@hidden>
> >
> > Reviewed-by: Alistair Francis <address@hidden>
> >
> > Can you share a git branch with this patch? From memory these binary
> > patches don't apply well from emails.
>
> Sure. Please grab the bits from http://github.com/lbmeng/qemu opensbi branch.

Thanks

>
> You can also use patchwork to help with your custodian work.
> http://patchwork.ozlabs.org/project/qemu-devel/patch/address@hidden/
>
> Click on the "mbox" button to download the patch and apply it.

I usually use the patches tool to apply patches, I just remember
(maybe incorrectly) there being issues in the past applying large
binary patches from the mailing list.

Alistair

> Regards,
> Bin