diff options
| -rw-r--r-- | meta/recipes-kernel/lttng/lttng-modules/0001-compaction-make-tracepoints-condtional-on-CONFIG_COM.patch | 103 | ||||
| -rw-r--r-- | meta/recipes-kernel/lttng/lttng-modules_2.13.4.bb | 1 |
2 files changed, 104 insertions, 0 deletions
diff --git a/meta/recipes-kernel/lttng/lttng-modules/0001-compaction-make-tracepoints-condtional-on-CONFIG_COM.patch b/meta/recipes-kernel/lttng/lttng-modules/0001-compaction-make-tracepoints-condtional-on-CONFIG_COM.patch new file mode 100644 index 0000000000..305ca7cb8b --- /dev/null +++ b/meta/recipes-kernel/lttng/lttng-modules/0001-compaction-make-tracepoints-condtional-on-CONFIG_COM.patch | |||
| @@ -0,0 +1,103 @@ | |||
| 1 | From 417bb00022c813c6dc11bfa652d74f3687df7626 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Bruce Ashfield <bruce.ashfield@gmail.com> | ||
| 3 | Date: Wed, 3 Aug 2022 14:55:56 -0400 | ||
| 4 | Subject: [PATCH] compaction: make tracepoints condtional on CONFIG_COMPACTION | ||
| 5 | |||
| 6 | When building against a v5.19 mips kernel, the following errors | ||
| 7 | were triggered: | ||
| 8 | |||
| 9 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:105:42: note: expected 'struct compact_control *' but argument is of type 'struct compact_control *' | ||
| 10 | | 105 | TP_PROTO(struct compact_control *cc, | ||
| 11 | | | ~~~~~~~~~~~~~~~~~~~~~~~~^~ | ||
| 12 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:779:68: note: in definition of macro 'LTTNG_TRACEPOINT_EVENT_CLASS_CODE' | ||
| 13 | | 779 | static inline size_t __event_get_align__##_name(void *__tp_locvar, _proto) \ | ||
| 14 | | | ^~~~~~ | ||
| 15 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:106:50: note: in expansion of macro 'PARAMS' | ||
| 16 | | 106 | LTTNG_TRACEPOINT_EVENT_CLASS_CODE(_name, PARAMS(_proto), PARAMS(_args), , , \ | ||
| 17 | | | ^~~~~~ | ||
| 18 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:41:9: note: in expansion of macro 'LTTNG_TRACEPOINT_EVENT_CLASS' | ||
| 19 | | 41 | LTTNG_TRACEPOINT_EVENT_CLASS(map, \ | ||
| 20 | | | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
| 21 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:42:30: note: in expansion of macro 'PARAMS' | ||
| 22 | | 42 | PARAMS(proto), \ | ||
| 23 | | | ^~~~~~ | ||
| 24 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:101:1: note: in expansion of macro 'LTTNG_TRACEPOINT_EVENT_MAP' | ||
| 25 | | 101 | LTTNG_TRACEPOINT_EVENT_MAP(mm_compaction_migratepages, | ||
| 26 | | | ^~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
| 27 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:105:9: note: in expansion of macro 'TP_PROTO' | ||
| 28 | | 105 | TP_PROTO(struct compact_control *cc, | ||
| 29 | | | ^~~~~~~~ | ||
| 30 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:112:57: error: invalid use of undefined type 'struct compact_control' | ||
| 31 | |||
| 32 | ... | ||
| 33 | |||
| 34 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:108:25: warning: 'struct compact_control' declared inside parameter list will not be visible outside of this definition or declaration | ||
| 35 | | 108 | TP_PROTO(struct compact_control *cc, | ||
| 36 | | | ^~~~~~~~~~~~~~~ | ||
| 37 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:159:43: note: in definition of macro 'LTTNG_TRACEPOINT_EVENT_INSTANCE_MAP' | ||
| 38 | | 159 | void __event_template_proto___##_template(_proto); | ||
| 39 | | | ^~~~~~ | ||
| 40 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:45:61: note: in expansion of macro 'PARAMS' | ||
| 41 | | 45 | LTTNG_TRACEPOINT_EVENT_INSTANCE_MAP(map, name, map, PARAMS(proto), PARAMS(args)) | ||
| 42 | | | ^~~~~~ | ||
| 43 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:104:1: note: in expansion of macro 'LTTNG_TRACEPOINT_EVENT_MAP' | ||
| 44 | | 104 | LTTNG_TRACEPOINT_EVENT_MAP(mm_compaction_migratepages, | ||
| 45 | | | ^~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
| 46 | | lttng-modules-2.13.4/src/probes/../../include/instrumentation/events/compaction.h:108:9: note: in expansion of macro 'TP_PROTO' | ||
| 47 | | 108 | TP_PROTO(struct compact_control *cc, | ||
| 48 | | | ^~~~~~~~ | ||
| 49 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:159:6: error: conflicting types for '__event_template_proto___compaction_migratepages'; have 'void(struct compact_control *, unsigned int)' | ||
| 50 | | 159 | void __event_template_proto___##_template(_proto); | ||
| 51 | | | ^~~~~~~~~~~~~~~~~~~~~~~~~ | ||
| 52 | | lttng-modules-2.13.4/src/probes/../../include/lttng/tracepoint-event-impl.h:45:9: note: in expansion of macro 'LTTNG_TRACEPOINT_EVENT_INSTANCE_MAP' | ||
| 53 | | 45 | LTTNG_TRACEPOINT_EVENT_INSTANCE_MAP(map, name, map, PARAMS(proto), PARAMS(args)) | ||
| 54 | | | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
| 55 | |||
| 56 | The root cause appears to be that this particular mips kernel does not | ||
| 57 | have CONFIG_COMPACTION avaiable, and hence we end up with the struct | ||
| 58 | declaration within this tracepoint, and then conflicting types. | ||
| 59 | |||
| 60 | While putting the conditional around only compaction_migratepages seemed | ||
| 61 | sufficient to get the build working, it doesn't look like any of the | ||
| 62 | tracepoints should be valid, so we extend it to all definitions. | ||
| 63 | |||
| 64 | Upstream-Status: Submitted [https://lists.lttng.org/pipermail/lttng-dev/2022-August/030246.html] | ||
| 65 | |||
| 66 | Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com> | ||
| 67 | --- | ||
| 68 | include/instrumentation/events/compaction.h | 5 +++++ | ||
| 69 | 1 file changed, 5 insertions(+) | ||
| 70 | |||
| 71 | diff --git a/include/instrumentation/events/compaction.h b/include/instrumentation/events/compaction.h | ||
| 72 | index ecae39a..155804a 100644 | ||
| 73 | --- a/include/instrumentation/events/compaction.h | ||
| 74 | +++ b/include/instrumentation/events/compaction.h | ||
| 75 | @@ -9,6 +9,8 @@ | ||
| 76 | #include <linux/types.h> | ||
| 77 | #include <lttng/kernel-version.h> | ||
| 78 | |||
| 79 | +#ifdef CONFIG_COMPACTION | ||
| 80 | + | ||
| 81 | #if LTTNG_LINUX_VERSION_CODE >= LTTNG_KERNEL_VERSION(4,0,0) | ||
| 82 | |||
| 83 | LTTNG_TRACEPOINT_EVENT_CLASS(compaction_isolate_template, | ||
| 84 | @@ -97,6 +99,7 @@ LTTNG_TRACEPOINT_EVENT_INSTANCE_MAP(compaction_isolate_template, | ||
| 85 | |||
| 86 | #endif /* #else #if LTTNG_LINUX_VERSION_CODE >= LTTNG_KERNEL_VERSION(4,0,0) */ | ||
| 87 | |||
| 88 | + | ||
| 89 | #if (LTTNG_LINUX_VERSION_CODE >= LTTNG_KERNEL_VERSION(5,18,0)) | ||
| 90 | LTTNG_TRACEPOINT_EVENT_MAP(mm_compaction_migratepages, | ||
| 91 | |||
| 92 | @@ -182,6 +185,8 @@ LTTNG_TRACEPOINT_EVENT_MAP(mm_compaction_migratepages, | ||
| 93 | ) | ||
| 94 | #endif /* #else #if (LTTNG_LINUX_VERSION_CODE >= LTTNG_KERNEL_VERSION(3,16,0)) */ | ||
| 95 | |||
| 96 | +#endif /* CONFIG_COMPACTION */ | ||
| 97 | + | ||
| 98 | #endif /* LTTNG_TRACE_COMPACTION_H */ | ||
| 99 | |||
| 100 | /* This part must be outside protection */ | ||
| 101 | -- | ||
| 102 | 2.25.1 | ||
| 103 | |||
diff --git a/meta/recipes-kernel/lttng/lttng-modules_2.13.4.bb b/meta/recipes-kernel/lttng/lttng-modules_2.13.4.bb index fea0e383c9..a81a0e580f 100644 --- a/meta/recipes-kernel/lttng/lttng-modules_2.13.4.bb +++ b/meta/recipes-kernel/lttng/lttng-modules_2.13.4.bb | |||
| @@ -15,6 +15,7 @@ SRC_URI = "https://lttng.org/files/${BPN}/${BPN}-${PV}.tar.bz2 \ | |||
| 15 | file://0002-fix-fs-Remove-flags-parameter-from-aops-write_begin-.patch \ | 15 | file://0002-fix-fs-Remove-flags-parameter-from-aops-write_begin-.patch \ |
| 16 | file://0003-fix-workqueue-Fix-type-of-cpu-in-trace-event-v5.19.patch \ | 16 | file://0003-fix-workqueue-Fix-type-of-cpu-in-trace-event-v5.19.patch \ |
| 17 | file://0001-fix-net-skb-introduce-kfree_skb_reason-v5.15.58.v5.1.patch \ | 17 | file://0001-fix-net-skb-introduce-kfree_skb_reason-v5.15.58.v5.1.patch \ |
| 18 | file://0001-compaction-make-tracepoints-condtional-on-CONFIG_COM.patch \ | ||
| 18 | " | 19 | " |
| 19 | 20 | ||
| 20 | # Use :append here so that the patch is applied also when using devupstream | 21 | # Use :append here so that the patch is applied also when using devupstream |
