Re: PR/50087 CVS commit: src/lib/libc/stdlib

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

On Mon, Aug 10, 2015 at 05:15:01AM +0000, Christos Zoulas wrote:
> The following reply was made to PR port-arm/50087; it has been noted by GNATS.
> 
> From: christos%zoulas.com@localhost (Christos Zoulas)
> To: gnats-bugs%NetBSD.org@localhost, port-arm-maintainer%netbsd.org@localhost, 
> 	gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost, martin%NetBSD.org@localhost
> Cc: 
> Subject: Re: PR/50087 CVS commit: src/lib/libc/stdlib
> Date: Mon, 10 Aug 2015 01:10:05 -0400
> 
>  On Aug 9,  8:10pm, joerg%britannica.bec.de@localhost (Joerg Sonnenberger) wrote:
>  -- Subject: Re: PR/50087 CVS commit: src/lib/libc/stdlib
>  
>  |  Oh, that's the use of sysctlbyname, which will internally call the
>  |  getoid function and that's caching the result. It should be using fixed
>  |  oids to reduce the startup overhead in general.
>  
>  We were thinking about fixing that, but then we wondered if it was the
>  last one... And fixing malloc to be more robust was more expedient. But
>  you are right; those should be using fixed oids.

Well, there are two approaches. The first would be to make the ordering
of constructors explicit by using different names for the sections. The
second would be to stick a diagassert in malloc for now, so that
___dlauxinfo() is not NULL.

Joerg