ACK: [B][SRU][PATCH 0/4] Enable fib-onlink-tests.sh and msg_zerocopy.sh in kselftests/net
kleber.souza at canonical.com
Fri Jul 9 08:27:18 UTC 2021
On 07.07.21 13:47, Po-Hsu Lin wrote:
> Found this issue while debugging missing tests with bug 1934282.
> On Bionic there are 3 scripts that are not in the Makefile of the net
> test suite of kselftests:
> $ for file in $(ls *.sh); do grep -q $file Makefile || echo $file; done
> * 830669e691464c selftests/net: enable msg_zerocopy test
> * 1751eb42ddb56b selftests: net: use TEST_PROGS_EXTENDED
> * a52b839752aab7 selftests: Add fib-onlink-tests.sh to TEST_PROGS
> * Set fib-onlink-tests.sh as executable
> They all need to be backported to Bionic.
> For the in_netns.sh it's not causing any issue to us when being
> called by run_afpackettests, but I think it's no harm to fix it to
> reduce confusions. Commit 1751eb42ddb56b will replace the changes in
> 9faedd64 selftests: net: add in_netns.sh TEST_GEN_PROGS_EXTENDED 
> and 5ff9c1a3 selftests: net: add in_netns.sh to TEST_PROGS .
> Since we need to backport it there is no need to work on these two.
> The fib-onlink-tests.sh must be set to executable otherwise it will
> fail with test not executable.
>  https://github.com/torvalds/linux/commit/9faedd643fd9
>  https://github.com/torvalds/linux/commit/5ff9c1a3dd92
> Run the "net" test suite in the kselftest directory from a patched
> source tree. fib-onlink-tests.sh and msg_zerocopy.sh tests will be
> [Where problems could occur]
> This change will bring in more test for our SRU, we might see new
> failures because of these test in the future.
> Anders Roxell (1):
> selftests: net: use TEST_PROGS_EXTENDED
> David Ahern (1):
> selftests: Add fib-onlink-tests.sh to TEST_PROGS
> Po-Hsu Lin (1):
> UBUNTU: SAUCE: selftests: Make fib-onlink-tests.sh executable
> Willem de Bruijn (1):
> selftests/net: enable msg_zerocopy test
> tools/testing/selftests/net/Makefile | 3 +++
> tools/testing/selftests/net/fib-onlink-tests.sh | 0
> tools/testing/selftests/net/msg_zerocopy.sh | 8 ++++++++
> 3 files changed, 11 insertions(+)
> mode change 100644 => 100755 tools/testing/selftests/net/fib-onlink-tests.sh
Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>
More information about the kernel-team