[PATCH v8 3/5] xen/riscv: align __bss_start

Oleksii Kurochko posted 5 patches 1 year, 6 months ago
There is a newer version of this series
[PATCH v8 3/5] xen/riscv: align __bss_start
Posted by Oleksii Kurochko 1 year, 6 months ago
bss clear cycle requires proper alignment of __bss_start.

ALIGN(PAGE_SIZE) before "*(.bss.page_aligned)" in xen.lds.S
was removed as any contribution to "*(.bss.page_aligned)" have to
specify proper aligntment themselves.

Fixes: cfa0409f7cbb ("xen/riscv: initialize .bss section")
Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
---
Changes in V8:
 * Remove ". = ALIGN(PAGE_SIZE);" before "*(.bss.page_aligned)" in
   vmlinux.lds.S file as any contribution to .bss.page_aligned have to specify
   proper alignment themselves.
 * Add "Fixes: cfa0409f7cbb ("xen/riscv: initialize .bss section")" to
   the commit message
 * Add "Reviewed-by: Jan Beulich <jbeulich@suse.com>" to the commit message
---
Changes in V7:
 * the patch was introduced in the current patch series.
---
 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 fe475d096d..df71d31e17 100644
--- a/xen/arch/riscv/xen.lds.S
+++ b/xen/arch/riscv/xen.lds.S
@@ -137,9 +137,9 @@ SECTIONS
     __init_end = .;
 
     .bss : {                     /* BSS */
+        . = ALIGN(POINTER_ALIGN);
         __bss_start = .;
         *(.bss.stack_aligned)
-        . = ALIGN(PAGE_SIZE);
         *(.bss.page_aligned)
         . = ALIGN(PAGE_SIZE);
         __per_cpu_start = .;
-- 
2.40.1