On 18.03.2020 12:46, David Woodhouse wrote: > From: David Woodhouse <[email protected]> > > The creation of dom0 can be relatively self-contained. Shift it into > a separate function and simplify __start_xen() a little bit. > > This is a cleanup in its own right, but will be even more desireable > when live update provides an alternative path through __start_xen() > that doesn't involve creating a new dom0 at all. > > Move the calculation of the 'initrd' parameter for create_dom0() > down past the cosmetic printk about NX support, because in the fullness > of time the whole initrd and create_dom0() part will be under the same > "not live update" conditional. And in the meantime it's just neater. > > Signed-off-by: David Woodhouse <[email protected]>
Acked-by: Jan Beulich <[email protected]> with one further small cosmetic aspect taken care of (which ought to be doable while committing): > --- a/xen/arch/x86/setup.c > +++ b/xen/arch/x86/setup.c > @@ -679,6 +679,92 @@ static unsigned int __init copy_bios_e820(struct > e820entry *map, unsigned int li > return n; > } > > +static struct domain * __init create_dom0(const module_t *image, We put blanks to the left of stars, but not to the right. (I'm sure you'd be able to point out examples to the contrary, but that's what we're at least striving for, from all I can tell.) Jan _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
