On Thu, May 25, 2023 at 06:28:18PM +0300, Oleksii Kurochko wrote:
> After introduction of initial pagetables there is no any sense
> in dummy_bss variable as bss section will not be empty anymore.
>
> Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>
> ---
> Changes in V9:
> - Nothing changed. Only rebase
> ---
> Changes in V8:
> - Nothing changed. Only rebase
> ---
> Changes in V7:
> - Nothing changed. Only rebase
> ---
> Changes in V6:
> - Nothing changed. Only rebase
> ---
> Changes in V5:
> - Nothing changed. Only rebase
> ---
> Changes in V4:
> - Nothing changed. Only rebase
> ---
> Changes in V3:
> * patch was introduced in the current one patch series (v3).
> ---
> Changes in V2:
> * patch was introduced in the current one patch series (v2).
> ---
> xen/arch/riscv/setup.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/xen/arch/riscv/setup.c b/xen/arch/riscv/setup.c
> index cf5dc5824e..845d18d86f 100644
> --- a/xen/arch/riscv/setup.c
> +++ b/xen/arch/riscv/setup.c
> @@ -8,14 +8,6 @@
> unsigned char __initdata cpu0_boot_stack[STACK_SIZE]
> __aligned(STACK_SIZE);
>
> -/*
> - * To be sure that .bss isn't zero. It will simplify code of
> - * .bss initialization.
> - * TODO:
> - * To be deleted when the first real .bss user appears
> - */
> -int dummy_bss __attribute__((unused));
> -
> void __init noreturn start_xen(unsigned long bootcpu_id,
> paddr_t dtb_addr)
> {
> --
> 2.40.1
>
>
Acked-by: Bobby Eshleman <bobbyeshleman@gmail.com>