From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (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 E8C423E5566 for ; Wed, 1 Apr 2026 10:18:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775038716; cv=none; b=KWuhV7otoPalig6A8xjzTp5wC0vohuESgh/tR0b2f45CX92Gss8p9SC6JweZCQk78uilEE3t+6uzUzuxcP136DJzHWfY3AEZwzUZb1S79U1o6A695bcJJ4X3bsBf0z1Gha81UQ47GPZ/vdmbU+pdeWyJjuEDcixboB+oLvQcI3c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775038716; c=relaxed/simple; bh=Wz+rTxMXFXCcsbcqYERrHHqU3UlCN8/24cOvHBXTsX0=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=VQueP1HoQIIg+7nWyuBdcH2Bm7ymPuiNBmwiRIz/OPqFJ9cESVIOwlX2EfQBQVLAu2Rr6lOAsRg2QDt1lzVAkaRfO59n/+C3QHhBtiLYL1Po8ksIahQSEh0GHEqZylYYbpItImyldbfDLIQsJiwiLI69n5hrapZzfLSC+NxrVag= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=r7RgJ0lK; arc=none smtp.client-ip=209.85.128.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="r7RgJ0lK" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-48374014a77so80058715e9.3 for ; Wed, 01 Apr 2026 03:18:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775038706; x=1775643506; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:from:to:cc:subject:date:message-id:reply-to; bh=anYzjF7hGyuvz8IMGvRz0rbiAkEEPyOMLi1vC2Btujg=; b=r7RgJ0lK9peugos1A1ObMup3WsgNM1guHw208SoGiYJcGcfjsM0lNSlgYF4BRD+t/7 fNF91wMvZhyZVDgxXTzeIMhZTbr9PIw5C75w8E9Zh6DjC8zP4zc6NKUNJyFixAhjjcD/ fcOFmqMj9fjkHoDWN4XzJ7s8EofmHLgNEPUpiVoO+2P+iuTOWix9hbbJCf1Xxw86Avyl OavWJ3dA9VL8HcgfLD4320n1tKm7rouQtzgd2K4rdQjN9niAGpOnIIMqfXmHdG1xC/FN /pLopLzXc9eeYyr1YrtkSODujes8HFACGwCuK8P2QkD3mv+enQbsYhJvnQZHCB4Mx2cl +tvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775038706; x=1775643506; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=anYzjF7hGyuvz8IMGvRz0rbiAkEEPyOMLi1vC2Btujg=; b=gw/wS+AZ7rF03yyLVcER0k1t5tkaCMkSwBG4lPQY378VkBHx/rtI8i2Nrlk78cGxYy HzfRa0mVXwswq/f0X53oi/u5Zkc1uAszl6a1+jhCsDFUnRSEbGkEJfQRtc3d+XKpZX8Z HodYwK+J8uqx/bQB8JPRon6SAub5TkNTtK/oOk3cMJZGH4xuxoXgEqQtt+PFd10n2N8C Zu1tif9sZsE5PXn5RvUjznI1Vwq1VxPl2XH+8B1dafmSnRRVbYQRgCtm2YJHbNNCkW5k c0rpemmBh7YxvjXSK8TwXS8msFZyXqouQ+6H9CMNactXuJ2QNtJIvysi15dNXo2EdMNG LZhw== X-Forwarded-Encrypted: i=1; AJvYcCXFlIkKb0+er0f/wHJ5LMuQ5eOLM9HCIUls89t7RLZDZapj/xyr38DzzLV2Gmj935X2sFJtrtdR0ooHzMs=@vger.kernel.org X-Gm-Message-State: AOJu0Yxz2jcbBwfHU3lqXZ5XIEb/bGepjH17nTbXN2LUZ/ZarTRdSgwd 0U5/gvQ8eKboIELleQxIuPHIJtDaASC+rpmuHbC6qvcUAKIU943d9KeV X-Gm-Gg: ATEYQzyP3henZIUR4rfTw5meDB7857HC5Z+uUOH/m/YvvkBgSLhSJdYybjMOfoVUWSU wWpT1rd9tmrRkGC1hZ2rs6HrnqNa2AIl3ZdvfBTF7D2IIsEGE6qZpH7FPWzks63/HZSGCMk5hHc Ro+zrnsmawj5+iPr0bTGfp0rs0tKjTQ5J5GXZZSdjxoihOQWEqH222Wbvmg4T4gQBurScmrebYA madyj79Y1l/CI3cP/L1LLh8qoQbm6EHFI9Z7ozs63F5CDSU9PxY3pDj9yY+j0KutzmvVqLZcwOK sgOx/YbO78llJwcZt708Ffke2O52H/ibKWDXFC645UrhF7+iDb2lAIrmlB1ledvI56/Dqj4UMkh rZXVpEF2XFoTc+7OUAqgEFAJz5vMcXDb9HP8U8KYrujad0srJRnd0JEdMJKfqYP+wKKerp4tr X-Received: by 2002:a05:600c:3f19:b0:485:35a4:939f with SMTP id 5b1f17b1804b1-48883599371mr44116175e9.28.1775038705666; Wed, 01 Apr 2026 03:18:25 -0700 (PDT) Received: from krava ([2a02:8308:a00c:e200::d99c]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43cf21e9e18sm32503913f8f.9.2026.04.01.03.18.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Apr 2026 03:18:25 -0700 (PDT) From: Jiri Olsa X-Google-Original-From: Jiri Olsa Date: Wed, 1 Apr 2026 12:18:23 +0200 To: Hoyeon Lee Cc: bpf@vger.kernel.org, Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Martin KaFai Lau , Eduard Zingerman , Kumar Kartikeya Dwivedi , Song Liu , Yonghong Song , Shuah Khan , Feng Yang , linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH bpf-next v6 1/3] libbpf: use direct error codes for kprobe/uprobe attach Message-ID: References: <20260401070738.122709-1-hoyeon.lee@suse.com> <20260401070738.122709-2-hoyeon.lee@suse.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260401070738.122709-2-hoyeon.lee@suse.com> On Wed, Apr 01, 2026 at 04:05:02PM +0900, Hoyeon Lee wrote: > perf_event_open_probe() and perf_event_{k,u}probe_open_legacy() helpers > are returning negative error codes directly on failure. This commit > changes bpf_program__attach_{k,u}probe_opts() to use those return > values directly instead of reading -errno again. because the errno might be changed by the error path cleanup code, right? Acked-by: Jiri Olsa jirka > > Signed-off-by: Hoyeon Lee > --- > tools/lib/bpf/libbpf.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c > index 9ea41f40dc82..536c19c14d21 100644 > --- a/tools/lib/bpf/libbpf.c > +++ b/tools/lib/bpf/libbpf.c > @@ -11835,7 +11835,7 @@ bpf_program__attach_kprobe_opts(const struct bpf_program *prog, > offset, -1 /* pid */); > } > if (pfd < 0) { > - err = -errno; > + err = pfd; > pr_warn("prog '%s': failed to create %s '%s+0x%zx' perf event: %s\n", > prog->name, retprobe ? "kretprobe" : "kprobe", > func_name, offset, > @@ -12825,7 +12825,7 @@ bpf_program__attach_uprobe_opts(const struct bpf_program *prog, pid_t pid, > binary_path, func_offset, pid); > } > if (pfd < 0) { > - err = -errno; > + err = pfd; > pr_warn("prog '%s': failed to create %s '%s:0x%zx' perf event: %s\n", > prog->name, retprobe ? "uretprobe" : "uprobe", > binary_path, func_offset, > -- > 2.52.0 >