From nobody Wed Dec 17 08:56:34 2025 Received: from mail-pj1-f74.google.com (mail-pj1-f74.google.com [209.85.216.74]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A39B41F4CAC for ; Thu, 20 Mar 2025 22:24:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.74 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742509486; cv=none; b=qC+z6SiYPSd0TtKz1ZU5urpVNk0B8b43jV3l8ijWUcOscDsdTaSX5y6085Htdpksjtge45svIGFpf8HhykVPhdYE+69M1XenK6yVsR1HV+NOhNbPiAeTUpMR5TvPZ6Zp2Qytxtba+mPlukk4507DRdHITBnfuu6uT6QoGh/ec8c= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742509486; c=relaxed/simple; bh=EBdmawavNviZH8o3V/8jqcpc3z8WzfTUJCXeSVJiEPE=; h=Date:Mime-Version:Message-ID:Subject:From:To:Cc:Content-Type; b=PW6p35yRPUjpMUacHFMOWYTJJ//tToh45QruUolCzpjTA7rNT/62C5zpnSrYoYw0RRO6kJo7grllfAfwwpKskifwyyAB/uYzCH3vXu+7j019BLpmLLDubbIfo7lYJqc819i0WRX3Ih8R8jY7I83UMZEAaNlCKFBzY0S12HhARhg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=XIIKRNEU; arc=none smtp.client-ip=209.85.216.74 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="XIIKRNEU" Received: by mail-pj1-f74.google.com with SMTP id 98e67ed59e1d1-2ff69646218so3073306a91.3 for ; Thu, 20 Mar 2025 15:24:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1742509484; x=1743114284; darn=vger.kernel.org; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=TTgK7Ngx2/T507+YgbCUhV8aPPXABYoFmzp+GmQKcz8=; b=XIIKRNEU/oNg4IQBqKQaUKfwWf5rITOVCQPZ61lEIG1krPuChMjE8zv1PrCtPp7S3a 3nQioT9uqO1lr+PDnQqPcx3SNG27iQZ8ZqNDVBhh9dzN6uLAcvvdCtpIs+ybLX5SXe4x 8t0XlnNXsnk75wzW2OC0Jhl5Ma/5mHIrfXCE1ZC34M2l0Cj3PPJmW3QLSSXr54b2T5pS /28bt+w3/msXqqjAeqp/coPzbJVmTAyfLMGGrNAkjzhDJkYMr28AigsodD7V5a1jQFf1 XYwQLG9Ql3H5eaaf6TcPfHEYEMKeelAuA8iN/JG3bG0GiyiOyJA8QrH+wDL6VfYGtjc+ 4epQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742509484; x=1743114284; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=TTgK7Ngx2/T507+YgbCUhV8aPPXABYoFmzp+GmQKcz8=; b=AgJkHhMtZY7L6iqcZBUkCFgKc9csdOY+Xz1zB/QzMljKx3aiQXP0TzdBTtMwLWItx9 eTsXqv9EaccE70W3dy7MNn8mkz/allY1YdkjzEqUxC0qEuAxEs+kb5GGpLjGbMOR59/6 6gX1OPQrCh3cws3E9xVzG0GY+ZFhHU81rLx3Mf9yPmfliSiBMMHYd6C+0tsClIaHsn7/ Tl2L39zowh526C7G2cYXNucDdloyrDj7mx4AYscw3+CO+cVbIJkxxUrHSA9tr5xZQg4V m+12cyEhkdkSAy5cUFCZSNrOIUPMC0mmpYTet+hd11rYoDnlcwt0UWUCAHUlItdpt19l 12HQ== X-Forwarded-Encrypted: i=1; AJvYcCXOf6T/reNiGMgtT+UJvv4pjKGCUU/CcmpAnWGSFoPEd1+NG47XH6xz9RbHhWMm7Sow9EVLLvzrEpkHsms=@vger.kernel.org X-Gm-Message-State: AOJu0YycRI4fjZYmgYoEio/MtTMkfbaOdoUjVBQly9Yj82/a83h8qetG C5tIlSoPIS+hdBDg+ZSHx2ajbBcvHpa/gFdjtk9/wrMqF3JpqZat8pr18pFru3nqyC0679J5NNS 2C41+AA== X-Google-Smtp-Source: AGHT+IE2Jm4eS4T9Ui9y1qLLey9xGKitHGZaGhB/U0tacIUCSXohkHLUjaUgny8wWNfuAFeHFkAz7vMEVaA/ X-Received: from pjk16.prod.google.com ([2002:a17:90b:5590:b0:2ea:29de:af10]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a17:90b:1f81:b0:2ee:b8ac:73b0 with SMTP id 98e67ed59e1d1-3030fe6df0amr1565280a91.2.1742509483864; Thu, 20 Mar 2025 15:24:43 -0700 (PDT) Date: Thu, 20 Mar 2025 15:24:39 -0700 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 X-Mailer: git-send-email 2.49.0.395.g12beb8f557-goog Message-ID: <20250320222439.1350187-1-irogers@google.com> Subject: [PATCH v2] libbpf: Add namespace for errstr making it libbpf_errstr From: Ian Rogers To: Andrii Nakryiko , Eduard Zingerman , Alexei Starovoitov , Daniel Borkmann , Martin KaFai Lau , Song Liu , Yonghong Song , John Fastabend , KP Singh , Stanislav Fomichev , Hao Luo , Jiri Olsa , Mykyta Yatsenko , bpf@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Ian Rogers Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When statically linking symbols can be replaced with those from other statically linked libraries depending on the link order and the hoped for "multiple definition" error may not appear. To avoid conflicts it is good practice to namespace symbols, this change renames errstr to libbpf_errstr. To avoid churn a #define is used to turn use of errstr(err) to libbpf_errstr(err). Fixes: 1633a83bf993 ("libbpf: Introduce errstr() for stringifying errno") Signed-off-by: Ian Rogers --- v2: Use #define to avoid churn renaming errstr as suggested by Andrii Nakryiko . v1: I feel like this patch shouldn't be strictly necessary, it turned out for a use-case it was and people who know better than me say the linker is working as intended. The conflicting errstr was from: https://sourceforge.net/projects/linuxquota/ The fixes tag may not be strictly necessary. --- tools/lib/bpf/str_error.c | 2 +- tools/lib/bpf/str_error.h | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/tools/lib/bpf/str_error.c b/tools/lib/bpf/str_error.c index 8743049e32b7..9a541762f54c 100644 --- a/tools/lib/bpf/str_error.c +++ b/tools/lib/bpf/str_error.c @@ -36,7 +36,7 @@ char *libbpf_strerror_r(int err, char *dst, int len) return dst; } =20 -const char *errstr(int err) +const char *libbpf_errstr(int err) { static __thread char buf[12]; =20 diff --git a/tools/lib/bpf/str_error.h b/tools/lib/bpf/str_error.h index 66ffebde0684..53e7fbffc13e 100644 --- a/tools/lib/bpf/str_error.h +++ b/tools/lib/bpf/str_error.h @@ -7,10 +7,13 @@ char *libbpf_strerror_r(int err, char *dst, int len); =20 /** - * @brief **errstr()** returns string corresponding to numeric errno + * @brief **libbpf_errstr()** returns string corresponding to numeric errno * @param err negative numeric errno * @return pointer to string representation of the errno, that is invalida= ted * upon the next call. */ -const char *errstr(int err); +const char *libbpf_errstr(int err); + +#define errstr(err) libbpf_errstr(err) + #endif /* __LIBBPF_STR_ERROR_H */ --=20 2.49.0.395.g12beb8f557-goog