Re: CVS commit: src/sys/kern

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

On Wed, Sep 01, 2010 at 05:27:29PM -0700, Chuck Silvers wrote:
> this change breaks some executables on some platforms (eg. tcsh on sparc64).
> here's what "objdump -p" shows for this file:
> 
>     LOAD off    0x0000000000000000 vaddr 0x0000000000100000 paddr 
> 0x0000000000100000 align 2**20
>          filesz 0x000000000004d533 memsz 0x000000000004d533 flags r-x
>     LOAD off    0x000000000004e000 vaddr 0x000000000024e000 paddr 
> 0x000000000024e000 align 2**20
>          filesz 0x0000000000004e78 memsz 0x0000000000031dd0 flags rwx
> 
> since the data PT_LOAD section is marked executable for some reason,

Hm. I see two possible options:
(1) Really provide no data section and set epp_daddr to end of all text
sections if noone was provided.

(2) Don't consider writeable sections as text segment if one is already
present.

I think the first option is making less assumptions.

Joerg