On 19/06/2019 12:02, Roger Pau Monne wrote: > If the hypervisor has been built with EFI support (ie: multiboot2).
Seeing as this continues the sentence from the subject, it should start without a capital. Otherwise the result is werd to read. > This allows to position the .reloc section correctly in the output > binary, or else the linker might place .reloc before the .text > section. Really? How can this be a legitimate transformation for the linker to make? > > Note that the .reloc section is moved before .bss for two reasons: in > order for the resulting binary to not contain any section with data > after .bss, so that the file size can be smaller than the loaded > memory size, and because the data it contains is read-only, so it > belongs with the other sections containing read-only data. The content of .relocs is transformed via mkreloc to become __base_relocs_{start,end} and shouldn't exist into the final linked image AFAICT. Since the MB1/MB2 builds aren't relocatable, I think we might be able to get away with simply excluding them in the non-EFI build. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel