Commit Graph

871 Commits

Author SHA1 Message Date
ohci1 4b8f3098a5 update configure info for 2.14
Co-authored-by: Pavel Begunkov<asml.silence@gmail.com>
Co-authored-by: Guillem Jover<guillem@hadrons.org>
Co-authored-by: Jens Axboe<axboe@kernel.dk>
Co-authored-by: Khem Raj<raj.khem@gmail.com>
Co-authored-by: Michael de Lang<kingoipo@gmail.com>
Co-authored-by: David Disseldorp<ddiss@suse.de>
Co-authored-by: Ming Lei<ming.lei@redhat.com>
2026-04-24 09:24:13 +08:00
fangzhiyi18 0ae77b95ef 升级2.7
Signed-off-by: fangzhiyi18 <fangzhiyi1@huawei.com>
2024-10-21 02:53:20 +00:00
Jens Axboe 1370f5a067 Add io_uring_submit_and_wait_timeout() test
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-10-22 10:41:12 -06:00
Jens Axboe dc73e72c4e test/recv-multishot.c: check for error specifically when submitting
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-10-22 10:39:50 -06:00
Dylan Yudaken d2b58aa77c fix recv-multishot test skipping in 6.1
This test was skipping in IORING_SETUP_DEFER_TASKRUN as it was not
flushing the work queue before checking for completions.

Additionally fix the test to only skip if it is the first loop.

Fixes: eaf8ff7ca9 ("update existing tests for defer taskrun")
Signed-off-by: Dylan Yudaken <dylany@meta.com>
Link: https://lore.kernel.org/r/20221021081207.2607808-1-dylany@meta.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-10-21 03:43:31 -07:00
Pavel Begunkov f49874e782 tests: use only UDP/TCP for zc testing
The initial implementation only supports TCP and UDP, so use them for
testing.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/41f1cf670c1ac33adf263942b8ebc68f3b32001b.1666210567.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-10-19 14:51:03 -07:00
Ammar Faizi 4fe3c560f3 test/helpers: Fix clang warning
Building with clang v13.0.1 yields the following warning:

  ./helpers.h:29:42: warning: declaration of 'struct sockaddr_in' \
  will not be visible outside of this function [-Wvisibility]

  int t_bind_ephemeral_port(int fd, struct sockaddr_in *addr);
                                           ^
This is because at that point struct sockaddr_in has not been
declared yet. Fix this by including <arpa/inet> in helpers.h.

Fixes: 1e63b19a5e ("test/helpers: Add `t_bind_ephemeral_port()` function")
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
2022-10-19 04:46:03 +07:00
Pavel Begunkov e329b7c2c0 tests: fix zc support checks
We should be checking cqe->res for -EINVAL to figure out whether we
support zerocopy or not. It makes the test fail with older kernels.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/0c8bf4d5dd7135c990f4fa9232a54c8cd6cc024f.1665958020.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-10-16 17:27:57 -06:00
Jens Axboe 08fe94d13f test/ring-leak.c: kick UNIX garbage collection
Provide a more deterministic runtime by avoiding waiting for a random
GC run. This can take a long time depending on what is going on in
the host, it's been observed to even timeout the test.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-10-03 09:17:27 -06:00
Pavel Begunkov 833d9cacc9 liburing: add more zc helpers
Add a helper for zc sendmsg and also for zc send with registered buffers
(aka fixed).

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/25a5e4ff7c480cf5c68be2e223a2fa9787ee6283.1664361932.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-28 07:53:52 -06:00
Jens Axboe 6e1968b6b6 Merge branch 'test-sigfd-deadlock' of https://github.com/ddiss/liburing
* 'test-sigfd-deadlock' of https://github.com/ddiss/liburing:
  test/sigfd-deadlock: skip test if poll returns -EOPNOTSUPP
  test/sigfd-deadlock: improve error handling
