Age | Commit message (Collapse) | Author |
|
Original change: https://android-review.googlesource.com/c/platform/external/liburing/+/2276974
Change-Id: I2a7ecac6e769b82bba38ebfbce3301259a663146
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/liburing/+/2276974
Change-Id: Ie4d6c0c023d49198c105b3d2fc1e5d51edef6398
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/liburing/+/2276974
Change-Id: I0ac5dbb2b0dadb76713528e6ab0110ec071bad63
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Merge branch 'liburingtag2_2' into liburing_upgrade
Test: atest liburing_cpp_tests
Bug: b/253385258
Change-Id: Ia9b4acfce599cd0f7a2ab22999e64d107bc1b96c
|
|
We expect tests to only complain on error, silence all the useless
garbage this old test prints.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
These two tests are probably useless these days. But at least make them
adhere to the current testing format, where we don't dump a bunch of
useless info when they are run. Only dump for errors.
Also make internal functions static, which then found a function that
isn't even used...
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
* 'master' of https://github.com/CarterLi/liburing:
man/io_uring_prep_cancel.3: clearify the synchronous manner
|
|
Signed-off-by: 李通洲 <carter.li@eoitek.com>
|
|
This does three things:
- change multishot_mask from uint to uint32_t
- change multishot_mask != UINT_MAX check to ~multishot_mask != 0
The above two avoid compiler and arch influence. Make the logic more robost.
- other cleaning to make code clearer.
Signed-off-by: Hao Xu <howeyxu@tencent.com>
Link: https://lore.kernel.org/r/20220617143603.179277-4-hao.xu@linux.dev
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
We don't need to minus one for the s_fd[i] since the returned cqe.res
is already the fixed file table slot which is indexed from zero.
Fixes: 66cf84527c34 ("test/accept.c: add test for multishot mode accept")
Signed-off-by: Hao Xu <howeyxu@tencent.com>
Link: https://lore.kernel.org/r/20220617143603.179277-3-hao.xu@linux.dev
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
This fixes a bug in accept_conn handling in the accept tests that caused it
to incorrectly skip the multishot tests and also lose the warning message
to a closed stdout. This can be seen in the strace output below.
close(1) = 0
io_uring_setup(32, { ...
...
write(1, "Fixed Multishot Accept not suppo"..., 47) = -1 EINVAL
Unfortunately this exposes a a bug with gcc -O2 where multishot_mask logic
gets optimized incorrectly and "Fixed Multishot Accept misses events" is
wrongly reported. I am investigating this separately.
Fixes: 66cf84527c34 ("test/accept.c: add test for multishot mode accept")
Signed-off-by: Donald Hunter <donald.hunter@gmail.com>
Link: https://lore.kernel.org/r/20220617143603.179277-2-hao.xu@linux.dev
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
We removed CQE32 for nops from the kernel, fix the tests and instead
test that we return zeroes in the extra fields instead of garbage.
Loop over the tests multiple times so it exhausts CQ and we also test
CQ entries recycling and internal caching mechanism. Also excersie
IOSQE_ASYNC.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/92f01041e5ef933a6018bd89dd54cc1fae57c6f6.1655455225.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
This reverts commit e5d017ab9cd9605248db68168ae5451f830e646c.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/a85bf82b68dca4f82fa8c275d5ed7e06cc065034.1655455225.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Add async to the mix as well, with the async test case being mixed in
itself by only doing half of them async.
Use ->user_data to tell which offset was used, so we don't have any
implied ordering there.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
This was removed as it was just a debug feature, remove it from the
test case as well.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
The NOP support for IOSQE_BUFFER_SELECT has been reverted, so use a
supported function with read. This also allows verifying that the
correct data has actually been read.
Signed-off-by: Dylan Yudaken <dylany@fb.com>
Link: https://lore.kernel.org/r/20220615113733.1424472-1-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
To keep things consistent between 2.1 and later versions of liburing,
make io_uring_prep_cancel() take a void * pointer again instead of a
__u64 type. Add io_uring_prep_cancel64() that takes a 64-bit type.
Link: https://github.com/axboe/liburing/discussions/601
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
* 'master' of https://github.com/CarterLi/liburing:
man/io_uring_setup.2: fix typo
|
|
Signed-off-by: 李通洲 <carter.li@eoitek.com>
|
|
We dropped this from 5.19, prune it from liburing as well for now.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
It's 5.19, not 5.10.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Fixes: 01e0120220e4 ("buf-ring: add tests that cycle through the provided buffer ring")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
https://github.com/fischerling/liburing
* 'fix-changelog-man-reference' of https://github.com/fischerling/liburing:
changelog: fix wrong man page reference
|
|
Add tests that make sure that the buffer ring logic works properly
without actually using the data in each buffer.
This exposes some bugs in 5.19-rc1
Signed-off-by: Dylan Yudaken <dylany@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Kernel expects the tail to start at 0, so provide an API to init the
ring appropriately.
Signed-off-by: Dylan Yudaken <dylany@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Without the mask parameter, it's not feasible to use this API without
knowing where the tail is and performing some arithmetic
Signed-off-by: Dylan Yudaken <dylany@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
This function doesn't exist
Signed-off-by: Dylan Yudaken <dylany@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
During reading the recently added CHANGELOG I stumbled over
IORING_SETUP_COOP_TASKRUN which is documented in man/io_uring_setup.2
rather than in man/io_uring_enter.3.
Fixes: b7911762 ("Add CHANGELOG file")
Signed-off-by: Florian Fischer <florian.fischer@muhq.space>
|
|
* 'fix/man-madvise' of https://github.com/topecongiro/liburing:
man/io_uring_prep_madvise.3: posix_madvise -> madvise
man/io_uring_prep_madvise.3: remove fd
|
|
`io_uring_prep_madvise` prepares an asynchronous madvise(2),
not posix_madvise(3).
Signed-off-by: Seiichi Uchida <topecongiro@fastmail.com>
|
|
The man page for `io_uring_prep_madvise` contains wrong function
signature and description. This commit fixes is by removing
the unused parameter.
Signed-off-by: Seiichi Uchida <topecongiro@fastmail.com>
|
|
* 'fix/man-accept' of https://github.com/topecongiro/liburing:
man/io_uring_prep_accept.3: fix parameter types
|
|
`io_uring_prep_accept` and its variants accept `socklen_t *` not
`socklen_t`.
Signed-off-by: Seiichi Uchida <topecongiro@fastmail.com>
|
|
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
If make cscope is used, we don't want to have the output file shown as
being not part of the repo.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
For newer kernels, the rings themselves are not accounted via
RLIMIT_MEMLOCK, but registered buffers still are.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Consistent use of -errno, remove double empty lines, some formatting,
and some clarifications in various spots.
Also add the data64 variants of the sqe setting and cqe user_data
getting helpers.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Pull GitHub bot updates from Ammar Faizi:
- Use clang latest version for better issue coverage.
- Add more architectures for GitHub bot: powerpc64, powerpc, alpha,
mips64, mips.
Link: https://github.com/axboe/liburing/pull/590
* tag 'github-bot-2022-06-02' of https://github.com/ammarfaizi2/liburing:
.github: Add mips build for GitHub bot
.github: Add mips64 build for GitHub bot
.github: Add alpha build for GitHub bot
.github: Add powerpc build for GitHub bot
.github: Add powerpc64 build for GitHub bot
.github: Use clang latest version
|
|
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
|
|
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
|
|
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
|
|
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
|
|
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
|
|
By default, when we install clang from the default `Ubuntu apt` on the
GitHub bot, we get clang-11, which is kinda old.
clang-15 has better static analysis than the previous versions, for
example, the fix in these commits were only found by clang-15:
7e3f7d6030fb6ce0872b876147d29fe230e7b2ba ("test/35fa71a030ca: Fix -Wunused-but-set-variable warning from clang-15")
045b8c5b183859fbc481ec7ab6b818442cc57ab0 ("test/submit-reuse: Fix -Wunused-but-set-variable warning from clang-15")
Let's install the latest clang version to get better issue coverage.
By default, the Ubuntu apt doesn't provide it. We have to pull from
the LLVM repo. The detail is available at: https://apt.llvm.org
Link: https://github.com/axboe/liburing/pull/567
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
|
|
Fixes: 971f3cf0f134 ("Fix type for io_uring_prep_socket_direct()")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Fixes: 80b7fa86872c ("Man page tweaks")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|