Message ID | 20220607025435.184105-1-zhaogongyi@huawei.com |
---|---|
State | Superseded |
Headers | show |
Series | [v2] futex_waitv01: Add test verifies EAGIN/ETIMEDOUT | expand |
Hi! > +static void test_mismatch_between_uaddr_and_val(void) > +{ > + struct timespec to; > + > + waitv->uaddr = (uintptr_t)futex; > + waitv->flags = FUTEX_32 | FUTEX_PRIVATE_FLAG; > + waitv->val = 1; > + > + init_timeout(&to); > + > + TST_EXP_FAIL(futex_waitv(waitv, 1, 0, &to, CLOCK_MONOTONIC), EAGAIN, > + "futex_waitv mismatch between value of uaddr and val"); The init_timeout(&to) inits the timeout with CLOCK_REALTIME time, so we should pass CLOCK_REALTIME to the futex_waitv() as well. > +} > + > +static void test_timeout(void) > +{ > + struct timespec to; > + > + waitv->uaddr = (uintptr_t)futex; > + waitv->flags = FUTEX_32 | FUTEX_PRIVATE_FLAG; > + waitv->val = 0; > + > + init_timeout(&to); I guess that we can as well just do: SAFE_CLOCK_GETTIME(CLOCK_REALTIME, &to); Which would make the test much faster as we will not have to wait for one second for the timeout. Eventually we can as well add a few miliseconds to the timeout, we do have a nice functions to work with different time structures in this case we can just do: to = tst_timespec_add_us(to, 10000); To add 10ms to the timeout. > + TST_EXP_FAIL(futex_waitv(waitv, 1, 0, &to, CLOCK_REALTIME), ETIMEDOUT, > + "futex_waitv timeout"); > + > +} > + > static void cleanup(void) > { > free(futex); > @@ -126,6 +154,8 @@ static void run(void) > test_null_waiters(); > test_invalid_clockid(); > test_invalid_nr_futexes(); > + test_mismatch_between_uaddr_and_val(); > + test_timeout(); > } > > static struct tst_test test = { > -- > 2.17.1 > > > -- > Mailing list info: https://lists.linux.it/listinfo/ltp
diff --git a/testcases/kernel/syscalls/futex/futex_waitv01.c b/testcases/kernel/syscalls/futex/futex_waitv01.c index f2c19b748..2d905fa5f 100644 --- a/testcases/kernel/syscalls/futex/futex_waitv01.c +++ b/testcases/kernel/syscalls/futex/futex_waitv01.c @@ -113,6 +113,34 @@ static void test_invalid_nr_futexes(void) "futex_waitv invalid nr_futexes"); } +static void test_mismatch_between_uaddr_and_val(void) +{ + struct timespec to; + + waitv->uaddr = (uintptr_t)futex; + waitv->flags = FUTEX_32 | FUTEX_PRIVATE_FLAG; + waitv->val = 1; + + init_timeout(&to); + + TST_EXP_FAIL(futex_waitv(waitv, 1, 0, &to, CLOCK_MONOTONIC), EAGAIN, + "futex_waitv mismatch between value of uaddr and val"); +} + +static void test_timeout(void) +{ + struct timespec to; + + waitv->uaddr = (uintptr_t)futex; + waitv->flags = FUTEX_32 | FUTEX_PRIVATE_FLAG; + waitv->val = 0; + + init_timeout(&to); + + TST_EXP_FAIL(futex_waitv(waitv, 1, 0, &to, CLOCK_REALTIME), ETIMEDOUT, + "futex_waitv timeout"); +} + static void cleanup(void) { free(futex); @@ -126,6 +154,8 @@ static void run(void) test_null_waiters(); test_invalid_clockid(); test_invalid_nr_futexes(); + test_mismatch_between_uaddr_and_val(); + test_timeout(); } static struct tst_test test = {
Add test verifies EAGIN/ETIMEDOUT for futex_waitv according to https://www.kernel.org/doc/html/latest/userspace-api/futex2.html. Signed-off-by: Zhao Gongyi <zhaogongyi@huawei.com> --- v1->v1: move test code to futex_waitv01.c .../kernel/syscalls/futex/futex_waitv01.c | 30 +++++++++++++++++++ 1 file changed, 30 insertions(+) -- 2.17.1