RE: [PATCH v5 06/12] gdb, gdbserver: Add support of Intel shadow stack pointer register.

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

> -----Original Message-----
> From: Andrew Burgess <aburgess@redhat.com>
> Sent: Tuesday, July 29, 2025 3:51 PM
> To: Schimpe, Christina <christina.schimpe@intel.com>; gdb-
> patches@sourceware.org
> Cc: thiago.bauermann@linaro.org; luis.machado@arm.com
> Subject: Re: [PATCH v5 06/12] gdb, gdbserver: Add support of Intel shadow
> stack pointer register.
> 
> Christina Schimpe <christina.schimpe@intel.com> writes:
> 
> > diff --git a/gdb/x86-tdep.c b/gdb/x86-tdep.c index
> > 6646b1157c8..06dec4e1f90 100644
> > --- a/gdb/x86-tdep.c
> > +++ b/gdb/x86-tdep.c
> > @@ -17,10 +17,31 @@
> >     You should have received a copy of the GNU General Public License
> >     along with this program.  If not, see
> > <http://www.gnu.org/licenses/>.  */
> >
> > +#include "i386-tdep.h"
> >  #include "x86-tdep.h"
> >  #include "symtab.h"
> >
> >
> > +/* See x86-tdep.h.  */
> > +
> > +void
> > +x86_supply_ssp (regcache *regcache, const uint64_t ssp) {
> > +  i386_gdbarch_tdep *tdep = gdbarch_tdep<i386_gdbarch_tdep>
> > +(regcache->arch ());
> > +  gdb_assert (tdep != nullptr && tdep->ssp_regnum > 0);
> > +  regcache->raw_supply (tdep->ssp_regnum, &ssp); }
> > +
> > +/* See x86-tdep.h.  */
> > +
> > +void
> > +x86_collect_ssp (const regcache *regcache, uint64_t& ssp)
> 
> While reviewing the next patch, I spotted a white space issue here, it should
> be 'uint64_t &ssp'.  And the same in the declaration too.
> 
> Thanks,
> Andrew

Yes, will fix.
 
Thank you!
Christina
Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Sean Fennelly, Jeffrey Schneiderman, Tiffany Doon Silva
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928