From nobody Tue Dec 2 00:00:39 2025 Received: from out-170.mta0.migadu.com (out-170.mta0.migadu.com [91.218.175.170]) (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 544CF2FB602 for ; Wed, 26 Nov 2025 01:29:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.170 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764120587; cv=none; b=iAdNq1q28MkY67vdr0sdDrDQHnXzNyAmdyRqdqGEEC4wZbtWn2BBH5JJg4yos8j8VPwndX0KPLfo5WulEHUzUAp9nR8xV+y5dsb7ahpqXiIU/Xs7Hjhq/mCqGWFXKdItw9EEBPaEskm84nz31mbSbF3mMI2CKphUx2azU3+DESE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764120587; c=relaxed/simple; bh=xCvFs1xbcVN6h+Vy2tQsGp0ZQAuYsMF+spiFOaf/Irg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=U/A/G+e/uRv7f2hTOr+AfhKZQorm/ryw/cCoTLsCQ1zyCp6PTYF5Fpl1yaxXOLE+UWsfFHLhI4HRQDBdX/svT1ifewzNojodDDlJdU/x+yB5ymIhAddqjM10ivo+zzmhXRY9ArQ8Z2BOBN4X/GwVnx4Kj6yhwGeNRCMBqKt3khM= 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=ozOqoMmw; arc=none smtp.client-ip=91.218.175.170 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="ozOqoMmw" 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=1764120583; 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=8OxpVlu97uQOGmIsABXF9JpduqMEidWW+QhnLTm9npg=; b=ozOqoMmwYEy7W1BpWVntj+K2UBfM18cylAmrgcVy5bN2TCF+A6jmJZ/GfdoJ7MSxh5s7QG Mh3Ssvs5JU9yzDE+uTsJj1SHogTx+3YHwb39wkmPpJTn0cSEzn0+VpTZT91ip94Lo6GgMl ipeh+NywOo7JuYy7x6MWBJIYgxG2etE= 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 Cc: bpf@vger.kernel.org, dwarves@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org, Alan Maguire , Donglin Peng Subject: [PATCH bpf-next v1 3/4] resolve_btfids: introduce enum btf_id_kind Date: Tue, 25 Nov 2025 17:26:55 -0800 Message-ID: <20251126012656.3546071-4-ihor.solodrai@linux.dev> In-Reply-To: <20251126012656.3546071-1-ihor.solodrai@linux.dev> References: <20251126012656.3546071-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 | 61 ++++++++++++++++++++++----------- 1 file changed, 41 insertions(+), 20 deletions(-) diff --git a/tools/bpf/resolve_btfids/main.c b/tools/bpf/resolve_btfids/mai= n.c index b7b44e72e765..7f5a9f7dde7f 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,10 @@ 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); + 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 +507,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 +655,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 +708,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 +728,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 +761,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