Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 24 Nov 2025 05:58:29 +0000
From:      Colin Percival <cperciva@tarsnap.com>
To:        Konstantin Belousov <kib@FreeBSD.org>, src-committers@FreeBSD.org,  dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: b2b3d2a962eb - main - rtld-elf: move powerpc-specific auxv compat code into arch hook
Message-ID:  <0100019ab47136f7-c6191ed4-563b-4c7a-b8bf-d26a6f0893ed-000000@email.amazonses.com>
In-Reply-To: <69206a47.27746.245d9408@gitrepo.freebsd.org>
References:  <69206a47.27746.245d9408@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On 11/21/25 05:33, Konstantin Belousov wrote:
> commit b2b3d2a962eb00005641546fbe672b95e5d0672a
> Author:     Konstantin Belousov <kib@FreeBSD.org>
> AuthorDate: 2025-11-18 11:06:04 +0000
> Commit:     Konstantin Belousov <kib@FreeBSD.org>
> CommitDate: 2025-11-21 13:33:17 +0000
> 
>      rtld-elf: move powerpc-specific auxv compat code into arch hook
>      
>      Tested by:      Timothy Pearson (tpearson_raptorengineering.com)
>      Sponsored by:   The FreeBSD Foundation
>      MFC after:      1 week
>      Differential revision:  https://reviews.freebsd.org/D53801
> --- a/libexec/rtld-elf/powerpc64/reloc.c
> +++ b/libexec/rtld-elf/powerpc64/reloc.c
> @@ -737,3 +737,29 @@ __tls_get_addr(tls_index* ti)
>   	return (tls_get_addr_common(_tcb_get(), ti->ti_module, ti->ti_offset +
>   	    TLS_DTV_OFFSET));
>   }
> +
> +void
> +arch_fix_auxv(Elf_Auxinfo *aux, Elf_Auxinfo *aux_info[])
> +{
> +	Elf_Auxinfo *aux;

This does not compile:
> /usr/src/libexec/rtld-elf/powerpc64/reloc.c:744:15: error: redefinition of 'aux'
>   744 |         Elf_Auxinfo *aux;
>       |                      ^
> /usr/src/libexec/rtld-elf/powerpc64/reloc.c:742:28: note: previous definition is here

Which makes me wonder just how much testing this precise patch has
received. ;-)

-- 
Colin Percival
FreeBSD Release Engineering Lead & EC2 platform maintainer
Founder, Tarsnap | www.tarsnap.com | Online backups for the truly paranoid




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0100019ab47136f7-c6191ed4-563b-4c7a-b8bf-d26a6f0893ed-000000>