2022-09-27 14:02:31 -06:00
David Disseldorp 6bbdd59b88 test/sigfd-deadlock: skip test if poll returns -EOPNOTSUPP
Some kernels may disable IORING_OP_POLL support for signalfd and Binder
file descriptors via the Stable commit fc78b2fc21f1
("io_uring: disable polling pollfree files"). Skip the test for this
case.

Signed-off-by: David Disseldorp <ddiss@suse.de>
2022-09-27 20:02:27 +02:00
Pavel Begunkov 2cc080f7e2 tests: a small fix for zc tests
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/4794c23f60a3a0f4c5f6e83af4598eca47dda68a.1664292508.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-27 09:59:25 -06:00
Dylan Yudaken 4ec9cb7044 give open-direct-pick.c a unique path
This was making make runtest-parallel flaky

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Reviewed-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220927102202.69069-4-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-27 08:14:58 -06:00
Dylan Yudaken 5450e2492a handle single issuer task registration at ring creation
IORING_SETUP_SINGLE_ISSUER now registers the task at ring creation, so
update the tests and documentation to reflect this.

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Link: https://lore.kernel.org/r/20220927102202.69069-2-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-27 08:14:58 -06:00
David Disseldorp 4d05e19aa0 test/sigfd-deadlock: improve error handling
The (cqe->res & POLLIN) check can incorrectly result in test success if
cqe->res carries a negative errno value. Fix this by explicitly checking
for (cqe->res < 0). Additionally, change return paths to use the new
enum t_test_result values.

Signed-off-by: David Disseldorp <ddiss@suse.de>
2022-09-27 15:45:13 +02:00
Pavel Begunkov 3b43779b08 tests: test async_data double-free with sendzc
Similar to send_recv.c:test_invalid().

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-26 08:33:49 -06:00
Jens Axboe 2a18c82ee3 Merge branch 'test_print_counts' of https://github.com/ddiss/liburing
* 'test_print_counts' of https://github.com/ddiss/liburing:
  test/runtests.sh: print failure / time-out counts
2022-09-23 18:58:59 -06:00
David Disseldorp 08dcad2f4b test/runtests.sh: print failure / time-out counts
Retain FAILED, SKIPPED and TIMED_OUT lists in Bash arrays so that the
counts can be dumped alongside test names. Remove RET, which can be
derived from non-zero FAILED length.

Signed-off-by: David Disseldorp <ddiss@suse.de>
2022-09-23 23:58:25 +02:00
Dylan Yudaken f2fede0965 test invalid sendmsg and recvmsg
Syzbot found a double free bug due to failure in sendmsg.
This test exposes the bug.

