This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] i686: Add missing IS_IN (libc) guards to vectorized strcspn
- From: Florian Weimer <fweimer at redhat dot com>
- To: Carlos O'Donell <carlos at redhat dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Wed, 14 Jun 2017 14:40:47 +0200
- Subject: Re: [PATCH] i686: Add missing IS_IN (libc) guards to vectorized strcspn
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx05.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx05.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=fweimer at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 4DEBE13AA1
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 4DEBE13AA1
- References: <20170614061950.400FE4010728F@oldenburg.str.redhat.com> <e08944a2-887f-a3f0-e3be-b086b58e8685@redhat.com>
On 06/14/2017 02:32 PM, Carlos O'Donell wrote:
> On 06/14/2017 02:19 AM, Florian Weimer wrote:
>> Since commit d957c4d3fa48d685ff2726c605c988127ef99395 (i386: Compile
>> rtld-*.os with -mno-sse -mno-mmx -mfpmath=387), vector intrinsics can
>> no longer be used in ld.so, even if the compiled code never makes it
>> into the final ld.so link. This commit adds the missing IS_IN (libc)
>> guard to the SSE 4.2 strcspn implementation, so that it can be used from
>> ld.so in the future.
>>
>> 2017-06-14 Florian Weimer <fweimer@redhat.com>
>>
>> * sysdeps/i386/i686/multiarch/strcspn-c.c: Add IS_IN (libc) guard.
>> * sysdeps/i386/i686/multiarch/varshift.c: Likewise.
>
> This looks good to me.
Thanks.
> We have struggled with these issues for years. I wish there was a more
> automatic way to enable new functions for use in ld.so, but there isn't.
>
> Out of curiosity, if you don't apply this patch, and use strcspn, does
> the tst-ld-sse-use.sh test fail?
No. The files containing IFUNC resolvers are compiled in rtld mode
(!IS_IN (libc)), and they do not reference the definitions in
strcpsn-c.c anymore. But the list of objects is constructed based on
the contents of libc-pic.a, and that obviously contains strcspn-c.o.
With current upstream, after adding a reference to strcspn within ld.so,
we attempt to recompile strcspn-c.c, but that fails with GCC errors due
to -mno-sse etc. added as a safeguard. Prior to commit
d957c4d3fa48d685ff2726c605c988127ef99395, strcspn-c.c would compile
successfully, but since the IFUNC resolver is compiled out due to !IS_IN
(libc), the link editor will not pick up strcspn-c.o, although it is
present in rtld-libc.a.
Florian