From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f49.google.com (mail-ej1-f49.google.com [209.85.218.49]) (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 B0BAE3CE4BA for ; Fri, 15 May 2026 11:12:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778843556; cv=none; b=qboo3rihR+o3I4Hj/6jHJMvFvzEpKiVfQECAFm0NtKVdVKJJU4ah//ixfQr/rC280lPyPV3Xfom8v+A/G25rwQmfhcy9qTMlDfP2Zb9piV6D0WpQso4MR3Fi29C68QmhEhXBuW/bAEDJ+gSsLpNiupTVtvB/Mc8saz6e2FwE7OU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778843556; c=relaxed/simple; bh=07S8qGhUlGHHZ+Z7kKrI+FqnQKBLMTTGSDN2n79cGXo=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=Sv3t3JsPJZQ8abNxo0lciP19JXHulHkHzR6mW5VvuA+WkKzxRD5oQFH1zdxhcchSe9rtZic8zOdkAVXFhRBQe3atzzNke0CVzbL6tuX0p/W/MX4JFBM5Qp4VQGCP48fyGWw5d9gHYtqDk9WhSNmwQz6cAsl5BISGPrBwalGvT5M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cloudflare.com; spf=pass smtp.mailfrom=cloudflare.com; dkim=pass (2048-bit key) header.d=cloudflare.com header.i=@cloudflare.com header.b=SO+yqBMj; arc=none smtp.client-ip=209.85.218.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cloudflare.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cloudflare.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=cloudflare.com header.i=@cloudflare.com header.b="SO+yqBMj" Received: by mail-ej1-f49.google.com with SMTP id a640c23a62f3a-b8f9568e074so1447241666b.0 for ; Fri, 15 May 2026 04:12:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cloudflare.com; s=google09082023; t=1778843553; x=1779448353; darn=vger.kernel.org; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=77xVpw+sT9Jy+PbMBmIBkrl0llFT6NHnPb++9YCl0NE=; b=SO+yqBMjbSgATYtqA+vV6rcCEBW84KWwMc1WzrTsjdLnH8emmQDVu9b6xF8URfsQTB diAv34V2Twlk/vAvmVBkQQGSy3fM/CzuEfu94E1l2BZTf0DeggbZZJfQIQUZWYOhokny v3QPegEel69+M2ijddl1DkcvbXFLJeGrhq0pXB7I26aHBkJdm7TT7RKTbSqjRNHyEqTL y7Qu+cKF3zeiFHpE9VZTQ6rKbg1SlR4E0WcT6vj6NyHtMi2FMjoxTrlOeDoayi0R/nES r9oYa+XCXkc7nym0Bxd9pFJVeBHK+uPjSqPrQfdRSti8Z1AfgLcj7J9V14SEEI5l5xv7 DGug== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778843553; x=1779448353; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=77xVpw+sT9Jy+PbMBmIBkrl0llFT6NHnPb++9YCl0NE=; b=n0OPPI5VAzt1pBjLOdkgIOZnKd5LYohs7alTWImJVZqq5u+jZQq+xBGkcHKlRL6L7R LlfEf6S7KzAaBYw/szOPigP1zayUBAZ+Ikl6iZvywuA2Tonf/2Gz42RdtX8ECOy3xnPO elfV+XPLeDTZ3+ZEUmZBS4sUP3EvtTPe/PF/Zt7WMFR6wa3ndSNgCgJoim/+bQu/u2I6 4neqFADna6ZPISMrLaHcQC9XY4QiZ6ckJY2sgbXH4tytKyVyspiH17d0pupLfq56Fesf f2Z5HO0v2WwOu3Zvddp+znY9jqT9jDEI8hR1rY+5r7exCKsPdlBFClNtFGEoRQqAXQpg codg== X-Forwarded-Encrypted: i=1; AFNElJ/KRq5AkQ0W018zg0LXLSSqDVgkyolDC/OU9THpWlat1RJmkpg+vSSJ6SX1QXxu21Ap0Eq/vM8WI8VSjfiMWc880Ww=@vger.kernel.org X-Gm-Message-State: AOJu0YyifbnuL9uaBP9jjtZSABCHwqCiCIDMjKttoiHH1VyX72JFmlhX DevWkapg6Xke8Lf/6qwGs1Rr8/0n9SrO5lkHeNeSxPALMHguEW451THKRzTL68q7Z+E= X-Gm-Gg: Acq92OHSla6F8dn7+wSQtjha4D3Wh24YtHNumAPmp/Hkdv0ppIbet1FGJghYCRyGlF1 6Tu1k7AopzpYcQ919qAEf//nrjTjcPZpgwABQ2j3/P3P506dmN3XL0S03ekiKV7mYxM+2JdDcSc uWb5SHlfGsZGtyadWDSRKqkPv+pJdYuF/7TpQCdkB+VD95K3jZPc3qwhw0mWs4V0IZyqV+QEsM9 U+pNKUL+NA/p0C1E7w5w39+zJ+ZD3nNOJPrMCAXxJXjv2FV1YdSMz6LJWEVsRf4CfAJGZqPExQ7 nx7GmktCmkeHFsfY7MZzk3GPPWWf5HnuGxewUSunB3FVFAERkDvba8mtNlwEcjwntqyMMloqIzg 38q9zyWpV0br8U4ybBIraKdnNhEFm7nDBQb4u12nDgnumLntz+bRbD1XnHz+G1aBVj0wxKIUhY4 gRnvjKYbvSBwW9LfRxnnzV6qieEQ== X-Received: by 2002:a17:907:94d2:b0:bd5:2e64:aeee with SMTP id a640c23a62f3a-bd52e64b4b4mr8208666b.35.1778843552408; Fri, 15 May 2026 04:12:32 -0700 (PDT) Received: from cloudflare.com ([104.28.21.182]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-68310b3e8e5sm1885489a12.4.2026.05.15.04.12.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 May 2026 04:12:31 -0700 (PDT) From: Jakub Sitnicki To: Jiri Olsa Cc: Oleg Nesterov , Peter Zijlstra , Ingo Molnar , Masami Hiramatsu , Andrii Nakryiko , bpf@vger.kernel.org, linux-trace-kernel@vger.kernel.org Subject: Re: [PATCH 2/7] libbpf: Change has_nop_combo to work on top of nop10 In-Reply-To: <20260514135342.22130-3-jolsa@kernel.org> (Jiri Olsa's message of "Thu, 14 May 2026 15:53:37 +0200") References: <20260514135342.22130-1-jolsa@kernel.org> <20260514135342.22130-3-jolsa@kernel.org> Date: Fri, 15 May 2026 13:12:29 +0200 Message-ID: <87fr3t7wpu.fsf@cloudflare.com> Precedence: bulk X-Mailing-List: linux-trace-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain On Thu, May 14, 2026 at 03:53 PM +02, Jiri Olsa wrote: > We now expect nop combo with 10 bytes nop instead of 5 bytes nop, > fixing has_nop_combo to reflect that. > > Signed-off-by: Jiri Olsa > --- > tools/lib/bpf/usdt.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/tools/lib/bpf/usdt.c b/tools/lib/bpf/usdt.c > index e3710933fd52..7e62e4d5bedd 100644 > --- a/tools/lib/bpf/usdt.c > +++ b/tools/lib/bpf/usdt.c > @@ -305,7 +305,7 @@ struct usdt_manager *usdt_manager_new(struct bpf_object *obj) > > /* > * Detect kernel support for uprobe() syscall, it's presence means we can > - * take advantage of faster nop5 uprobe handling. > + * take advantage of faster nop10 uprobe handling. > * Added in: 56101b69c919 ("uprobes/x86: Add uprobe syscall to speed up uprobe") > */ > man->has_uprobe_syscall = kernel_supports(obj, FEAT_UPROBE_SYSCALL); > @@ -596,14 +596,14 @@ static int parse_usdt_spec(struct usdt_spec *spec, const struct usdt_note *note, > #if defined(__x86_64__) > static bool has_nop_combo(int fd, long off) > { > - unsigned char nop_combo[6] = { > - 0x90, 0x0f, 0x1f, 0x44, 0x00, 0x00 /* nop,nop5 */ > + unsigned char nop_combo[11] = { > + 0x90, 0x66, 0x66, 0x0f, 0x1f, 0x84, 0x00, 0x00, 0x00, 0x00, 0x00, > }; > - unsigned char buf[6]; > + unsigned char buf[11]; > > - if (pread(fd, buf, 6, off) != 6) > + if (pread(fd, buf, 11, off) != 11) > return false; > - return memcmp(buf, nop_combo, 6) == 0; > + return memcmp(buf, nop_combo, 11) == 0; > } > #else > static bool has_nop_combo(int fd, long off) Nit: Would use ARRAY_SIZE(buf) instead of repeating the scalar value in multiple places. Otherwise: Reviewed-by: Jakub Sitnicki