-
Notifications
You must be signed in to change notification settings - Fork 402
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ppc64le: Tests failed (2): <no-mmap-inval.t> <reg-fd-only.t> #1207
Comments
Also, the same version suddenly got on
|
Test for IORING_FEAT_RECVSEND_BUNDLE before setting up the connection thread, or we could be tearing down pthread data at inopportune moments leading to odd behavior. Link: #1207 Fixes: 184e6ec ("test/accept-non-empty: add accept IORING_CQE_F_SOCK_NONEMPTY test") Signed-off-by: Jens Axboe <axboe@kernel.dk>
Pushed a fix for accept-non-empty, that was a bug in the test. For ppc64, the -ENOENT for mmap-no-inval is very (very) odd. For reg-fd-only, I'll push a commit to dump 'ret'. Can you try and re-run it? I'm wondering what it's returning. Maybe both are the same arch odditiy and it'll be -ENOENT?! |
Link: #1207 Signed-off-by: Jens Axboe <axboe@kernel.dk>
Thanks. Now with ebd6c8f it failed only on ppc64le (and i586 where is usually exclude
|
...
It's UDP for which we "expect" 65536 bytes in a datagram, more than usually supported by UDP. Looks the test wasn't prepared for 16K pages.
|
For 2.8 test failures on ppc64le on Linux 6.11.5:
Additionally, 1 test fail on i586:
Temporary build logs: On x86_64 and aarch64, with same build env and same kernel version tests do not fail. |
I additionally tested on 6.12-rc5, and the list of failed tests is identical across all architectures.
Temporary build logs: |
Thanks for running these. I'll check x86, but I don't have any powerpc to test on... Oh maybe this is a page size thing. What page size is your ppc box running? |
|
100 usec / 1 ms is a bit short, make it 100x larger in the hopes that it'll fix the test case on some boxes. Link: #1207 Signed-off-by: Jens Axboe <axboe@kernel.dk>
Can you test sqpoll-sleep after the commit I just made? |
Can you try and strace no-mmap-inval on ppc and attach it here? It should be using page size dependent code already. |
This is most likely because there are no huge pages available, so just skip the test in that case. Link: #1207 Signed-off-by: Jens Axboe <axboe@kernel.dk>
We should loop until WIFEXITED() is true. Link: #1207 Signed-off-by: Jens Axboe <axboe@kernel.dk>
Pushed some fixes, hopefully fixing some of them. |
Thanks. After updates applied, up to 59c0cb3 2024-10-30
ppc64le:
strace -v -f test/no-mmap-inval.t
Temporary build logs |
The bundle one needs a bit more investigation. no-mmap-inval should skip now too on ppc. I'll check the sqpoll-sleep on x86, that's very odd. |
Rewrite this test to be a bit better: - Read wakeup properly with IO_URING_READ_ONCE() - Check if wakeup has been seen - Check elapsed time before wakeup flag is seen - Prepare and push a nop request first, to ensure the thread is up and running Hopefully this will fix the quirks with this test. Link: #1207 Signed-off-by: Jens Axboe <axboe@kernel.dk>
Pushed another fix for sqpoll-sleep, can you give it a spin on x86? |
JFYI, two tests fail on ppc64 on Linux v6.6.46 for liburing-2.7. They succeed or skipped on x86_64.
On x86_64:
The text was updated successfully, but these errors were encountered: