Re: Re: [LEDE-DEV] [PATCH 1/2] musl: update to 1.1.16+ git HEAD 2017-09-28

[Szabolcs Nagy <nsz@port70.net>]

* Felix Fietkau <nbd@nbd.name> [2017-10-16 16:09:16 +0200]:
> On 2017-10-16 16:03, Szabolcs Nagy wrote:
> > * Koen Vandeputte <koen.vandeputte@ncentric.com> [2017-10-16 10:31:11 +0200]:
> >> I've proposed to update the musl libc within the LEDE project and received
> >> some feedback from Syrone Wong (in cc).
> >> 
> >> According to him, your upstream commit in musl [1] seems to break something.
> >> (see below)
> >> 
> >> - Can you confirm this?
> >> - Would you have any clue what the reason could be?
> > ...
> >> [1]
> >> "powerpc{64}: fix MAP_NORESERVE and MAP_LOCKED in mman.h"
> >> http://git.musl-libc.org/cgit/musl/commit/?id=c10bc61508dc52b8315084e628f36a6c3c2dabb1
> >> 
> > 
> > these are rarely used mmap flags, i think my patch is correct.
> > 
> > the old value in musl was wrong on powerpc (i think the old
> > value did not collide with other map flags so the kernel would
> > report EINVAL if these flags were ever used).
> I don't think this commit is the cause at all. The bug reporter was
> using an ARM device and this commit only touches powerpc header files.
> Probably the issue is somewhere between the version that LEDE uses now
> and the version with this commit id.
> 

some more details would be useful, e.g. strace
output of the failing processes.

or libc.so before and after (mmap code in my libc.so
looks ok, but there might be some toolchain issue).

does the system use >4k page size?