diff options
Diffstat (limited to 'meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch')
| -rw-r--r-- | meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch | 133 |
1 files changed, 0 insertions, 133 deletions
diff --git a/meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch b/meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch deleted file mode 100644 index 351184dabb..0000000000 --- a/meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch +++ /dev/null | |||
| @@ -1,133 +0,0 @@ | |||
| 1 | From 0a0d736ec89dffdbc83e7181166a99d5563acfe8 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Michael Jeanson <mjeanson@efficios.com> | ||
| 3 | Date: Mon, 5 Nov 2018 11:35:52 -0500 | ||
| 4 | Subject: [PATCH 1/9] Fix: signal: Distinguish between kernel_siginfo and | ||
| 5 | siginfo (v4.20) | ||
| 6 | |||
| 7 | See upstream commit : | ||
| 8 | |||
| 9 | commit ae7795bc6187a15ec51cf258abae656a625f9980 | ||
| 10 | Author: Eric W. Biederman <ebiederm@xmission.com> | ||
| 11 | Date: Tue Sep 25 11:27:20 2018 +0200 | ||
| 12 | |||
| 13 | signal: Distinguish between kernel_siginfo and siginfo | ||
| 14 | |||
| 15 | Linus recently observed that if we did not worry about the padding | ||
| 16 | member in struct siginfo it is only about 48 bytes, and 48 bytes is | ||
| 17 | much nicer than 128 bytes for allocating on the stack and copying | ||
| 18 | around in the kernel. | ||
| 19 | |||
| 20 | The obvious thing of only adding the padding when userspace is | ||
| 21 | including siginfo.h won't work as there are sigframe definitions in | ||
| 22 | the kernel that embed struct siginfo. | ||
| 23 | |||
| 24 | So split siginfo in two; kernel_siginfo and siginfo. Keeping the | ||
| 25 | traditional name for the userspace definition. While the version that | ||
| 26 | is used internally to the kernel and ultimately will not be padded to | ||
| 27 | 128 bytes is called kernel_siginfo. | ||
| 28 | |||
| 29 | The definition of struct kernel_siginfo I have put in include/signal_types.h | ||
| 30 | |||
| 31 | A set of buildtime checks has been added to verify the two structures have | ||
| 32 | the same field offsets. | ||
| 33 | |||
| 34 | To make it easy to verify the change kernel_siginfo retains the same | ||
| 35 | size as siginfo. The reduction in size comes in a following change. | ||
| 36 | |||
| 37 | Signed-off-by: Michael Jeanson <mjeanson@efficios.com> | ||
| 38 | Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com> | ||
| 39 | |||
| 40 | Upstream-Status: backport https://github.com/lttng/lttng-modules/commit/0a0d736ec89dffdbc83e7181166a99d5563acfe8 | ||
| 41 | |||
| 42 | Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com> | ||
| 43 | --- | ||
| 44 | instrumentation/events/lttng-module/signal.h | 41 ++++++++++++++++++-- | ||
| 45 | 1 file changed, 37 insertions(+), 4 deletions(-) | ||
| 46 | |||
| 47 | diff --git a/instrumentation/events/lttng-module/signal.h b/instrumentation/events/lttng-module/signal.h | ||
| 48 | index b3c9126..8783b52 100644 | ||
| 49 | --- a/instrumentation/events/lttng-module/signal.h | ||
| 50 | +++ b/instrumentation/events/lttng-module/signal.h | ||
| 51 | @@ -35,21 +35,24 @@ | ||
| 52 | * SEND_SIG_NOINFO means that si_code is SI_USER, and SEND_SIG_PRIV | ||
| 53 | * means that si_code is SI_KERNEL. | ||
| 54 | */ | ||
| 55 | -#if (LINUX_VERSION_CODE < KERNEL_VERSION(3,4,0)) | ||
| 56 | +#if (LINUX_VERSION_CODE >= KERNEL_VERSION(4,20,0)) | ||
| 57 | LTTNG_TRACEPOINT_EVENT(signal_generate, | ||
| 58 | |||
| 59 | - TP_PROTO(int sig, struct siginfo *info, struct task_struct *task), | ||
| 60 | + TP_PROTO(int sig, struct kernel_siginfo *info, struct task_struct *task, | ||
| 61 | + int group, int result), | ||
| 62 | |||
| 63 | - TP_ARGS(sig, info, task), | ||
| 64 | + TP_ARGS(sig, info, task, group, result), | ||
| 65 | |||
| 66 | TP_FIELDS( | ||
| 67 | ctf_integer(int, sig, sig) | ||
| 68 | LTTNG_FIELDS_SIGINFO(info) | ||
| 69 | ctf_array_text(char, comm, task->comm, TASK_COMM_LEN) | ||
| 70 | ctf_integer(pid_t, pid, task->pid) | ||
| 71 | + ctf_integer(int, group, group) | ||
| 72 | + ctf_integer(int, result, result) | ||
| 73 | ) | ||
| 74 | ) | ||
| 75 | -#else | ||
| 76 | +#elif (LINUX_VERSION_CODE >= KERNEL_VERSION(3,4,0)) | ||
| 77 | LTTNG_TRACEPOINT_EVENT(signal_generate, | ||
| 78 | |||
| 79 | TP_PROTO(int sig, struct siginfo *info, struct task_struct *task, | ||
| 80 | @@ -66,6 +69,20 @@ LTTNG_TRACEPOINT_EVENT(signal_generate, | ||
| 81 | ctf_integer(int, result, result) | ||
| 82 | ) | ||
| 83 | ) | ||
| 84 | +#else | ||
| 85 | +LTTNG_TRACEPOINT_EVENT(signal_generate, | ||
| 86 | + | ||
| 87 | + TP_PROTO(int sig, struct siginfo *info, struct task_struct *task), | ||
| 88 | + | ||
| 89 | + TP_ARGS(sig, info, task), | ||
| 90 | + | ||
| 91 | + TP_FIELDS( | ||
| 92 | + ctf_integer(int, sig, sig) | ||
| 93 | + LTTNG_FIELDS_SIGINFO(info) | ||
| 94 | + ctf_array_text(char, comm, task->comm, TASK_COMM_LEN) | ||
| 95 | + ctf_integer(pid_t, pid, task->pid) | ||
| 96 | + ) | ||
| 97 | +) | ||
| 98 | #endif | ||
| 99 | |||
| 100 | /** | ||
| 101 | @@ -82,6 +99,21 @@ LTTNG_TRACEPOINT_EVENT(signal_generate, | ||
| 102 | * This means, this can show which signals are actually delivered, but | ||
| 103 | * matching generated signals and delivered signals may not be correct. | ||
| 104 | */ | ||
| 105 | +#if (LINUX_VERSION_CODE >= KERNEL_VERSION(4,20,0)) | ||
| 106 | +LTTNG_TRACEPOINT_EVENT(signal_deliver, | ||
| 107 | + | ||
| 108 | + TP_PROTO(int sig, struct kernel_siginfo *info, struct k_sigaction *ka), | ||
| 109 | + | ||
| 110 | + TP_ARGS(sig, info, ka), | ||
| 111 | + | ||
| 112 | + TP_FIELDS( | ||
| 113 | + ctf_integer(int, sig, sig) | ||
| 114 | + LTTNG_FIELDS_SIGINFO(info) | ||
| 115 | + ctf_integer(unsigned long, sa_handler, (unsigned long) ka->sa.sa_handler) | ||
| 116 | + ctf_integer(unsigned long, sa_flags, ka->sa.sa_flags) | ||
| 117 | + ) | ||
| 118 | +) | ||
| 119 | +#else | ||
| 120 | LTTNG_TRACEPOINT_EVENT(signal_deliver, | ||
| 121 | |||
| 122 | TP_PROTO(int sig, struct siginfo *info, struct k_sigaction *ka), | ||
| 123 | @@ -95,6 +127,7 @@ LTTNG_TRACEPOINT_EVENT(signal_deliver, | ||
| 124 | ctf_integer(unsigned long, sa_flags, ka->sa.sa_flags) | ||
| 125 | ) | ||
| 126 | ) | ||
| 127 | +#endif | ||
| 128 | |||
| 129 | #if (LINUX_VERSION_CODE < KERNEL_VERSION(3,4,0)) | ||
| 130 | LTTNG_TRACEPOINT_EVENT_CLASS(signal_queue_overflow, | ||
| 131 | -- | ||
| 132 | 2.19.1 | ||
| 133 | |||
