From nobody Mon Dec 1 22:03:45 2025 Received: from out-174.mta1.migadu.com (out-174.mta1.migadu.com [95.215.58.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4AE94302752 for ; Thu, 27 Nov 2025 18:53:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.174 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764269616; cv=none; b=UA2oalggUcZNQ/EHiULKH1MyiKCt7npNBZYbhZFgyiDpfWhPEthp5VPrKBdopcRRATsgXjq4GRn/kQmBWGciJ+S4qPw1cAvyRDu7nGHK38V2HiFhqEBbd62iHIKgxc5XCttYS0ysOChSUBiFdonVYzsAFlyInZp0tWiE1vkrnvM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764269616; c=relaxed/simple; bh=j5XXpTzzdMx3lJGLdsHVgHP8craYMQ5WOsTTtqlKWns=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=avXdAJZBXapZKZqS7iFBjfKk2Q6xOPpIdvKNzymBC9WGhg+qCFWmLpKXRdrgEjb5eFJf7uVMf2E9xlP58NjEjLggrpcpmsVfpp7ccpaJf2ZBC+9/G5Q9TbwefOWt9aNqrGfV+Za47Jf+GfWqE7Hx3/HjkJRyDoFpg6gut/d+Zk4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=ghYFeHis; arc=none smtp.client-ip=95.215.58.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="ghYFeHis" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1764269612; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=OmAGUFuRPKWCczXyIbvhxa8eaYDKpDkc1HqhWvoVOMI=; b=ghYFeHis+9VIIcviZuBskWl4tcsQ1HYmsLjxfi5DCplimNJw4uwiKJIQ9i5SM6oWAubzOQ 3Od4CLymOR3b0kh4KScYonkmST/Jy58Nib4KjkB8sf08QNzgkOeUx9l5UUdoWhLS3RY2CU eyFVGPWfHqIY8UKqqUBDSLB7cA6gTyc= From: Ihor Solodrai To: Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Martin KaFai Lau , Eduard Zingerman , Song Liu , Yonghong Song , John Fastabend , KP Singh , Stanislav Fomichev , Hao Luo , Jiri Olsa , Nathan Chancellor , Nicolas Schier , Nick Desaulniers , Bill Wendling , Justin Stitt , Alan Maguire , Donglin Peng Cc: bpf@vger.kernel.org, dwarves@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org Subject: [PATCH bpf-next v2 3/4] resolve_btfids: introduce enum btf_id_kind Date: Thu, 27 Nov 2025 10:52:41 -0800 Message-ID: <20251127185242.3954132-4-ihor.solodrai@linux.dev> In-Reply-To: <20251127185242.3954132-1-ihor.solodrai@linux.dev> References: <20251127185242.3954132-1-ihor.solodrai@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT Content-Type: text/plain; charset="utf-8" Instead of using multiple flags, make struct btf_id tagged with an enum value indicating its kind in the context of resolve_btfids. Signed-off-by: Ihor Solodrai --- tools/bpf/resolve_btfids/main.c | 62 ++++++++++++++++++++++----------- 1 file changed, 42 insertions(+), 20 deletions(-) diff --git a/tools/bpf/resolve_btfids/main.c b/tools/bpf/resolve_btfids/mai= n.c index b4caae1170dd..c60d303ca6ed 100644 --- a/tools/bpf/resolve_btfids/main.c +++ b/tools/bpf/resolve_btfids/main.c @@ -98,6 +98,13 @@ # error "Unknown machine endianness!" #endif =20 +enum btf_id_kind { + BTF_ID_KIND_NONE, + BTF_ID_KIND_SYM, + BTF_ID_KIND_SET, + BTF_ID_KIND_SET8 +}; + struct btf_id { struct rb_node rb_node; char *name; @@ -105,9 +112,8 @@ struct btf_id { int id; int cnt; }; - int addr_cnt; - bool is_set; - bool is_set8; + enum btf_id_kind kind:8; + int addr_cnt:8; Elf64_Addr addr[ADDR_CNT]; }; =20 @@ -260,26 +266,33 @@ static char *get_id(const char *prefix_end) return id; } =20 -static struct btf_id *add_set(struct object *obj, char *name, bool is_set8) +static struct btf_id *add_set(struct object *obj, char *name, enum btf_id_= kind kind) { /* * __BTF_ID__set__name * name =3D ^ * id =3D ^ */ - char *id =3D name + (is_set8 ? sizeof(BTF_SET8 "__") : sizeof(BTF_SET "__= ")) - 1; + int prefixlen =3D kind =3D=3D BTF_ID_KIND_SET8 ? sizeof(BTF_SET8 "__") : = sizeof(BTF_SET "__"); + char *id =3D name + prefixlen - 1; int len =3D strlen(name); + struct btf_id *btf_id; =20 if (id >=3D name + len) { pr_err("FAILED to parse set name: %s\n", name); return NULL; } =20 - return btf_id__add(&obj->sets, id, true); + btf_id =3D btf_id__add(&obj->sets, id, true); + if (btf_id) + btf_id->kind =3D kind; + + return btf_id; } =20 static struct btf_id *add_symbol(struct rb_root *root, char *name, size_t = size) { + struct btf_id *btf_id; char *id; =20 id =3D get_id(name + size); @@ -288,7 +301,11 @@ static struct btf_id *add_symbol(struct rb_root *root,= char *name, size_t size) return NULL; } =20 - return btf_id__add(root, id, false); + btf_id =3D btf_id__add(root, id, false); + if (btf_id) + btf_id->kind =3D BTF_ID_KIND_SYM; + + return btf_id; } =20 /* Older libelf.h and glibc elf.h might not yet define the ELF compression= types. */ @@ -491,28 +508,24 @@ static int symbols_collect(struct object *obj) id =3D add_symbol(&obj->funcs, prefix, sizeof(BTF_FUNC) - 1); /* set8 */ } else if (!strncmp(prefix, BTF_SET8, sizeof(BTF_SET8) - 1)) { - id =3D add_set(obj, prefix, true); + id =3D add_set(obj, prefix, BTF_ID_KIND_SET8); /* * SET8 objects store list's count, which is encoded * in symbol's size, together with 'cnt' field hence * that - 1. */ - if (id) { + if (id) id->cnt =3D sym.st_size / sizeof(uint64_t) - 1; - id->is_set8 =3D true; - } /* set */ } else if (!strncmp(prefix, BTF_SET, sizeof(BTF_SET) - 1)) { - id =3D add_set(obj, prefix, false); + id =3D add_set(obj, prefix, BTF_ID_KIND_SET); /* * SET objects store list's count, which is encoded * in symbol's size, together with 'cnt' field hence * that - 1. */ - if (id) { + if (id) id->cnt =3D sym.st_size / sizeof(int) - 1; - id->is_set =3D true; - } } else { pr_err("FAILED unsupported prefix %s\n", prefix); return -1; @@ -643,7 +656,7 @@ static int id_patch(struct object *obj, struct btf_id *= id) int i; =20 /* For set, set8, id->id may be 0 */ - if (!id->id && !id->is_set && !id->is_set8) { + if (!id->id && id->kind =3D=3D BTF_ID_KIND_SYM) { pr_err("WARN: resolve_btfids: unresolved symbol %s\n", id->name); warnings++; } @@ -696,6 +709,7 @@ static int sets_patch(struct object *obj) { Elf_Data *data =3D obj->efile.idlist; struct rb_node *next; + int cnt; =20 next =3D rb_first(&obj->sets); while (next) { @@ -715,11 +729,15 @@ static int sets_patch(struct object *obj) return -1; } =20 - if (id->is_set) { + switch (id->kind) { + case BTF_ID_KIND_SET: set =3D data->d_buf + off; + cnt =3D set->cnt; qsort(set->ids, set->cnt, sizeof(set->ids[0]), cmp_id); - } else { + break; + case BTF_ID_KIND_SET8: set8 =3D data->d_buf + off; + cnt =3D set8->cnt; /* * Make sure id is at the beginning of the pairs * struct, otherwise the below qsort would not work. @@ -744,10 +762,14 @@ static int sets_patch(struct object *obj) bswap_32(set8->pairs[i].flags); } } + break; + case BTF_ID_KIND_SYM: + default: + pr_err("Unexpected btf_id_kind %d for set '%s'\n", id->kind, id->name); + return -1; } =20 - pr_debug("sorting addr %5lu: cnt %6d [%s]\n", - off, id->is_set ? set->cnt : set8->cnt, id->name); + pr_debug("sorting addr %5lu: cnt %6d [%s]\n", off, cnt, id->name); =20 next =3D rb_next(next); } --=20 2.52.0