RE: [PATCH 1/1] gdb: Deprecate MPX commands.

["Schimpe, Christina" <christina.schimpe@intel.com>]

Hi Eli, 

Thanks a lot for your quick reply.

> "Will be removed" or "might be removed"?  IOW, are we certain we will
> definitely remove this, or do we just want to warn users that we might?

Our goal would be to deprecate the MPX commands for gdb 15, if possible.
Please let me know if there is a problem with that or if there is something required to track this properly for gdb 15. I am not familiar with the procedure. I am adding Joel due to that to this conversation.

We would like to remove MPX for gdb 16.  So yes, we certainly will remove MPX. Hopefully for gdb 16 already.

BR + thanks a lot in advance,
Christina

> -----Original Message-----
> From: Eli Zaretskii <eliz@gnu.org>
> Sent: Thursday, March 7, 2024 1:35 PM
> To: Schimpe, Christina <christina.schimpe@intel.com>
> Cc: gdb-patches@sourceware.org
> Subject: Re: [PATCH 1/1] gdb: Deprecate MPX commands.
> 
> > From: "Schimpe, Christina" <christina.schimpe@intel.com>
> > Date: Thu,  7 Mar 2024 12:11:11 +0000
> >
> > This patch deprecates the MPX commands "show/set mpx bound".
> > Intel listed Intel(R) Memory Protection Extensions (MPX) as removed in
> > 2019.  Following gcc v9.1, the linux kernel v5.6 and glibc v2.35,
> > deprecate MPX in GDB.
> > ---
> >  gdb/NEWS            | 3 +++
> >  gdb/doc/gdb.texinfo | 3 +++
> >  gdb/i386-tdep.c     | 6 ++++--
> >  3 files changed, 10 insertions(+), 2 deletions(-)
> 
> Thanks.
> 
> > diff --git a/gdb/NEWS b/gdb/NEWS
> > index 2638b3e0d9c..d8ac0bb06a7 100644
> > --- a/gdb/NEWS
> > +++ b/gdb/NEWS
> > @@ -3,6 +3,9 @@
> >
> >  *** Changes since GDB 14
> >
> > +* The MPX commands "show/set mpx bound" have been deprecated, as
> > +Intel
> > +  listed MPX as removed in 2019.
> 
> This part is okay.
> 
> > --- a/gdb/doc/gdb.texinfo
> > +++ b/gdb/doc/gdb.texinfo
> > @@ -26652,6 +26652,9 @@ whose bounds are to be changed,
> @var{lbound}
> > and @var{ubound} are new values  for lower and upper bounds
> respectively.
> >  @end table
> >
> > +Both commands are deprecated and will be removed in future versions
> > +of @value{GDBN}.  MPX itself was listed as removed by Intel in 2019.
> 
> "Will be removed" or "might be removed"?  IOW, are we certain we will
> definitely remove this, or do we just want to warn users that we might?
> 
> Reviewed-By: Eli Zaretskii <eliz@gnu.org>
Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928