ltp icon indicating copy to clipboard operation
ltp copied to clipboard

Write test for futex_waitv()

Open metan-ucw opened this issue 4 years ago • 3 comments
trafficstars

https://lwn.net/Articles/866112/ https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=595b28fb0c8949463d8ec1e485f36d17c870ddb2

There are some selftests but I haven't checked if the coverage is sufficient.

metan-ucw avatar Nov 08 '21 10:11 metan-ucw

Maybe it's a bit straight forward as a suggestion, but why not bringing Linux kernel selftests inside LTP? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests/futex

acerv avatar Dec 27 '21 13:12 acerv

Maybe it's a bit straight forward as a suggestion, but why not bringing Linux kernel selftests inside LTP? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests/futex

I'm not so sure, how well could we manage different kernel versions with "integrated kselftests"? From the looks, it seems that we would need to rewrite those anyways adding "tst_kvercmp()" there and there. Please correct me if I'm wrong.

kamiccolo avatar Dec 27 '21 15:12 kamiccolo

There are many other problems in there, apart from the missing check for the syscall existence. These tests are written so that they are all executed by a single script, there is no proper result reporting etc. Also most of these tests should be already present in LTP, since the original futex tests the selftest are based on have been part of the LTP for years. And as these tests were rewritten and cleaned up there is no reason to do verbatim copy to LTP at all. We only need to implement LTP test for the recent additions that are not covered by LTP yet.

metan-ucw avatar Jan 03 '22 12:01 metan-ucw

futex_waitv tests have been added inside LTP, so the ticket can be cosed

acerv avatar Oct 06 '22 13:10 acerv