[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <0a3933ee-17aa-c6fa-a493-89b96e1c01db@intel.com>
Date: Fri, 17 Mar 2023 14:40:23 +0100
From: Alexander Lobakin <aleksander.lobakin@...el.com>
To: Ilya Leoshkevich <iii@...ux.ibm.com>
CC: Alexei Starovoitov <ast@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>,
Andrii Nakryiko <andrii@...nel.org>,
Martin KaFai Lau <martin.lau@...ux.dev>,
Maciej Fijalkowski <maciej.fijalkowski@...el.com>,
Larysa Zaremba <larysa.zaremba@...el.com>,
Toke Høiland-Jørgensen <toke@...hat.com>,
Song Liu <song@...nel.org>,
Jesper Dangaard Brouer <hawk@...nel.org>,
Jakub Kicinski <kuba@...nel.org>, <bpf@...r.kernel.org>,
<netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH bpf-next 2/2] selftests/bpf: fix "metadata marker" getting
overwritten by the netstack
From: Ilya Leoshkevich <iii@...ux.ibm.com>
Date: Thu, 16 Mar 2023 22:22:26 +0100
> On Thu, 2023-03-16 at 18:50 +0100, Alexander Lobakin wrote:
>> Alexei noticed xdp_do_redirect test on BPF CI started failing on
>> BE systems after skb PP recycling was enabled:
[...]
>> @@ -52,7 +52,7 @@ int xdp_redirect(struct xdp_md *xdp)
>>
>> *payload = MARK_IN;
>>
>> - if (bpf_xdp_adjust_meta(xdp, 4))
>> + if (bpf_xdp_adjust_meta(xdp, sizeof(__u64)))
>> return XDP_ABORTED;
>>
>> if (retcode > XDP_PASS)
>
> Tested-by: Ilya Leoshkevich <iii@...ux.ibm.com>
Much thanks again for the debugging and testing! It definitely wouldn't
have been quick without you :D
Olek
Powered by blists - more mailing lists