The patch titled
Use decimal for PTRACE_ATTACH and PTRACE_DETACH.
has been added to the -mm tree. Its filename is
use-decimal-for-ptrace_attach-and-ptrace_detach.patch
See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this
------------------------------------------------------
Subject: Use decimal for PTRACE_ATTACH and PTRACE_DETACH.
From: Roland McGrath <roland@xxxxxxxxxx>
It is sure confusing that linux/ptrace.h has:
#define PTRACE_SINGLESTEP 9
#define PTRACE_ATTACH 0x10
#define PTRACE_DETACH 0x11
#define PTRACE_SYSCALL 24
All the low-numbered constants are in decimal, but the last two in hex.
It sure makes it likely that someone will look at this and think that
9, 10, 11 are used, and that 16 and 17 are not used.
How about we use the same notation for all the numbers [0,24] in the
same short list?
Signed-off-by: Roland McGrath <roland@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
include/linux/ptrace.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff -puN include/linux/ptrace.h~use-decimal-for-ptrace_attach-and-ptrace_detach include/linux/ptrace.h
--- a/include/linux/ptrace.h~use-decimal-for-ptrace_attach-and-ptrace_detach
+++ a/include/linux/ptrace.h
@@ -16,8 +16,8 @@
#define PTRACE_KILL 8
#define PTRACE_SINGLESTEP 9
-#define PTRACE_ATTACH 0x10
-#define PTRACE_DETACH 0x11
+#define PTRACE_ATTACH 16
+#define PTRACE_DETACH 17
#define PTRACE_SYSCALL 24
_
Patches currently in -mm which might be from roland@xxxxxxxxxx are
fix-wrong-error-code-on-interrupted-close-syscalls-fix.patch
posix-timers-fix-clock_nanosleep-doesnt-return-the-remaining-time-in-compatibility-mode-2.patch
posix-timers-fix-the-flags-handling-in-posix_cpu_nsleep-2.patch
has_stopped_jobs-cleanup.patch
__dequeue_signal-cleanup.patch
kexec-warning-fix.patch
use-decimal-for-ptrace_attach-and-ptrace_detach.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
