[PATCH] S390: Sync ptrace.h with kernel. [BZ #21539]

Stefan Liebler stli@linux.vnet.ibm.com
Thu Jul 20 08:32:00 GMT 2017


On 07/20/2017 10:06 AM, Carlos O'Donell wrote:
> On 07/20/2017 03:38 AM, Stefan Liebler wrote:
>> Carlos, shall we commit Dimitry's patch before the release?
>> Then we don't have this regression compared to glibc 2.25 release.
> 
> I'm OK with Dmitry's patch going in for 2.26 to fix the regression.
Okay.
> > Could you please work with Dmitry to get this committed?
Okay.
Dimitry, I have one small note to your patch:
Can you add a space before the undefs and add the comment like this:
(according to 
https://sourceware.org/glibc/wiki/Style_and_Conventions#Nested_C_Preprocessor_Directives)

#if defined _LINUX_PTRACE_H || defined _S390_PTRACE_H
/* Kludge to stop stuff gdb & strace compiles from getting upset
  */
# undef PTRACE_TRACEME
...
#endif /* defined _LINUX_PTRACE_H || defined _S390_PTRACE_H  */

Then the patch is okay.
Can you please commit the patch?
Otherwise, post the final patch, then I'll commit it.

> 
> I'll work on fixing this coordination issue with the kernel when we
> reopen for development.
> 
Okay.


Thanks.
Stefan



More information about the Libc-alpha mailing list