diff options
| author |   Markus Schneider-Pargmann (The Capable Hub) <msp@baylibre.com> | 2026-04-28 10:30:29 +0200 |
|---|---|---|
| committer |   Masami Hiramatsu (Google) <mhiramat@kernel.org> | 2026-07-14 22:43:39 +0900 |
| commit | b20c8177092f3ff4804e0271e3fd338d34e09097 (patch) | |
| tree | c3ba2542463ff269b330dca674451d926a2c88f1 | |
| parent | tracing/probes: Add a new testcase for BTF typecasts (diff) | |
tracing: fprobe: Remove __packed from generic __fprobe_header
fp pointer and unsigned long have the same size on all relevant
architectures that build Linux. Furthermore this struct is only used in
architectures that do not set ARCH_DEFINE_ENCODE_FPROBE_HEADER which is
set only for 64bit architectures (apart from LoongArch).
Both fields are aligned on these architectures so the struct with
__packed and without it are the same.
Remove the __packed as it is unnecessary.
[Masami: Removed Fixes tag because this is not fixing any problem.]
Link: https://lore.kernel.org/all/20260428-topic-fprobe-packed-v7-1-v1-1-9abc9b866b4c@baylibre.com/
Signed-off-by: Markus Schneider-Pargmann (The Capable Hub) <msp@baylibre.com>
Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
| -rw-r--r-- | kernel/trace/fprobe.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/fprobe.c b/kernel/trace/fprobe.c index f215990b9061..2727f8861fa3 100644 --- a/kernel/trace/fprobe.c +++ b/kernel/trace/fprobe.c @@ -181,7 +181,7 @@ static inline void read_fprobe_header(unsigned long *stack, struct __fprobe_header { struct fprobe *fp; unsigned long size_words; -} __packed; +}; #define FPROBE_HEADER_SIZE_IN_LONG SIZE_IN_LONG(sizeof(struct __fprobe_header)) |