Re: [PATCH] sh: select BINFMT_FLAT_NO_DATA_START_OFFSET for noMMU

[Waldemar Brodkorb <wbx@openadk.org>]

Hi Geert,
Geert Uytterhoeven wrote,

> Hi Waldemar,
> 
> On Tue, 20 Oct 2026 at 06:00, Waldemar Brodkorb <wbx@openadk.org> wrote:
> > binfmt_flat offsets the in-memory start of the data segment by
> > MAX_SHARED_LIBS words and then aligns it to FLAT_DATA_ALIGN, so FLAT_FLAG_RAM
> > binaries end up with a gap between text and data (commit 2217b9826246
> > ("binfmt_flat: revert "binfmt_flat: don't offset the data start"")).
> >
> > SH position-independent code finds the GOT with a PC-relative mova of
> > _GLOBAL_OFFSET_TABLE_, which assumes data follows text directly, as elf2flt
> > lays out the image. With the gap, the GOT register points below the real GOT,
> > so every GOT load reads the neighbouring slot and gets a wrong pointer. On a
> > J2 (sh2eb, noMMU) this made userland crash or hang right after start-up.
> > In my case the data was moved up by 0x20 bytes.
> >
> > Commit 04d82a6d0881 ("binfmt_flat: allow not offsetting data start") added
> > CONFIG_BINFMT_FLAT_NO_DATA_START_OFFSET for architectures that cannot
> > tolerate such a gap; currently only riscv noMMU selects it. Select it on SH
> > as well when building without an MMU.
> 
> Thanks for your patch!
> 
> > Tested on a J2 board (Numato Mimas V2, sh2eb, noMMU) with Linux 6.18 and
> > FLAT binaries built with elf2flt: with this change the data segment starts
> > exactly at the end of text and init runs.
> 
> V6.18 is getting a bit old. Can you please verify on v7.3?

The problem with Linus 7.3 tree is, that Florian's patch which fixes the
compile error with nstree.c is not in. Or do you mean linux-sh tree
for-next branch with Florian's commit eb549e808390eeecf07618b8c3a82a6f2a887e70?
 
best regards
 Waldemar