On 15/01/2020 16:27, Jan Beulich wrote: > On 15.01.2020 15:36, Andrew Cooper wrote: >> On 15/01/2020 10:26, Jan Beulich wrote: >>> While it has been me to introduce this, the use of | there has become >>> (and perhaps was from the very beginning) misleading. Rather than >>> avoiding the right side of it when linking the xen.efi intermediate file >>> at a different base address, make the expression cope with that case, >>> thus verifying placement on every step. >>> >>> Signed-off-by: Jan Beulich <jbeul...@suse.com> >> Acked-by: Andrew Cooper <andrew.coop...@citrix.com> as this is simply a >> rearranging, but... >> >>> --- a/xen/arch/x86/xen.lds.S >>> +++ b/xen/arch/x86/xen.lds.S >>> @@ -351,8 +351,8 @@ SECTIONS >>> .comment 0 : { *(.comment) } >>> } >>> >>> -ASSERT(__image_base__ > XEN_VIRT_START | >>> - __2M_rwdata_end <= XEN_VIRT_END - NR_CPUS * PAGE_SIZE, >>> +ASSERT(__2M_rwdata_end <= XEN_VIRT_END - XEN_VIRT_START + __XEN_VIRT_START >>> - >>> + NR_CPUS * PAGE_SIZE, >> ... doesn't this want a stubs_per_page term? We don't have 4k per cpu. > Good point - let me see if I can fix this at this same occasion. > I guess when introducing these pages I had one per CPU initially, > and then forgot to update the assertion here (it being too strict > of course is better than it being too lax).
With some suitable term, feel free to up my A-by to R-by. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel