| Message ID | 1500928945-32220-1-git-send-email-gromero@linux.vnet.ibm.com |
|---|---|
| State | New |
| Headers | show |
Hi, Since master is open now, could the following simple change be reviewed please? It just adds a new known note type in elf.h, and since other projects sync from glibc, notably elfutils (which currently supports the TM SPRs [1]) - I understand that it's good to update that on glibc. Thank you. Regards, Gustavo [1] https://sourceware.org/git/?p=elfutils.git;a=commit;h=55a471f5fe44945414af243613d4590c4e7cd8d1 On 24-07-2017 17:42, Gustavo Romero wrote: > Linux kernel 4.8 added the NT_PPC_TM_SPR regset for ppc64. > > * elf/elf.h: Add NT_PPC_TM_SPR. > --- > ChangeLog | 4 ++++ > elf/elf.h | 1 + > 2 files changed, 5 insertions(+) > > diff --git a/ChangeLog b/ChangeLog > index 554fdac..04d680a 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,7 @@ > +2017-07-24 Gustavo Romero <gromero@linux.vnet.ibm.com> > + > + * elf/elf.h: Add NT_PPC_TM_SPR. > + > 2017-07-24 Adhemerval Zanella <adhemerval.zanella@linaro.org> > H.J. Lu <hongjiu.lu@intel.com> > > diff --git a/elf/elf.h b/elf/elf.h > index 3900b4c..433f310 100644 > --- a/elf/elf.h > +++ b/elf/elf.h > @@ -763,6 +763,7 @@ typedef struct > #define NT_PPC_SPE 0x101 /* PowerPC SPE/EVR registers */ > #define NT_PPC_VSX 0x102 /* PowerPC VSX registers */ > #define NT_386_TLS 0x200 /* i386 TLS slots (struct user_desc) */ > +#define NT_PPC_TM_SPR 0x10c /* PowerPC HW Transactional Memory SPRs */ > #define NT_386_IOPERM 0x201 /* x86 io permission bitmap (1=deny) */ > #define NT_X86_XSTATE 0x202 /* x86 extended state using xsave */ > #define NT_S390_HIGH_GPRS 0x300 /* s390 upper register halves */ >
On Jul 24 2017, Gustavo Romero <gromero@linux.vnet.ibm.com> wrote: > diff --git a/elf/elf.h b/elf/elf.h > index 3900b4c..433f310 100644 > --- a/elf/elf.h > +++ b/elf/elf.h > @@ -763,6 +763,7 @@ typedef struct > #define NT_PPC_SPE 0x101 /* PowerPC SPE/EVR registers */ > #define NT_PPC_VSX 0x102 /* PowerPC VSX registers */ > #define NT_386_TLS 0x200 /* i386 TLS slots (struct user_desc) */ > +#define NT_PPC_TM_SPR 0x10c /* PowerPC HW Transactional Memory SPRs */ It doesn't make sense to add this symbol without adding the other ones too. Also, keep the list sorted. Andreas.
Hi Andreas, On 02-08-2017 12:09, Andreas Schwab wrote: > It doesn't make sense to add this symbol without adding the other ones > too. Also, keep the list sorted. Thanks for reviewing it. I'll send a v2. Regards, Gustavo
diff --git a/ChangeLog b/ChangeLog index 554fdac..04d680a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2017-07-24 Gustavo Romero <gromero@linux.vnet.ibm.com> + + * elf/elf.h: Add NT_PPC_TM_SPR. + 2017-07-24 Adhemerval Zanella <adhemerval.zanella@linaro.org> H.J. Lu <hongjiu.lu@intel.com> diff --git a/elf/elf.h b/elf/elf.h index 3900b4c..433f310 100644 --- a/elf/elf.h +++ b/elf/elf.h @@ -763,6 +763,7 @@ typedef struct #define NT_PPC_SPE 0x101 /* PowerPC SPE/EVR registers */ #define NT_PPC_VSX 0x102 /* PowerPC VSX registers */ #define NT_386_TLS 0x200 /* i386 TLS slots (struct user_desc) */ +#define NT_PPC_TM_SPR 0x10c /* PowerPC HW Transactional Memory SPRs */ #define NT_386_IOPERM 0x201 /* x86 io permission bitmap (1=deny) */ #define NT_X86_XSTATE 0x202 /* x86 extended state using xsave */ #define NT_S390_HIGH_GPRS 0x300 /* s390 upper register halves */