Re: PR/54093 CVS commit: src

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

On Tue, Nov 05, 2019 at 08:25:01PM +0000, Takeshi Nakayama wrote:
> The following reply was made to PR port-sparc/54093; it has been noted by GNATS.
> 
> From: Takeshi Nakayama <nakayama%NetBSD.org@localhost>
> To: gnats-bugs%netbsd.org@localhost, joerg%netbsd.org@localhost
> Cc: martin%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost,
>         gson%gson.org@localhost
> Subject: Re: PR/54093 CVS commit: src
> Date: Wed, 06 Nov 2019 05:24:24 +0900 (JST)
> 
>  >>> "Joerg Sonnenberger" <joerg%netbsd.org@localhost> wrote
>  
>  > The following reply was made to PR port-sparc/54093; it has been noted by GNATS.
>  > 
>  > From: "Joerg Sonnenberger" <joerg%netbsd.org@localhost>
>  > To: gnats-bugs%gnats.NetBSD.org@localhost
>  > Cc: 
>  > Subject: PR/54093 CVS commit: src
>  > Date: Mon, 4 Nov 2019 12:45:10 +0000
>  > 
>  >  Module Name:	src
>  >  Committed By:	joerg
>  >  Date:		Mon Nov  4 12:45:10 UTC 2019
>  >  
>  >  Modified Files:
>  >  	src/lib/libc/tls: Makefile.inc tls.c
>  >  	src/libexec/ld.elf_so: Makefile tls.c
>  >  	src/usr.bin/ldd: Makefile.elf
>  >  
>  >  Log Message:
>  >  PR 54093: Align static TLS area to max_align_t.
>  
>  I think static TLS area should align with p_align in elf program
>  header, not with sizeof(max_align_t).

We don't support non-standard alignments of TLS. Fixing that is
intrusive in the dynamic linker and I have no plan to do that ATM.

>  sizeof(max_align_t) is 12 on i386, it is not valid on roundup2.
>  This seems to cause some atf tests to fail.

That would be a separate bug on i386.

Joerg