- Mar 28, 2022
-
-
Jeff Moyer authored
- Fix padding in vector and sockaddr structures on 32 bit (Brett Holman) - Add a test case for missed poll events (Eric Biggers) - Add loongarch support (Yehui Ren) - man page cleanup and fixes (Guillem Jover) - assorted test harness fixess (Guillem Jover, Jeff Moyer)
-
- Mar 25, 2022
-
-
Eric Biggers authored
Add a regression test for a recently-fixed kernel bug where aio polls sometimes didn't complete even if the file is ready. This is a cleaned-up version of the test which I originally posted at https://lore.kernel.org/r/YbMKtAjSJdXNTzOk@sol.localdomain Signed-off-by:
Eric Biggers <ebiggers@google.com> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Sep 29, 2021
-
-
Jeff Moyer authored
Padding for the various structures in the iocb.u union should be the same. This patch verifies they are at build time. Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Jeff Moyer authored
The io_iocb_sockaddr structure definition is missing padding for @addr and @len, which could result in -EINVAL being returned from io_submit() on applications compiled for 32 bit architectures. Fix it. This was found after Brett reported a similar bug in the io_iocb_vector structure. Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Brett Holman authored
The io_iocb_vector structure definition is missing padding for @vec and @nr, which results in -EINVAL being returned from io_submit() on applications compiled for 32 bit architectures. Fix it. Signed-off-by:
Brett Holman <bholman.devel@gmail.com> [JEM: add commit message; use PADDEDul for nr] Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Jul 20, 2021
-
-
yehui ren authored
Signed-off-by:
yehui ren <renyehui@uniontech.com> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
yehui ren authored
Signed-off-by:
yehui ren <renyehui@uniontech.com> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
yehui ren authored
Signed-off-by:
yehui ren <renyehui@uniontech.com> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
yehui ren authored
Signed-off-by:
yehui ren <renyehui@uniontech.com> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Feb 15, 2021
-
-
Lee Duncan authored
Test 3.t exercises io_submit() with invalid addresses, and one test attempted to pass in "-1" as an invalid pointer. But gcc-11 realizes you cannot take an offset from an invalid pointer, making it impossible to compile or run this test: > sh# make CC=gcc-11 partcheck > make[1]: Entering directory '/alt/public_software/libaio/src' > make[1]: Nothing to be done for 'all'. > make[1]: Leaving directory '/alt/public_software/libaio/src' > make[1]: Entering directory '/alt/public_software/libaio/harness' > gcc-11 -Wall -Werror -I../src -g -O2 -DTEST_NAME=\"cases/2.t\" -o cases/2.p main.c ../src/libaio.a -lpthread > gcc-11 -Wall -Werror -I../src -g -O2 -DTEST_NAME=\"cases/3.t\" -o cases/3.p main.c ../src/libaio.a -lpthread > In file included from main.c:24: > cases/3.t: In function ‘test_main’: > cases/3.t:18:19: error: ‘attempt_io_submit’ accessing 8 bytes in a region of size 0 [-Werror=stringop-overflow=] > 18 | status |= attempt_io_submit( io_ctx, 1, (void *)-1, -EFAULT); > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > cases/3.t:18:19: note: referencing argument 3 of type ‘struct iocb **’ > In file included from cases/3.t:5, > from main.c:24: > cases/aio_setup.h:15:5: note: in a call to function ‘attempt_io_submit’ > 15 | int attempt_io_submit(io_context_t ctx, long nr, struct iocb *ios[], int expect) > | ^~~~~~~~~~~~~~~~~ > cc1: all warnings being treated as errors > make[1]: *** [Makefile:19: cases/3.p] Error 1 > make[1]: Leaving directory '/alt/public_software/libaio/harness' > make: *** [Makefile:23: partcheck] Error 2 The fix is to tell gcc to relax it's stringop overflow testing for this one test. Changes since v1: * First version just skipped the test in question Signed-off-by:
Lee Duncan <leeman.duncan@gmail.com> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Aug 22, 2019
-
-
Guillem Jover authored
Either the kernel or the syscall wrapper will return -ENOSYS when this syscall is not implemented. So we should cope with this in the test suite and SKIP the test case. This is currently the case for alpha and ia64, even though they have already been wired up very recently in the kernel mainline. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
These should help diagnose problems when dealing with error failures. In particular this helped distinguish the problem with io_pgetevents() not being implemented and it failing due to the sigset_t layout bug. Signed-off-by:
Guillem Jover <guillem@hadrons.org> [JEM: fix up 80 columns violations] Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Aug 14, 2019
-
-
Guillem Jover authored
This partially reverts commit d7f50654. The actual problem in the original code was that read() was being used to assert whether the buffer was readable, but the kernel was instead reading from the file descriptor and then writing into the buffer, so no EFAULT was being generated (on architectures that do so). We needed to use a write() so that the kernel would read from the buffer. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Aug 13, 2019
-
-
Jeff Moyer authored
This test has been broken forever. Fix it up to perform an aio_read using the result of a regular read as the expected return code. Reported-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Jul 29, 2019
-
-
Jeff Moyer authored
Use the new skip error code instead of failing the test. Also add in a Local variables: section for emacs. Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
On filesystems such as tmpfs the syscall might return -ENOTSUP instead of EINVAL when it does not support the RWF_NOWAIT flag. Signed-off-by:
Guillem Jover <guillem@hadrons.org> [JEM: skip the test instead of returning success] [JEM: make the error message differentiate between kernel and fs support] Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
When running the harness on a filesystem such as a tmpfs, which do not support O_DIRECT, fallback to calls without the flag. Signed-off-by:
Guillem Jover <guillem@hadrons.org> [JEM: change from duplicating the open call to using F_SETFL] [JEM: 18 and 21 require O_DIRECT-skip if not present] Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Jeff Moyer authored
Skipped tests will not cause the test harness to return failure. An exit status of "3" was chosen for skipped tests. This doesn't conflict with any of the current tests. Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
This way we can use the exit code to check whether the tests passed or failed, and fail the package build. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
This is a recent architecture and as such does not provide legacy support for SYS_eventfd. Declare that we need to use the new syscall. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
The getconf(1) command is inherently not cross-compilation friendly. In addition PAGESIZE depends on the specific system, even within a specific arch, so using a hard-coded value is never safe. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Even though this is the same size, as the sizeof() is derived from the source expression, recent gcc versions will emit a warning, which is turned into an error by -Werror: error: argument to ‘sizeof’ in ‘strncpy’ call is the same expression as the source; did you mean to use the size of the destination? [-Werror=sizeof-pointer-memaccess] Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Stephan Springl authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
- Remove unnecessary macro argument quoting. - Variables, pathnames in italics. - Keywords in bold. - Man page references in bold, followed by the man page number. - Fix TP/TQ macro usage. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
- Update year. - Balance double quotes. - Remove version from source argument as recommended in man-pages(7). - Fix all sections numbers to 3. Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Guillem Jover authored
Signed-off-by:
Guillem Jover <guillem@hadrons.org> Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
Jeff Moyer authored
A user can now specify "LIBAIO=/path/to/libaio.so" in order to run the test harness against the installed library. Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>
-
- Oct 22, 2018
-
-
Jeff Moyer authored
- Add async poll support (Christoph Hellwig) - Use canonical DESTDIR= environment variable (Thomas Petazzoni) - Add ability to disable building the shared library (Thomas Petazzoni) Signed-off-by:
Jeff Moyer <jmoyer@redhat.com>