Re: [PATCH v2 03/26] xen/riscv: Implement construct_domain()

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

On 5/18/26 5:33 PM, Jan Beulich wrote:
> On 08.05.2026 16:43, Oleksii Kurochko wrote:
> > Implement construct_domain() function for RISC-V, which performs initial setup
> > for the domain's first vCPU, loads the kernel, initrd, and device tree,
> > and sets up guest CPU registers for boot.
> >
> > It also creates additional vCPUs up to max_vcpus and assigns the device tree
> > address and boot cpuid in registers.
> >
> > Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
> with ...
>
> > --- /dev/null
> > +++ b/xen/arch/riscv/domain-build.c
> > @@ -0,0 +1,52 @@
> > +/* SPDX-License-Identifier: GPL-2.0-or-later */
> > +
> > +#include <xen/fdt-domain-build.h>
> > +#include <xen/fdt-kernel.h>
> > +#include <xen/init.h>
> > +#include <xen/sched.h>
> > +
> > +#include <asm/current.h>
> > +#include <asm/guest_access.h>
> > +
> > +int __init construct_domain(struct domain *d, struct kernel_info *kinfo)
> > +{
> > +    struct vcpu *v = d->vcpu[0];
> > +    struct cpu_user_regs *regs = vcpu_guest_cpu_user_regs(v);
> > +
> > +    BUG_ON(v->is_initialised);
> > +
> > +    /*
> > +     * At the moment *_load() don't return value and will just panic()
> > +     * inside.
> > +     * TODO: it will be good to change that.
> > +     */
> > +    kernel_load(kinfo);
> > +    initrd_load(kinfo, copy_to_guest_phys);
> > +    dtb_load(kinfo, copy_to_guest_phys);
> > +
> > +    regs->sepc = kinfo->entry;
> > +
> > +    /* Guest boot cpuid = 0 */
> > +    regs->a0 = 0;
> > +    regs->a1 = kinfo->dtb_paddr;
> > +
> > +    for ( unsigned int i = 1; i < d->max_vcpus; i++ )
> > +    {
> > +        const struct vcpu *tmp_v = vcpu_create(d, i);
> > +
> > +        if ( !tmp_v )
> > +        {
> > +            printk("Failed to allocate %pd v%d\n", d, i);
> ... %u used here and ...
>
> > +            break;
> > +        }
> > +
> > +        dprintk(XENLOG_INFO, "Created vcpu %pv\n", tmp_v);
> ... this line preferably dropped. Can do when committing, provided you agree.
I agree with these changes.

Thanks a lot.

~ Oleksii