Reported-by: syzbot+4c597a574a3f5a251bda@syzkaller.appspotmail.com
Signed-off-by: Dylan Yudaken <dylany@fb.com>
Link: https://lore.kernel.org/r/20220923151858.968528-1-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-23 14:57:48 -06:00
Jens Axboe 7476d1204c Add IOPOLL potential leak test case
Setup ring and issue an IOPOLL read, exit process without reaping
completions.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-23 14:42:22 -06:00
Jens Axboe ce3869a442 test/rsrc_tags: turn assert into actual error logging
It's hard to debug a failure when you just get an assert, since we
don't know what the errors were or which sub-test failed. Turn a
noticed error into something legible so we get decent output.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-23 09:14:39 -06:00
Pavel Begunkov df332dd03d tests: add sendmsg_zc tests
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/d698961d4bf8f0ab3bb7ad36b42c309c3395e0c2.1663759148.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-21 09:00:26 -06:00
Pavel Begunkov 4b3c9826c0 test: fix zc tests
Decouple result from F_NOTIF/F_MORE, even failed requests may produce a
notification.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/1529ac2cc55de274f3cf96aa150f56b5d550a19c.1663759148.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-21 09:00:26 -06:00
Jens Axboe 1d71762ae4 test/msg-ring: add registered file test case
Augment the invalid fd destination test with a registered file to
test that path as well. You cannot register an io_uring fd itself,
so there's no way to use it normally.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-16 06:56:23 -06:00
Pavel Begunkov 8fbb6afb93 tests: fixup defer-taskrun.c after api change
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/2ed8db2b22f7840977bdc58b6d282be750c878bb.1662900803.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-13 09:23:20 -06:00
Pavel Begunkov 13e6a2d766 tests/zc: skip tcp w/ addr
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/77860403d6fa96a9288e037ede9dcae23a4314e2.1662404421.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 16:51:14 -06:00
Pavel Begunkov 3b85e1ff9c tests/zc: name buffer flavours
Remove duplicating tests and pass a buf index instead of dozens of
flags to specify the buffer we want to use.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/0ad14b1b4cf74ab503c32e945e150dbcbf678961.1662404421.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 16:51:14 -06:00
Pavel Begunkov 5a378cae1e tests/zc: fix udp testing
The tcp vs large_buf skip condition is not what we want and it skips udp
testing, fix it and also make sure we serialise cork requests with
IOSQE_IO_LINK as they might get executed OOO.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/f85ddc0dd0712ce17a5e7f73639c55d9c612cedf.1662404421.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 16:51:14 -06:00
Pavel Begunkov b396b81021 tests/zc: use io_uring for rx
Recieve via io_uring, this simplifies code by removing forking.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/fa126293c94da4ca445bc606fcc6ff16c832eabe.1662404421.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 16:51:14 -06:00
Pavel Begunkov 3d87e3546c tests/zc: move send size calc into do_test_inet_send
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/e9e818b93fd3c0f969ebf1c4a0869edaadb161dd.1662404421.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 16:51:14 -06:00
Ammar Faizi c742391836 test/ringbuf-read: Delete .ringbuf-read.%d before exit
Running test/ringbuf-read.t leaves untracked files in git status:

  Untracked files:
    (use "git add <file>..." to include in what will be committed)
          .ringbuf-read.163521
          .ringbuf-read.163564
          .ringbuf-read.163605
          .ringbuf-read.163648

Make sure we unlink it properly. While in there, fix the exit code,
use T_EXIT_*.

v2:
  - Use T_EXIT_* for exit code (comment from Alviro).

