V2 [PATCH] x86: Adjust tst-cpu-features-supports.c for GCC 11

Florian Weimer fweimer@redhat.com
Fri Dec 4 13:14:42 GMT 2020


* H. J. Lu:

> On Fri, Dec 4, 2020 at 12:12 AM Florian Weimer <fweimer@redhat.com> wrote:
>>
>> * H. J. Lu via Libc-alpha:
>>
>> > Set RDRAND usable if CPU supports RDRAND.  Also check HAS_CPU_FEATURE
>> > instead of CPU_FEATURE_USABLE for FSGSBASE, LM and XSAVES since
>> > FSGSBASE/LM require kernel support and XSAVES is supervisor-mode only.
>>
>> I think the commit subject is misleading because the RDRAND part is a
>> glibc bug fix.  Rest looks okay to me.
>>
>
> Here is the V2 patch.  It changed IBT and SHSTK to check HAS_CPU_FEATURE
> since they need OS support.

Patch is okay.  But I do wonder why __builtin_cpu_supports doesn't
handle this.  Based on the GCC documention, I would say it should
perform what you call a “usable” check.

Thanks,
Florian
-- 
Red Hat GmbH, https://de.redhat.com/ , Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Brian Klemm, Laurie Krebs, Michael O'Neill



More information about the Libc-alpha mailing list