Re: CVS commit: src/sys/kern

2010-09-02 Thread Joerg Sonnenberger
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 off0x vaddr 0x0010 paddr 
 0x0010 align 2**20
  filesz 0x0004d533 memsz 0x0004d533 flags r-x
 LOAD off0x0004e000 vaddr 0x0024e000 paddr 
 0x0024e000 align 2**20
  filesz 0x4e78 memsz 0x00031dd0 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


Re: CVS commit: src/sys/kern

2010-09-02 Thread Martin Husemann
For the record: this is PR 43817, and it seems to affect quite a lot of
archs.

Martin