* [PATCH v2] perf/x86: Annotate struct bts_buffer with __counted_by()
@ 2025-03-05 12:31 Thorsten Blum
2025-03-05 17:39 ` [tip: perf/core] perf/x86: Annotate struct bts_buffer::buf " tip-bot2 for Thorsten Blum
0 siblings, 1 reply; 2+ messages in thread
From: Thorsten Blum @ 2025-03-05 12:31 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Ian Rogers, Adrian Hunter, Liang, Kan, Thomas Gleixner,
Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Kees Cook,
Gustavo A. R. Silva
Cc: Thorsten Blum, linux-perf-users, linux-kernel, linux-hardening
Add the __counted_by() compiler attribute to the flexible array member
buf to improve access bounds-checking via CONFIG_UBSAN_BOUNDS and
CONFIG_FORTIFY_SOURCE.
No functional changes intended.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
Changes in v2:
- struct_size() changes should be a separate patch as suggested by Ingo
- Link to v1: https://lore.kernel.org/r/20250304183056.78920-2-thorsten.blum@linux.dev/
---
arch/x86/events/intel/bts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/events/intel/bts.c b/arch/x86/events/intel/bts.c
index 8f78b0c900ef..c7be60c0b94f 100644
--- a/arch/x86/events/intel/bts.c
+++ b/arch/x86/events/intel/bts.c
@@ -58,7 +58,7 @@ struct bts_buffer {
local_t head;
unsigned long end;
void **data_pages;
- struct bts_phys buf[];
+ struct bts_phys buf[] __counted_by(nr_bufs);
};
static struct pmu bts_pmu;
--
2.48.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [tip: perf/core] perf/x86: Annotate struct bts_buffer::buf with __counted_by()
2025-03-05 12:31 [PATCH v2] perf/x86: Annotate struct bts_buffer with __counted_by() Thorsten Blum
@ 2025-03-05 17:39 ` tip-bot2 for Thorsten Blum
0 siblings, 0 replies; 2+ messages in thread
From: tip-bot2 for Thorsten Blum @ 2025-03-05 17:39 UTC (permalink / raw)
To: linux-tip-commits; +Cc: Thorsten Blum, Ingo Molnar, x86, linux-kernel
The following commit has been merged into the perf/core branch of tip:
Commit-ID: 5e7adc81ae1b27ff565714d2933b291cf1e1271f
Gitweb: https://git.kernel.org/tip/5e7adc81ae1b27ff565714d2933b291cf1e1271f
Author: Thorsten Blum <thorsten.blum@linux.dev>
AuthorDate: Wed, 05 Mar 2025 13:31:34 +01:00
Committer: Ingo Molnar <mingo@kernel.org>
CommitterDate: Wed, 05 Mar 2025 18:28:22 +01:00
perf/x86: Annotate struct bts_buffer::buf with __counted_by()
Add the __counted_by() compiler attribute to the flexible array member
buf to improve access bounds-checking via CONFIG_UBSAN_BOUNDS and
CONFIG_FORTIFY_SOURCE.
No functional changes intended.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20250305123134.215577-2-thorsten.blum@linux.dev
---
arch/x86/events/intel/bts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/events/intel/bts.c b/arch/x86/events/intel/bts.c
index 8e09319..953868d 100644
--- a/arch/x86/events/intel/bts.c
+++ b/arch/x86/events/intel/bts.c
@@ -58,7 +58,7 @@ struct bts_buffer {
local_t head;
unsigned long end;
void **data_pages;
- struct bts_phys buf[];
+ struct bts_phys buf[] __counted_by(nr_bufs);
};
static struct pmu bts_pmu;
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-03-05 17:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-05 12:31 [PATCH v2] perf/x86: Annotate struct bts_buffer with __counted_by() Thorsten Blum
2025-03-05 17:39 ` [tip: perf/core] perf/x86: Annotate struct bts_buffer::buf " tip-bot2 for Thorsten Blum
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox