On Tue, Jun 25, 2024 at 5:08 PM Geliang Tang <geliang@xxxxxxxxxx> wrote: > > On Tue, 2024-06-25 at 16:29 +0800, Huacai Chen wrote: > > On Tue, Jun 25, 2024 at 4:25 PM Geliang Tang <geliang@xxxxxxxxxx> > > wrote: > > > > > > From: Geliang Tang <tanggeliang@xxxxxxxxxx> > > > > > > v2: > > > - add patch 2, a new fix for sk_msg_memcopy_from_iter. > > > - update patch 3, only test "sk->sk_prot->close" as Eric > > > suggested. > > > - update patch 4, use "goto err" instead of "return" as Eduard > > > suggested. > > > - add "fixes" tag for patch 1-3. > > > - change subject prefixes as "bpf-next" to trigger BPF CI. > > > - cc Loongarch maintainers too. > > > > > > BPF selftests seem to have not been fully tested on Loongarch. When > > > I > > > ran these tests on Loongarch recently, some errors occur. This > > > patch set > > > contains some null-check related fixes for these errors. > > Is the root cause that LoongArch lacks bpf trampoline? > > No. These errors don't seem to be directly related to the lack of BPF > trampoline. I have indeed got some errors since lacking BPF trampoline, > which is probably like this: If so, these errors seem not specific to LoongArch. Huacai > > test_dctcp:PASS:bpf_dctcp__open_and_load 0 nsec > test_dctcp:FAIL:bpf_map__attach_struct_ops unexpected error: -524 > #29/1 bpf_tcp_ca/dctcp:FAIL > test_cubic:PASS:bpf_cubic__open_and_load 0 nsec > test_cubic:FAIL:bpf_map__attach_struct_ops unexpected error: -524 > #29/2 bpf_tcp_ca/cubic:FAIL > test_dctcp_fallback:PASS:dctcp_skel 0 nsec > test_dctcp_fallback:PASS:bpf_dctcp__load 0 nsec > test_dctcp_fallback:FAIL:dctcp link unexpected error: -524 > #29/4 bpf_tcp_ca/dctcp_fallback:FAIL > test_write_sk_pacing:PASS:open_and_load 0 nsec > test_write_sk_pacing:FAIL:attach_struct_ops unexpected error: -524 > #29/6 bpf_tcp_ca/write_sk_pacing:FAIL > > Thanks, > -Geliang > > > > > Huacai > > > > > > > > Geliang Tang (4): > > > skmsg: null check for sg_page in sk_msg_recvmsg > > > skmsg: null check for sg_page in sk_msg_memcopy_from_iter > > > inet: null check for close in inet_release > > > selftests/bpf: Null checks for link in bpf_tcp_ca > > > > > > net/core/skmsg.c | 4 ++++ > > > net/ipv4/af_inet.c | 3 ++- > > > .../selftests/bpf/prog_tests/bpf_tcp_ca.c | 16 > > > ++++++++++++---- > > > 3 files changed, 18 insertions(+), 5 deletions(-) > > > > > > -- > > > 2.43.0 > > > >
