On 06.05.2025 18:51, Oleksii Kurochko wrote:
> --- a/xen/arch/riscv/setup.c
> +++ b/xen/arch/riscv/setup.c
> @@ -4,12 +4,16 @@
> #include <xen/bug.h>
> #include <xen/bootfdt.h>
> #include <xen/compile.h>
> +#include <xen/console.h>
> #include <xen/device_tree.h>
> #include <xen/init.h>
> #include <xen/irq.h>
> #include <xen/mm.h>
> +#include <xen/percpu.h>
Why's this needed? I can't spot anything ...
> +#include <xen/serial.h>
> #include <xen/shutdown.h>
> #include <xen/smp.h>
> +#include <xen/timer.h>
> #include <xen/vmap.h>
> #include <xen/xvmalloc.h>
>
> @@ -136,8 +140,17 @@ void __init noreturn start_xen(unsigned long bootcpu_id,
>
> intc_preinit();
>
> + uart_init();
> + console_init_preirq();
> +
> intc_init();
>
> + timer_init();
> +
> + local_irq_enable();
> +
> + console_init_postirq();
> +
> printk("All set up\n");
>
> machine_halt();
... relevant here. With the need clarified or with the #include dropped:
Acked-by: Jan Beulich <[email protected]>
Jan
