[PATCH v3 05/10] firmware: fw_base.S: Move _link_start to .rodata section

Anup Patel anup at brainfault.org
Mon Feb 5 23:26:22 PST 2024


On Wed, Jan 17, 2024 at 4:12 PM Xiang W <wxjstz at 126.com> wrote:
>
> _link_start is readonly, so move it to rodata section.
>
> Signed-off-by: Xiang W <wxjstz at 126.com>

Looks good to me.

Reviewed-by: Anup Patel <anup at brainfault.org>

Regards,
Anup

> ---
>  firmware/fw_base.S | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/firmware/fw_base.S b/firmware/fw_base.S
> index 18dd0c5..7c7157d 100644
> --- a/firmware/fw_base.S
> +++ b/firmware/fw_base.S
> @@ -482,6 +482,9 @@ _relocate_lottery:
>         RISCV_PTR       0
>  _boot_status:
>         RISCV_PTR       0
> +
> +       .section .rodata
> +       .align 3
>  _link_start:
>         RISCV_PTR       FW_TEXT_START
>
> --
> 2.43.0
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi



More information about the opensbi mailing list