netdev - Re: [PATCH bpf-next 4/4] bpf/selftests: add simple selftest for bpf_smc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <bf535357-74ad-4eaa-96d7-5e527789d10d@linux.dev>
Date: Tue, 26 Nov 2024 09:29:49 +0100
From: Zhu Yanjun <yanjun.zhu@...ux.dev>
To: Martin KaFai Lau <martin.lau@...ux.dev>,
 "D. Wythe" <alibuda@...ux.alibaba.com>
Cc: kgraul@...ux.ibm.com, wenjia@...ux.ibm.com, jaka@...ux.ibm.com,
 ast@...nel.org, daniel@...earbox.net, andrii@...nel.org, pabeni@...hat.com,
 song@...nel.org, sdf@...gle.com, haoluo@...gle.com, yhs@...com,
 edumazet@...gle.com, john.fastabend@...il.com, kpsingh@...nel.org,
 jolsa@...nel.org, guwen@...ux.alibaba.com, kuba@...nel.org,
 davem@...emloft.net, netdev@...r.kernel.org, linux-s390@...r.kernel.org,
 linux-rdma@...r.kernel.org, bpf@...r.kernel.org, dtcccc@...ux.alibaba.com
Subject: Re: [PATCH bpf-next 4/4] bpf/selftests: add simple selftest for
 bpf_smc_ops

在 2024/11/26 0:32, Martin KaFai Lau 写道:
> On 11/25/24 2:52 AM, Zhu Yanjun wrote:
>>>> # ./test_progs -t smc
>>>> #27/1    bpf_smc/load:OK
>>>> #27      bpf_smc:OK
>>>> Summary: 1/1 PASSED, 0 SKIPPED, 0 FAILED
>>>>
>>>> The above command is based on several kernel modules. After these 
>>>> dependent kernel modules are loaded, then can run the above command 
>>>> successfully.
> 
>>>
>>> This is indeed a problem, a better way may be to create a separate 
>>> testing directory for SMC, and we are trying to do this.
>>
>> Got it. In the latest patch series, if a test program in sample/bpf 
>> can verify this bpf feature, it is better than a selftest program in 
>> the directory tools/ testing/selftests/bpf.
>>
>> I delved into this selftest tool. It seems that this selftest tool 
>> only makes the basic checks. A test program in sample/bpf can do more.
> 
> sample(s)/bpf? No new test should be added to samples/bpf which is 
> obsolete. The bpf CI only runs tests under selftests/bpf.

Thanks for letting me know this, Martin.

In the past, with samples/bpf, we can know a lot of details of bpf 
samples. But in the selftests/bpf, it seems that only load method can be 
found. For example, in this smc bpf selftests commit, we can not find 
how to change parameters in smc. In the past, this method about how to 
change parameters can be found in samples/bpf.

I am not sure whether this selftests/bpf is designed for this simple 
tests or the detailed information can be found in other places.

Zhu Yanjun

> 
> There is selftests/bpf/config to tell the bpf CI about what kconfig 
> needs to turn on.
>