Re: CVS commit: src/sys/arch/amd64

[Maxime Villard <max%m00nbsd.net@localhost>]

Le 30/12/2019 à 16:15, Emmanuel Dreyfus a écrit :
> On Sat, Dec 28, 2019 at 02:22:21AM +0000, Emmanuel Dreyfus wrote:
>>> Regardless of whether it is needed in this specific case, cutting the 2MBs
>>> of zero in the binary is wanted. Unfortunately last I looked at this (two
>>> years ago) there were some non-obvious consequences, and it needs to be
>>> carefully done.
>>
>> Any hints about the problems you encountered? Perhaps we can work it
>> around with an . = ALIGN(__LARGEE_PAGE_SIZE); before including .text.user ?
> 
> No anwser here? It is difficult to address an unknown problem...

Sorry for the delay. I don't remember the specifics, but it had to do with
misaligned sections in the end. There is also some pretty retarded code in
loadfile_elf32.c that should like some investigation, at least for this
kind of change.

I think max-page-size=0x1000 is the right thing to do, but someone needs to
verify that the resulting binary is correct and that the resulting in-memory
layout is correct too.