From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f54.google.com (mail-wr1-f54.google.com [209.85.221.54]) (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 012BE38BF92 for ; Mon, 23 Mar 2026 10:59:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774263580; cv=none; b=jYFRgAstBpV+U6gfVM4x4KC+uCptLo5kGsym8TtOZieadUCnnvomHnzb3JEIkJBHUih2+4b3B508B81K8CDyfS/EySoB4zL9eu0tLflPNAKCg5PNaASaWWNfUgEVVvYQgJCm/yJom9ayKLcnI8mO3FEuV85ih9zWOi5PtgsjN1E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774263580; c=relaxed/simple; bh=V3qIVBeIYZc6o6LUFDbfme/F0QW48dL8M3JlrifxMCo=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ssqQEyiJ62Oga7qPrQD0SqrfzF672AputQwEWHZNSU3H8RJzE1MLEfHYXS/c3JqhdqCp9vIg+4e8wPl7vi70bcToxAK3lbTgflOimLsW1wDJAi/pwaC3bEzRXN5+ZcPaGSssEGQGgIkBLIN8oFdZZTI4MiIoGXXimK3QdBVhGHw= 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=LWseFFTw; arc=none smtp.client-ip=209.85.221.54 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="LWseFFTw" Received: by mail-wr1-f54.google.com with SMTP id ffacd0b85a97d-43b3d9d0695so3742726f8f.0 for ; Mon, 23 Mar 2026 03:59:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1774263577; x=1774868377; 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=PdHGJawNgmz3Mnxlo8DQZ2ezVhQNMuS/IKfPtN1AcXY=; b=LWseFFTwOjloZ2wNHPfiGJwDkZNrtVdTGAqR9geD3yoY6TrT/zQMTS03ZAqLle6Jvy w5vwEwYIsfXWlPqDk9B56vTZJBhvGnd7baMAICJ+zY7R1rWBBz44AZ/crpRkaVyA0zZY UE8Lp4LuH5X0QRKPxkfRBrg/2dnK9pvd96T6ICX2P3aNQehcOI29ElPVhpPYPHAwlXFy Klv9wRuCR64bS52jwkhT8hH3SA5SBIoZdVDAlt945MHloUYyQ1A3qu2vzHpLs7XrcWBs mq0Kqlzy7tXgR8gk9s62mZ6Qc0ZyFufoffOvF2u868pNmh9Sj548IM6Rhw7/4pNjPP4h yygQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774263577; x=1774868377; 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=PdHGJawNgmz3Mnxlo8DQZ2ezVhQNMuS/IKfPtN1AcXY=; b=BBfhZxJT1jAshtnYwRZow13aAddlM6OureHDvuxlCqkj+MH2lyIQzriwOXsAbK3DXH 8Qy3hReXr+pKbOSO803wrWfBdf1h+BVF6gbK8sd7xRyNPerc/jK5qwhQNrN0riiuAmEs AQUMI9MwmVMpTFLwE7lBFfgEpug/VEWaQkVCZtK7Rje05O/XyKrrxXf8mf/8gCn7DzaV 1b8ZnSLLUWWADK8KGgdOEmQxxM54ket4m9tvbR4w66zZBJZ9SUWiRzImc3nU2pn50itR UqhpygJTz7No/XDp3bgB4nd/c51jBBQvJx8edMUgcynA2omUMwf6mF8y5K7CjAR9B0ax OXmA== X-Forwarded-Encrypted: i=1; AJvYcCW4MgAQtVbSSDdlgyJNDgVXnoeF+/gypw5QqK21T86UCS3PWFtEn5XFtbdXf2UgJe1jvx7O30pbTlAXExI=@vger.kernel.org X-Gm-Message-State: AOJu0YzhW5RTMncC5TGsTB9ejOwUqab4vIlb7eW66OYViLBWEA8sKxNq X2X2ZBIGZASMZ49oD1MwFnlQHuls9wxtqeFR+oQFfBua49VQxInTwcQz X-Gm-Gg: ATEYQzw6Eax2PwVSBda+lI0wCkF8fMvMTJ017IIkh4HVb9vthd6dEYPiWfq7A0RDogF OHlCtHJNcPOpCOvgDwnAoK78VCHwlJI6LgCZLp+pGukg3gy4jCvMsqgqSYFRKEM+u+mEIAo7Ipy nRJLa32YIClxLYMEAwwOun0IwdmSzz53iCP+aZwSsxr6Fnb1QO2oh49QWQva2SFhwK/d1kDMt7R QjjfCOCgJHZW3Xg5O3I62X/qkFXaU1aUyDmentCN7NX8A2sK7uPKkDYRx4sgevLDD5C8nH44Unm IGNEEn9V7S329aHczhOktZS379cllH23K2dIbNI6daGE+SKExi/XCe7i6EsevzIdUMp2dkXIVfQ zJYQP3xWn6rx4TQgj4CJoXXmyExPoAHrsqcUyZKwMe9KMAZBXX+NqZNxqopUz4+gRfpMv5kfvpv HmdZGPe16GG9tS84SGg17kT9qmpVU= X-Received: by 2002:a05:6000:220f:b0:43b:7481:f899 with SMTP id ffacd0b85a97d-43b7481fabfmr6744251f8f.7.1774263577027; Mon, 23 Mar 2026 03:59:37 -0700 (PDT) Received: from krava ([2a00:102a:502b:faa8:f878:ffa9:3b65:76be]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43b646b0b0csm27307584f8f.15.2026.03.23.03.59.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 23 Mar 2026 03:59:36 -0700 (PDT) From: Jiri Olsa X-Google-Original-From: Jiri Olsa Date: Mon, 23 Mar 2026 11:59:34 +0100 To: Menglong Dong Cc: ast@kernel.org, olsajiri@gmail.com, daniel@iogearbox.net, andrii@kernel.org, martin.lau@linux.dev, eddyz87@gmail.com, song@kernel.org, yonghong.song@linux.dev, john.fastabend@gmail.com, kpsingh@kernel.org, sdf@fomichev.me, haoluo@google.com, shuah@kernel.org, bpf@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH bpf-next v2] selftests/bpf: test access to ringbuf position with map pointer Message-ID: References: <20260323021839.101316-1-dongml2@chinatelecom.cn> 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: <20260323021839.101316-1-dongml2@chinatelecom.cn> On Mon, Mar 23, 2026 at 10:18:39AM +0800, Menglong Dong wrote: > Add the testing to access the bpf_ringbuf with the map pointer. > "consumer_pos" and "producer_pos" is accessed in this testing. We reserve > 128 bytes in the ringbuf to test the producer_pos, which should be > "128 + 8", and the "8" is BPF_RINGBUF_HDR_SZ. > > It will be helpful if we want to evaluate the usage of the ringbuf in bpf > prog with the consumer and producer position. lgtm, question though.. is this related to some kernel change or some ongoing work? looks like basic operation that's already tested indirectly by existing tests Acked-by: Jiri Olsa jirka > > Signed-off-by: Menglong Dong > Reviewed-by: Emil Tsalapatis > --- > v2: > - don't set the max_entries for the ringbuf map > - add comment for the producer_pos > --- > .../testing/selftests/bpf/progs/map_ptr_kern.c | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/tools/testing/selftests/bpf/progs/map_ptr_kern.c b/tools/testing/selftests/bpf/progs/map_ptr_kern.c > index efaf622c28dd..d7611e7018ca 100644 > --- a/tools/testing/selftests/bpf/progs/map_ptr_kern.c > +++ b/tools/testing/selftests/bpf/progs/map_ptr_kern.c > @@ -647,8 +647,14 @@ static inline int check_devmap_hash(void) > return 1; > } > > +struct bpf_ringbuf { > + unsigned long consumer_pos; > + unsigned long producer_pos; > +} __attribute__((preserve_access_index)); > + > struct bpf_ringbuf_map { > struct bpf_map map; > + struct bpf_ringbuf *rb; > } __attribute__((preserve_access_index)); > > struct { > @@ -659,9 +665,21 @@ static inline int check_ringbuf(void) > { > struct bpf_ringbuf_map *ringbuf = (struct bpf_ringbuf_map *)&m_ringbuf; > struct bpf_map *map = (struct bpf_map *)&m_ringbuf; > + struct bpf_ringbuf *rb; > + void *ptr; > > VERIFY(check(&ringbuf->map, map, 0, 0, page_size)); > > + ptr = bpf_ringbuf_reserve(&m_ringbuf, 128, 0); > + VERIFY(ptr); > + > + bpf_ringbuf_discard(ptr, 0); > + rb = ringbuf->rb; > + VERIFY(rb); > + VERIFY(rb->consumer_pos == 0); > + /* The "8" here is BPF_RINGBUF_HDR_SZ */ > + VERIFY(rb->producer_pos == 128 + 8); > + > return 1; > } > > -- > 2.53.0 >