Re: [net 1/2] selftests/net: fix opening object file failed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




在 2022/10/18 10:57, wangyufen 写道:

在 2022/10/13 9:51, Martin KaFai Lau 写道:
On 10/11/22 2:57 AM, Wang Yufen wrote:
The program file used in the udpgro_frglist testcase is "../bpf/nat6to4.o",
but the actual nat6to4.o file is in "bpf/" not "../bpf".
The following error occurs:
   Error opening object ../bpf/nat6to4.o: No such file or directory

hmm... so it sounds like the test never works...

The test seems like mostly exercising the tc-bpf?  It makes sense to move it to the selftests/bpf. or staying in net is also fine for now and only need to fix up the path here.

However, if moving to selftests/bpf, I don't think it is a good idea to only move the bpf prog but not moving the actual test program (the script here) such that the bpf CI can continuously testing it.  Otherwise, it will just drift and rot slowly like patch 2.

Also, if you prefer to move it to selftests/bpf, the bpf prog cannot be moved in the current form.  eg. There is some convention on the SEC name in the selftests/bpf/progs.  Also, the testing script needs to be adapted to the selftests/bpf/test_progs infra.

hmm... if moving to selftests/bpf, the actual test programs also needs to move to selftests/bpf, e.g. udpgso_bench_*, in_netns.sh, udpgso*.sh, which may not be a good idea.

So, only fix up the path here.

Also fix up the bpf/nat6to4.o compile error as following:

    make -C tools/testing/selftests/net got the following err:
    bpf/nat6to4.c:43:10: fatal error: 'bpf/bpf_helpers.h' file not found
             ^~~~~~~~~~~~~~~~~~~


After revert commit 7b92aa9e61350("selftests net: fix kselftest net fatal error"),

make -C tools/testing/selftests got the following err:

In file included from bpf/nat6to4.c:43:
../../../lib/bpf/bpf_helpers.h:11:10: fatal error: 'bpf_helper_defs.h' file not found
#include "bpf_helper_defs.h"
         ^~~~~~~~~~~~~~~~~~~

"bpf_helper_defs.h"  is generated by libbpf;


So, there are two possible approaches:  the first moving nat6to4.c and the actual test programs to selftests/bpf;

second add make dependency on libbpf for the nat6to4.c.

Which one is better?






[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux