On 06.10.2024 23:49, Daniel P. Smith wrote: > @@ -302,6 +304,13 @@ static struct boot_info __init > *multiboot_fill_boot_info(unsigned long mbi_p) > bi->memmap_length = mbi->mmap_length; > } > > + /* > + * This will iterate over all modules to include an extra mb module, > which > + * should have been reserved to hold an entry for Xen. > + */ > + for ( i = 0; i <= bi->nr_modules; i++ ) > + bi->mods[i].mod = &mods[i];
I find the comment difficult to follow / match with code here and elsewhere. How about "Iterate over all modules, including the extra one which should have been reserved for Xen itself"? Jan
