[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5964481.DvuYhMxLoT@7950hx>
Date: Sun, 07 Dec 2025 13:07:58 +0800
From: Menglong Dong <menglong.dong@...ux.dev>
To: Yonghong Song <yonghong.song@...ux.dev>,
Alexei Starovoitov <ast@...nel.org>, Daniel Borkmann <daniel@...earbox.net>,
Andrii Nakryiko <andrii@...nel.org>, Martin KaFai Lau <martin.lau@...ux.dev>,
Eduard Zingerman <eddyz87@...il.com>, Song Liu <song@...nel.org>,
John Fastabend <john.fastabend@...il.com>, KP Singh <kpsingh@...nel.org>,
Stanislav Fomichev <sdf@...ichev.me>, Hao Luo <haoluo@...gle.com>,
Jiri Olsa <jolsa@...nel.org>, Alan Maguire <alan.maguire@...cle.com>,
"T.J. Mercier" <tjmercier@...gle.com>
Cc: "T.J. Mercier" <tjmercier@...gle.com>, bpf@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] bpf: Fix bpf_seq_read docs for increased buffer size
On 12/7/25 8:58 AM, T.J. Mercier wrote:
> Commit af65320948b8 ("bpf: Bump iter seq size to support BTF
> representation of large data structures") increased the fixed buffer
> size from PAGE_SIZE to PAGE_SIZE << 3, but the docs for the function
> didn't get updated at the same time. Update them.
>
> Fixes: af65320948b8 ("bpf: Bump iter seq size to support BTF representation of large data structures")
I think we don't need the "Fixes" tag for the document fix?
Therefore, it's better to go to the "bpf-next" tree with the
corresponding tag:
[PATCH bpf-next]
Thanks!
Menglong Dong
> Signed-off-by: T.J. Mercier <tjmercier@...gle.com>
> ---
> kernel/bpf/bpf_iter.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/bpf/bpf_iter.c b/kernel/bpf/bpf_iter.c
> index eec60b57bd3d..4b58d56ecab1 100644
> --- a/kernel/bpf/bpf_iter.c
> +++ b/kernel/bpf/bpf_iter.c
> @@ -86,7 +86,7 @@ static bool bpf_iter_support_resched(struct seq_file *seq)
>
> /* bpf_seq_read, a customized and simpler version for bpf iterator.
> * The following are differences from seq_read():
> - * . fixed buffer size (PAGE_SIZE)
> + * . fixed buffer size (PAGE_SIZE << 3)
> * . assuming NULL ->llseek()
> * . stop() may call bpf program, handling potential overflow there
> */
>
Powered by blists - more mailing lists