On 24/09/15 10:52, Jan Beulich wrote:
>>>> On 24.09.15 at 10:14, <andrew.coop...@citrix.com> wrote:
>> The ALIGN(STACK_SIZE) actually belongs with .bss.stack_aligned, but
>> __init_end still needs page alignment because of the init sections being
>> freed and returned to the domheap after boot.
> Logically that change makes sense, but ...
>
>> --- a/xen/arch/x86/xen.lds.S
>> +++ b/xen/arch/x86/xen.lds.S
>> @@ -158,11 +158,13 @@ SECTIONS
>> __xsm_initcall_start = .;
>> *(.xsm_initcall.init)
>> __xsm_initcall_end = .;
>> +
>> + . = ALIGN(PAGE_SIZE);
>> } :text
>> - . = ALIGN(STACK_SIZE);
>> __init_end = .;
>>
>> .bss : { /* BSS */
>> + . = ALIGN(STACK_SIZE);
>> __bss_start = .;
>> *(.bss.stack_aligned)
>> . = ALIGN(PAGE_SIZE);
> ... any pages between __init_end and __bss_start will all of the
> sudden no longer get freed. I.e. you'll want to move __init_end
> as well (which belongs inside some section anyway).
Hmm so it does. I will just defer this change to my superpage mapping
series which will fix this issue in a slightly different way.
~Andrew
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
