From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (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 F3BCF256D for ; Sat, 21 Feb 2026 12:00:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771675240; cv=none; b=mcR/8AYdGWM7yZM/JxgAY69nc+lDXExrEfVtLfOHxNIVUUYaXfj7cEpkyHh9uTnCW2POwM4DZ8Smd5LlVoW0BKgffOGNYenZ4XTU7Sgn9h0W/hU+tUGN5DRRBS+fF+chHi3eIlYlWE+KlX2ss4wESkeZPyOJJzOOuUN/nfD0AvE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771675240; c=relaxed/simple; bh=RHjWx0X9Hlmy0k5PHSWWRSn4r0L4sFycfYFf/yMVnnk=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=WCJbVj1BViauYjerxwwzr5UqRMdlGkfcnlcYEVrIg3waq/38qt2VdTMJy1olY3dzJtRrdmo+HEUoIzWY0fVB9wnUAzKAUjNlI9I3XLq9ZmZis41GqRzymRDc2ahMpjt/6LBsPI1x2qveQLeGLoPMwCtFjTecQil4V4mKiXhwnKk= 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=dKuhDNd6; arc=none smtp.client-ip=209.85.128.46 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="dKuhDNd6" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-4836d4c26d3so25934515e9.2 for ; Sat, 21 Feb 2026 04:00:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1771675237; x=1772280037; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=gtUgp0+6jkXn56WgZsr0JG4gU+/EJFHQjq7Gj5fUqxo=; b=dKuhDNd60zt8eBQtDl6sNqw7j7q/+cWq20A96PzNv1isd65SbxhNWCLnIUPDLVQMTU zFsI+OAXXUF4cQyoB35Ngmu47vRSn6NLNhzS0obSVeGSR3Tm//KFhEXLYbVcPVray4ML ww27PBF9idkqrgNwSDQsPHP3Smhllo7dXeq3y5IFk0N0fzlPc5rxJVEy/lrCGoRfb1iZ iVZXTZIdb2JjUuXIdRA5QN201JNqnhvLGvd87Uc6/QcGetxREOW7P7OggOcRNPHr0TdE Tsur1oYfjq7fSO0mOxfW/Kc75w55yNGk7l6ATdEAnmf9kxthL49sNcqhK4Ut+rgoSufd 12bw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1771675237; x=1772280037; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=gtUgp0+6jkXn56WgZsr0JG4gU+/EJFHQjq7Gj5fUqxo=; b=t90bdoumIEjrOIjzPfQlhuES1VEp72sETcgbZXdGGD9HGx++tWddgbi5Acm539aVKf 4o0kTgU6/ahtM0OsdcChpOYWqeMEbkurb+E4EPWuKqzTQbAP4H8O5nfR7rgNqIOBIRaT hVJnCFlCs4EQixuHGfN+PPHBd7W0mYx+RXkAP346H/aOLumFLVhRzF3+ZqUGFLRfIHQh g77iomvE1CwUTiDuGUatNPyOmc3uRiz1KUU/QFGIsxMqh9ICL36btlKXYwqNgSm7TnAq ge0ltDw3P2vXIdeBpuTlub7w5KWQJT9y84feCQ5QW6wZLMNXRA8X96UVJgln5mB9Z3wm TzSA== X-Forwarded-Encrypted: i=1; AJvYcCWBdsq+QXEwlHGPoN31u+56a0zb4m729+euKXOu7RQQmAX2gjkXxEjaYINgZa9nndsb7NTw22bS2Z4MypU=@vger.kernel.org X-Gm-Message-State: AOJu0YwAb8k9U8CAdK0mNUKVL431n+VKpy1FK+O8DE6mEknvYCcHNzNh iyj3kevETXwo/q+ugTHWS2MEvpZdvAE0QrlsxDKg+3UgX9ZlGQr0sDZX X-Gm-Gg: AZuq6aKWFDyPa6kaQ0qI71BZpVZ6efbTdYphYsup3IkDtCMc5NH12/RxwGp4xNJiVbi HJKdQ742i9w5nxOXuLFfnZIpFKLFw9jy9F8cpJGAa7o1qZKg7JGUl6TnLb/8s56Pv1pJQp7Gt8T WpgpR+uWKw14U0siPdSvUIk3DZtuQc3tACSRxj3vGfAAKK6PNMLhZQIvutX6/DbxhsdbrBN/yXF NzE/a3WRJD1KGIOLuGhGW+EEIyDEYv2PKuT4cuoiQJ9KGWwPOiqLfhC3roRms2woswyOd7C42+g qZq321sCE3KdJWqKmYa3xp60ht0f+NsR6U3XDRpVBG5YUyUm3q9BcAlbsrWUiAFLadIqjCfGPrs fWZEEUCrlaWWHFmWmHdKSUBuMMUjaTBmADeDkMk+oCAhTNvtnkppUnaT1EuWOJsOtl5kdZxsnL0 FA8Xsi8XK0jGGiUOe8vS8gxdAD7VVFpoCXjpImXbxanLOf1fUY9OGe4XKtf78U7Vi/ X-Received: by 2002:a05:600c:45ce:b0:477:93f7:bbc5 with SMTP id 5b1f17b1804b1-483a95b5835mr42331815e9.10.1771675237088; Sat, 21 Feb 2026 04:00:37 -0800 (PST) Received: from pumpkin (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-483a3dfd902sm39760875e9.7.2026.02.21.04.00.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 21 Feb 2026 04:00:36 -0800 (PST) Date: Sat, 21 Feb 2026 12:00:35 +0000 From: David Laight To: bot+bpf-ci@kernel.org Cc: jolsa@kernel.org, andrii@kernel.org, bpf@vger.kernel.org, linux-kernel@vger.kernel.org, songliubraving@fb.com, yhs@fb.com, john.fastabend@gmail.com, ast@kernel.org, daniel@iogearbox.net, martin.lau@kernel.org, eddyz87@gmail.com, yonghong.song@linux.dev, clm@meta.com, ihor.solodrai@linux.dev Subject: Re: [PATCHv4 bpf-next 2/5] libbpf: Add support to detect nop,nop5 instructions combo for usdt probe Message-ID: <20260221120035.4562df85@pumpkin> In-Reply-To: References: <20260220104220.634154-3-jolsa@kernel.org> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) 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-Transfer-Encoding: 7bit On Fri, 20 Feb 2026 11:24:45 +0000 (UTC) bot+bpf-ci@kernel.org wrote: > > diff --git a/tools/lib/bpf/usdt.c b/tools/lib/bpf/usdt.c > > index d1524f6f54ae..c81e0a58caf5 100644 > > --- a/tools/lib/bpf/usdt.c > > +++ b/tools/lib/bpf/usdt.c > > [ ... ] > > > +#if defined(__x86_64__) > > +static bool has_nop_combo(int fd, long off) > > +{ > > + static unsigned char nop_combo[6] = { > ^^^^^^^^^^^^^^^^^^^^^^ > > Should nop_combo be declared const? It is a read-only lookup table > used only in the memcmp() below. Actually you either want 'static const unsigned char' or just 'unsigned char'. In either case gcc optimises away the array, the call to memcmp(), and just compares with 32bit and 64bit constants. > > > + 0x90, 0x0f, 0x1f, 0x44, 0x00, 0x00 /* nop,nop5 */ > > + }; > > + unsigned char buf[6] = {}; No need to initialise buf[]; David > > + > > + if (pread(fd, buf, 6, off) != 6) > > + return false; > > + return memcmp(buf, nop_combo, 6) == 0; > > +} > > [ ... ] > > > --- > AI reviewed your patch. Please fix the bug or email reply why it's not a bug. > See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md > > CI run summary: https://github.com/kernel-patches/bpf/actions/runs/22221513473 > > AI-authorship-score: low > AI-authorship-explanation: Domain-specific optimization from established BPF/tracing maintainer with iterative lore review history and deep x86_64 uprobe internals knowledge. > issues-found: 1 > issue-severity-score: low > issue-severity-explanation: Missing const qualifier on a static read-only byte array is a minor C best practice deviation with no runtime impact.