Cc: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Link: https://lore.kernel.org/r/20220905093126.376009-1-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 11:41:44 -06:00
Dylan Yudaken 7116b59ee4 file-verify test: log if short read
This test assumes any success is a full read. If it was a short read the
test would still fail (as verification would fail) but tracking down the
reason can be annoying.
For now we can just log if it's short and fail at that point.

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Tested-by: Ammar Faizi <ammarfaizi2@gnuweeb.org> # Build
Link: https://lore.kernel.org/r/20220905132258.1858915-12-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 11:41:24 -06:00
Dylan Yudaken 586a4b30db overflow: add tests
Add tests that verify that overflow conditions behave appropriately.
Specifically:
 * if overflow is continually flushed, then CQEs should arrive mostly in
 order to prevent starvation of some completions
 * if CQEs are dropped due to GFP_ATOMIC allocation failures it is
 possible to terminate cleanly. This is not tested by default as it
 requires debug kernel config, and also has system-wide effects

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Tested-by: Ammar Faizi <ammarfaizi2@gnuweeb.org> # Build
Link: https://lore.kernel.org/r/20220905132258.1858915-11-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 11:41:24 -06:00
Dylan Yudaken 0689e82366 add a defer-taskrun test
Add a test specifically for IORING_SETUP_DEFER_TASKRUN

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Tested-by: Ammar Faizi <ammarfaizi2@gnuweeb.org> # Build
Link: https://lore.kernel.org/r/20220905132258.1858915-7-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 11:41:24 -06:00
Dylan Yudaken eaf8ff7ca9 update existing tests for defer taskrun
Add defer_taskrun to a few choice tests that can expose some bad
behaviour.
This requires adding some io_uring_get_events calls to make sure deferred
tasks are run

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Tested-by: Ammar Faizi <ammarfaizi2@gnuweeb.org> # Build
Link: https://lore.kernel.org/r/20220905132258.1858915-6-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 11:41:24 -06:00
Dylan Yudaken 014f46e794 add a t_probe_defer_taskrun helper function for tests
Help tests to determine if they can use IORING_SETUP_DEFER_TASKRUN

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Tested-by: Ammar Faizi <ammarfaizi2@gnuweeb.org> # Build
Link: https://lore.kernel.org/r/20220905132258.1858915-5-dylany@fb.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-05 11:41:24 -06:00
Pavel Begunkov e64e29627d test: test iowq zc sends
Add tests exercising io-wq paths with zc by setting IOSQE_ASYNC.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/8562fc9c8860b0e6b92d20e4f773ce3bde7d9137.1662116617.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:53 -06:00
Pavel Begunkov 4d730231ee zc: adjust sendzc to the simpler uapi
We update the zerocopy API, reflect it in liburing, this includes
removing the whole notification story, updating headers and fixing
up tests.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/f56ab1638b9704d5745082e4a8a4b610a784d62f.1662116617.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:53 -06:00
Pavel Begunkov c35692ba50 tests: verify that send addr is copied when async
The kernel should not be touching timespec after returning from submit
syscall, make sure it's not reloaded after a request goes async.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/67fa1af95c4565ef0e9f43833b67d6c4d50d66f6.1662116617.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:53 -06:00
Ammar Faizi 311f98573e t/recv-msgall-stream: Don't use a static port number
Don't use a static port number. It might already be in use, resulting
in a test failure. Use an ephemeral port to make this test reliable.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-13-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi d953fe4b77 t/232c93d07b74: Don't use a static port number
Don't use a static port number. It might already be in use, resulting
in a test failure. Use an ephemeral port to make this test reliable.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-12-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi e6bf9923ed t/recv-msgall: Don't use a static port number
Don't use a static port number. It might already be in use, resulting
in a test failure. Use an ephemeral port to make this test reliable.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-11-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi d1d9388856 t/shutdown: Don't use a static port number
Don't use a static port number. It might already be in use, resulting
in a test failure. Use an ephemeral port to make this test reliable.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-10-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi eeb33bea98 t/connect: Don't use a static port number
Don't use a static port number. It might already be in use, resulting
in a test failure. Use an ephemeral port to make this test reliable.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-9-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi c8b211f561 t/socket: Don't use a static port number
Don't use a static port number. It might already be in use, resulting
in a test failure. Use an ephemeral port to make this test reliable.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-8-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi 1e94284cae t/files-exit-hang-poll: Don't brute force the port number
Don't brute force the port number, use `t_bind_ephemeral_port()`,
much simpler and reliable for choosing a port number that is not
in use.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-7-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi 5aefb7c06f t/socket-rw-offset: Don't brute force the port number
Don't brute force the port number, use `t_bind_ephemeral_port()`,
much simpler and reliable for choosing a port number that is not
in use.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-6-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi 481e258c22 t/socket-rw-eagain: Don't brute force the port number
Don't brute force the port number, use `t_bind_ephemeral_port()`,
much simpler and reliable for choosing a port number that is not
in use.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-5-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00
Ammar Faizi 1a8defca17 t/socket-rw: Don't brute force the port number
Don't brute force the port number, use `t_bind_ephemeral_port()`,
much simpler and reliable for choosing a port number that is not
in use.

Cc: Dylan Yudaken <dylany@fb.com>
Cc: Facebook Kernel Team <kernel-team@fb.com>
Cc: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Tested-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
Link: https://lore.kernel.org/r/20220902071153.3168814-4-ammar.faizi@intel.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2022-09-02 05:56:14 -06:00