diff options
author | Bruce Ashfield <bruce.ashfield@gmail.com> | 2019-03-06 11:37:12 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-03-06 16:44:37 +0000 |
commit | cb3194828a3a83372c222103bfe3c2a61cea16d9 (patch) | |
tree | e8d966a23c12c76d939153d586664f4a01eba8bd /meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch | |
parent | 0f2b2b6c9877fbffe2bffddbcb920e1d9aa63c18 (diff) | |
download | poky-cb3194828a3a83372c222103bfe3c2a61cea16d9.tar.gz |
lttng: add 5.x fixup patches to 2.10.8 release
lttng hasn't released 2.10.9, but the stable/2.10 branch
contains changes that we need to build against the 5.0
kernel.
We add them here as patches, and we can drop them in the
future when lttng is updated.
(From OE-Core rev: f8bcf3486d44ebd867e9089d4dbd3b56726c9075)
Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
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, 133 insertions, 0 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 new file mode 100644 index 0000000000..351184dabb --- /dev/null +++ b/meta/recipes-kernel/lttng/lttng-modules/0001-Fix-signal-Distinguish-between-kernel_siginfo-and-si.patch | |||
@@ -0,0 +1,133 @@ | |||
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 | |||