Re: [Buildroot] [PATCH] package/x264: disable assembly code on x86 + musl + PIC/PIE

["J. Neuschäfer via buildroot" <buildroot@buildroot.org>]

On Sun, Aug 25, 2024 at 03:06:52PM +0200, Thomas Petazzoni wrote:
> On Sat, 24 Aug 2024 22:11:02 +0200
> J. Neuschäfer via buildroot <buildroot@buildroot.org> wrote:
>
> > The x264 package uses large amounts of non-PIC assembly code
> > (e.g. common/x86/dct-a.asm), which results in textrels, which aren't
> > supported by musl-libc's dynamic linker.
> >
> > Disable x264's assembly code when compiling for x86 with PIC/PIE and
> > musl-libc to avoid this particular incompatibility.
> >
> > Reported-by: Yann Morin <yann.morin@orange.com>
> > Fixes: https://lore.kernel.org/buildroot/ZrsirnrvgsEIpAJI@tl-lnx-nyma7486-2/
> > Signed-off-by: J. Neuschäfer <j.neuschaefer@gmx.net>
> > ---
> >  package/x264/x264.mk | 6 ++++++
> >  1 file changed, 6 insertions(+)
>
> I've applied to master, but to be honest, I'm not really thrilled. Has
> this been taken up to the musl's maintainers? It's rather annoying than
> on a major architecture like x86 we can't get the assembly optimized
> x264 code.

I've raised this topic in the musl IRC channel and the consensus was
that x264 should be fixed not to use textrels.


-- jn
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot