This is the mail archive of the
mailing list for the glibc project.
Re: [PATCH] NEWS: Intel CET support hasn't be validated for i686
- From: Carlos O'Donell <carlos at redhat dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>, GNU C Library <libc-alpha at sourceware dot org>
- Date: Thu, 19 Jul 2018 13:14:54 -0400
- Subject: Re: [PATCH] NEWS: Intel CET support hasn't be validated for i686
- References: <CAMe9rOpBM0-tanrxEu9Vk=jdHpg=GGugP3TSm9BD+KUfDCVqEQ@mail.gmail.com>
On 07/19/2018 01:12 PM, H.J. Lu wrote:
> On Wed, Jul 18, 2018 at 2:16 PM, Joseph Myers <email@example.com> wrote:
>> On Wed, 18 Jul 2018, H.J. Lu wrote:
>>> 2. On 64-bit CET SDV under CET kernel with --enable-cet
>>> a. Only x86_64 and x32 tested.
>>> b. i686 isn't tested since 64-bit CET SDV doesn't support i686.
>> So the key gap in test coverage is the lack of any evidence that the i686
>> code does actually work on CET processors. I think that at least merits a
>> warning in the NEWS and install.texi entries about the unvalidated nature
>> of the i686 CET support (which would be removed from install.texi at a
>> later date once the i686 support has been validated, on a simulator or
>> hardware, and any issues resolved).
> How about this?
This looks good to me and is very clear about the i686 support.
OK for 2.28.
Reviewed-by: Carlos O'Donell <firstname.lastname@example.org>
> -- H.J.
> From 3191cbcf1ff664aa2f12a6d83f1349639e051208 Mon Sep 17 00:00:00 2001
> From: "H.J. Lu" <email@example.com>
> Date: Wed, 18 Jul 2018 14:55:46 -0700
> Subject: [PATCH] NEWS: Intel CET support hasn't be validated for i686
> NEWS | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
> diff --git a/NEWS b/NEWS
> index daef815ae7..b4055bf99e 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -17,7 +17,10 @@ Major new features:
> feature is currently supported on i386, x86_64 and x32 with GCC 8 and
> binutils 2.29 or later. Note that CET-enabled glibc requires CPUs
> capable of multi-byte NOPs, like x86-64 processors as well as Intel
> - Pentium Pro or newer.
> + Pentium Pro or newer. NOTE: --enable-cet has been tested for i686,
> + x86_64 and x32 on non-CET processors. --enable-cet has been tested
> + for x86_64 and x32 on CET SDVs. But Intel CET support hasn't be
> + validated for i686.
> * The GNU C Library now has correct support for ABSOLUTE symbols
> (SHN_ABS-relative symbols). Previously such ABSOLUTE symbols were
> -- 2.17.1