[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAADnVQKSWymsw4eu7LuHMCiYU=dfq7TCCh3LNA_mg12G+Y1=Ng@mail.gmail.com>
Date: Wed, 9 Apr 2025 08:37:09 -0700
From: Alexei Starovoitov <alexei.starovoitov@...il.com>
To: Maciej Fijalkowski <maciej.fijalkowski@...el.com>
Cc: Tushar Vyavahare <tushar.vyavahare@...el.com>, bpf <bpf@...r.kernel.org>,
Network Development <netdev@...r.kernel.org>, Björn Töpel <bjorn@...nel.org>,
"Karlsson, Magnus" <magnus.karlsson@...el.com>, Jonathan Lemon <jonathan.lemon@...il.com>,
"David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Alexei Starovoitov <ast@...nel.org>, Daniel Borkmann <daniel@...earbox.net>,
"Sarkar, Tirthendu" <tirthendu.sarkar@...el.com>
Subject: Re: [PATCH bpf-next v4 0/2] selftests/xsk: Add tests for XDP tail
adjustment in AF_XDP
On Tue, Apr 8, 2025 at 9:55 AM Maciej Fijalkowski
<maciej.fijalkowski@...el.com> wrote:
>
> On Fri, Mar 21, 2025 at 12:54:17AM +0000, Tushar Vyavahare wrote:
> > This patch series adds tests to validate the XDP tail adjustment
> > functionality, focusing on its use within the AF_XDP context. The tests
> > verify dynamic packet size manipulation using the bpf_xdp_adjust_tail()
> > helper function, covering both single and multi-buffer scenarios.
> >
> > v1 -> v2:
> > 1. Retain and extend stream replacement: Keep `pkt_stream_replace`
> > unchanged. Add `pkt_stream_replace_ifobject` for targeted ifobject
> > handling.
> >
> > 2. Consolidate patches: Merge patches 2 to 6 for tail adjustment tests and
> > check.
> >
> > v2 -> v3:
> > 1. Introduce `adjust_value` to replace `count` for clearer communication
> > with userspace.
> >
> > v3 -> v4:
> > 1. Remove `testapp_adjust_tail_common()`. [Maciej]
> >
> > 2. Add comments and modify code for buffer resizing logic in test cases
> > (shrink/grow by specific byte sizes for testing purposes). [Maciej]
>
> Hi BPF maintainers,
>
> could we merge this patch set as i have acked the patches? Or is there
> something that stops us? I suppose this might have slipped during the
> merge window?
Sorry. It got lost. Please resend the set.
Powered by blists - more mailing lists