From: Donglin Peng <dolinux.peng@gmail.com>
To: rostedt@goodmis.org, ast@kernel.org
Cc: mhiramat@kernel.org, andrii.nakryiko@gmail.com,
linux-kernel@vger.kernel.org,
Donglin Peng <pengdonglin@xiaomi.com>,
Andrii Nakryiko <andrii@kernel.org>,
Eduard Zingerman <eddyz87@gmail.com>,
bpf@vger.kernel.org
Subject: [PATCH 1/2] btf: Add for_each_enum and for_each_enum64 helper macros
Date: Mon, 2 Feb 2026 19:15:47 +0800 [thread overview]
Message-ID: <20260202111548.3555306-2-dolinux.peng@gmail.com> (raw)
In-Reply-To: <20260202111548.3555306-1-dolinux.peng@gmail.com>
From: Donglin Peng <pengdonglin@xiaomi.com>
Introduce two new helper macros, for_each_enum and for_each_enum64,
to clean up code.
No functional changes.
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Andrii Nakryiko <andrii@kernel.org>
Cc: Eduard Zingerman <eddyz87@gmail.com>
Cc: bpf@vger.kernel.org
Signed-off-by: Donglin Peng <pengdonglin@xiaomi.com>
---
include/linux/btf.h | 10 +++++++
kernel/bpf/btf.c | 66 +++++++++++++++++++--------------------------
2 files changed, 38 insertions(+), 38 deletions(-)
diff --git a/include/linux/btf.h b/include/linux/btf.h
index f06976ffb63f..cde2deb8e25e 100644
--- a/include/linux/btf.h
+++ b/include/linux/btf.h
@@ -254,6 +254,16 @@ const char *btf_type_str(const struct btf_type *t);
i < btf_type_vlen(datasec_type); \
i++, member++)
+#define for_each_enum(i, enum_type, member) \
+ for (i = 0, member = btf_enum(enum_type); \
+ i < btf_type_vlen(enum_type); \
+ i++, member++)
+
+#define for_each_enum64(i, enum_type, member) \
+ for (i = 0, member = btf_enum64(enum_type); \
+ i < btf_type_vlen(enum_type); \
+ i++, member++)
+
static inline bool btf_type_is_ptr(const struct btf_type *t)
{
return BTF_INFO_KIND(t->info) == BTF_KIND_PTR;
diff --git a/kernel/bpf/btf.c b/kernel/bpf/btf.c
index 0de8fc8a0e0b..c80a7a0cbe1d 100644
--- a/kernel/bpf/btf.c
+++ b/kernel/bpf/btf.c
@@ -728,11 +728,6 @@ static const struct btf_array *btf_type_array(const struct btf_type *t)
return (const struct btf_array *)(t + 1);
}
-static const struct btf_enum *btf_type_enum(const struct btf_type *t)
-{
- return (const struct btf_enum *)(t + 1);
-}
-
static const struct btf_var *btf_type_var(const struct btf_type *t)
{
return (const struct btf_var *)(t + 1);
@@ -743,11 +738,6 @@ static const struct btf_decl_tag *btf_type_decl_tag(const struct btf_type *t)
return (const struct btf_decl_tag *)(t + 1);
}
-static const struct btf_enum64 *btf_type_enum64(const struct btf_type *t)
-{
- return (const struct btf_enum64 *)(t + 1);
-}
-
static const struct btf_kind_operations *btf_type_ops(const struct btf_type *t)
{
return kind_ops[BTF_INFO_KIND(t->info)];
@@ -4317,14 +4307,14 @@ static s32 btf_enum_check_meta(struct btf_verifier_env *env,
const struct btf_type *t,
u32 meta_left)
{
- const struct btf_enum *enums = btf_type_enum(t);
+ const struct btf_enum *enump;
struct btf *btf = env->btf;
const char *fmt_str;
u16 i, nr_enums;
u32 meta_needed;
nr_enums = btf_type_vlen(t);
- meta_needed = nr_enums * sizeof(*enums);
+ meta_needed = nr_enums * sizeof(*enump);
if (meta_left < meta_needed) {
btf_verifier_log_basic(env, t,
@@ -4347,16 +4337,16 @@ static s32 btf_enum_check_meta(struct btf_verifier_env *env,
btf_verifier_log_type(env, t, NULL);
- for (i = 0; i < nr_enums; i++) {
- if (!btf_name_offset_valid(btf, enums[i].name_off)) {
+ for_each_enum(i, t, enump) {
+ if (!btf_name_offset_valid(btf, enump->name_off)) {
btf_verifier_log(env, "\tInvalid name_offset:%u",
- enums[i].name_off);
+ enump->name_off);
return -EINVAL;
}
/* enum member must have a valid name */
- if (!enums[i].name_off ||
- !btf_name_valid_identifier(btf, enums[i].name_off)) {
+ if (!enump->name_off ||
+ !btf_name_valid_identifier(btf, enump->name_off)) {
btf_verifier_log_type(env, t, "Invalid name");
return -EINVAL;
}
@@ -4365,8 +4355,8 @@ static s32 btf_enum_check_meta(struct btf_verifier_env *env,
continue;
fmt_str = btf_type_kflag(t) ? "\t%s val=%d\n" : "\t%s val=%u\n";
btf_verifier_log(env, fmt_str,
- __btf_name_by_offset(btf, enums[i].name_off),
- enums[i].val);
+ __btf_name_by_offset(btf, enump->name_off),
+ enump->val);
}
return meta_needed;
@@ -4382,8 +4372,8 @@ static void btf_enum_show(const struct btf *btf, const struct btf_type *t,
u32 type_id, void *data, u8 bits_offset,
struct btf_show *show)
{
- const struct btf_enum *enums = btf_type_enum(t);
- u32 i, nr_enums = btf_type_vlen(t);
+ const struct btf_enum *enump;
+ u32 i;
void *safe_data;
int v;
@@ -4393,13 +4383,13 @@ static void btf_enum_show(const struct btf *btf, const struct btf_type *t,
v = *(int *)safe_data;
- for (i = 0; i < nr_enums; i++) {
- if (v != enums[i].val)
+ for_each_enum(i, t, enump) {
+ if (v != enump->val)
continue;
btf_show_type_value(show, "%s",
__btf_name_by_offset(btf,
- enums[i].name_off));
+ enump->name_off));
btf_show_end_type(show);
return;
@@ -4425,14 +4415,14 @@ static s32 btf_enum64_check_meta(struct btf_verifier_env *env,
const struct btf_type *t,
u32 meta_left)
{
- const struct btf_enum64 *enums = btf_type_enum64(t);
+ const struct btf_enum64 *enump;
struct btf *btf = env->btf;
const char *fmt_str;
u16 i, nr_enums;
u32 meta_needed;
nr_enums = btf_type_vlen(t);
- meta_needed = nr_enums * sizeof(*enums);
+ meta_needed = nr_enums * sizeof(*enump);
if (meta_left < meta_needed) {
btf_verifier_log_basic(env, t,
@@ -4455,16 +4445,16 @@ static s32 btf_enum64_check_meta(struct btf_verifier_env *env,
btf_verifier_log_type(env, t, NULL);
- for (i = 0; i < nr_enums; i++) {
- if (!btf_name_offset_valid(btf, enums[i].name_off)) {
+ for_each_enum64(i, t, enump) {
+ if (!btf_name_offset_valid(btf, enump->name_off)) {
btf_verifier_log(env, "\tInvalid name_offset:%u",
- enums[i].name_off);
+ enump->name_off);
return -EINVAL;
}
/* enum member must have a valid name */
- if (!enums[i].name_off ||
- !btf_name_valid_identifier(btf, enums[i].name_off)) {
+ if (!enump->name_off ||
+ !btf_name_valid_identifier(btf, enump->name_off)) {
btf_verifier_log_type(env, t, "Invalid name");
return -EINVAL;
}
@@ -4474,8 +4464,8 @@ static s32 btf_enum64_check_meta(struct btf_verifier_env *env,
fmt_str = btf_type_kflag(t) ? "\t%s val=%lld\n" : "\t%s val=%llu\n";
btf_verifier_log(env, fmt_str,
- __btf_name_by_offset(btf, enums[i].name_off),
- btf_enum64_value(enums + i));
+ __btf_name_by_offset(btf, enump->name_off),
+ btf_enum64_value(enump));
}
return meta_needed;
@@ -4485,8 +4475,8 @@ static void btf_enum64_show(const struct btf *btf, const struct btf_type *t,
u32 type_id, void *data, u8 bits_offset,
struct btf_show *show)
{
- const struct btf_enum64 *enums = btf_type_enum64(t);
- u32 i, nr_enums = btf_type_vlen(t);
+ const struct btf_enum64 *enump;
+ u32 i;
void *safe_data;
s64 v;
@@ -4496,13 +4486,13 @@ static void btf_enum64_show(const struct btf *btf, const struct btf_type *t,
v = *(u64 *)safe_data;
- for (i = 0; i < nr_enums; i++) {
- if (v != btf_enum64_value(enums + i))
+ for_each_enum64(i, t, enump) {
+ if (v != btf_enum64_value(enump))
continue;
btf_show_type_value(show, "%s",
__btf_name_by_offset(btf,
- enums[i].name_off));
+ enump->name_off));
btf_show_end_type(show);
return;
--
2.34.1
next prev parent reply other threads:[~2026-02-02 11:15 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-02 11:15 [PATCH 0/2] tracing: resolve enum names for function arguments via BTF Donglin Peng
2026-02-02 11:15 ` Donglin Peng [this message]
2026-02-06 0:29 ` [PATCH 1/2] btf: Add for_each_enum and for_each_enum64 helper macros Masami Hiramatsu
2026-02-02 11:15 ` [PATCH 2/2] tracing: resolve enum names for function arguments via BTF Donglin Peng
2026-02-02 16:12 ` Steven Rostedt
2026-02-03 2:17 ` Donglin Peng
2026-02-03 13:50 ` Donglin Peng
2026-02-03 15:17 ` Steven Rostedt
2026-02-03 16:00 ` Alexei Starovoitov
2026-02-04 14:52 ` Donglin Peng
2026-02-05 9:21 ` Donglin Peng
2026-02-05 15:56 ` Alexei Starovoitov
2026-02-06 4:09 ` Donglin Peng
2026-02-06 16:04 ` Alexei Starovoitov
2026-02-08 13:08 ` Donglin Peng
2026-02-05 18:04 ` Steven Rostedt
2026-02-06 4:04 ` Donglin Peng
2026-02-08 13:07 ` Donglin Peng
2026-02-08 15:27 ` Steven Rostedt
2026-02-08 15:42 ` Donglin Peng
2026-02-08 16:47 ` Steven Rostedt
2026-02-04 14:16 ` Donglin Peng
2026-02-06 0:12 ` Masami Hiramatsu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260202111548.3555306-2-dolinux.peng@gmail.com \
--to=dolinux.peng@gmail.com \
--cc=andrii.nakryiko@gmail.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=eddyz87@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mhiramat@kernel.org \
--cc=pengdonglin@xiaomi.com \
--cc=rostedt@goodmis.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®