On 06.06.2023 21:55, Oleksii Kurochko wrote: > Sometimes variables are located in .sbss section but it won't > be mapped after MMU will be enabled. > To avoid MMU failures .sbss should be mapped > > Signed-off-by: Oleksii Kurochko <oleksii.kuroc...@gmail.com> > --- > xen/arch/riscv/xen.lds.S | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/arch/riscv/xen.lds.S b/xen/arch/riscv/xen.lds.S > index 74afbaab9b..9a2799bab5 100644 > --- a/xen/arch/riscv/xen.lds.S > +++ b/xen/arch/riscv/xen.lds.S > @@ -151,7 +151,7 @@ SECTIONS > *(.bss.percpu.read_mostly) > . = ALIGN(SMP_CACHE_BYTES); > __per_cpu_data_end = .; > - *(.bss .bss.*) > + *(.bss .bss.* .sbss) > . = ALIGN(POINTER_ALIGN); > __bss_end = .; > } :text
Two remarks, despite Alistair's ack: Wouldn't it be better to add .sbss.* right away as well? And strictly speaking wouldn't it be more logical to have .sbss ahead of .bss? Jan
