| 58ae1bdd | 30-Oct-2024 |
Ankit Kumar <ankit.kumar@samsung.com> |
stdinc.h: move epoll header over here
Change-Id: Iaee83f78f3586548ded3c54af862011c380d0d75 Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk
stdinc.h: move epoll header over here
Change-Id: Iaee83f78f3586548ded3c54af862011c380d0d75 Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25395 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
show more ...
|
| 186b109d | 20-Aug-2024 |
Jim Harris <jim.harris@samsung.com> |
env: add SPDK_ENV_NUMA_ID_ANY and replace socket_id with numa_id
We will try to avoid further proliferation of "SOCKET_ID" to refer to a NUMA socket ID moving forward, and just use "NUMA_ID" to avoi
env: add SPDK_ENV_NUMA_ID_ANY and replace socket_id with numa_id
We will try to avoid further proliferation of "SOCKET_ID" to refer to a NUMA socket ID moving forward, and just use "NUMA_ID" to avoid confusion with TCP sockets.
Change all of the existing in-tree SPDK_ENV_SOCKET_ID_ANY uses to SPDK_ENV_NUMA_ID_ANY, but keep the old #define around, at least for now. Also change all 'socket_id' parameters to 'numa_id'.
We still have spdk_env_get_socket_id(), we will need to keep this but next patch will add spdk_env_get_numa_id().
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: Idc31c29e32b708c24d88f9c6fecaf9a99e34ba1e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24607 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Reviewed-by: Ben Walker <ben@nvidia.com> Community-CI: Mellanox Build Bot
show more ...
|
| 9475ec8d | 20-Aug-2024 |
Jim Harris <jim.harris@samsung.com> |
sock: remove "socket" from "numa_socket_id"
We will just use "numa_id" to refer to a NUMA socket ID (also called NUMA node ID). This will eliminate any confusion over conflicts with "socket" used to
sock: remove "socket" from "numa_socket_id"
We will just use "numa_id" to refer to a NUMA socket ID (also called NUMA node ID). This will eliminate any confusion over conflicts with "socket" used to describe TCP sockets.
These APIs were all added since last release, so we can change this freely.
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: Ifb8591c1066a61a78c31ad0e40745a59abb2759f Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24606 Community-CI: Mellanox Build Bot Reviewed-by: Ben Walker <ben@nvidia.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
| 2b46c17d | 16-Aug-2024 |
Jim Harris <jim.harris@samsung.com> |
sock: change socket_id to int32_t
These changes were added since last release, so it's safe to just change them now without breaking API/ABI.
I mistakenly used uint32_t for the socket_id, when in f
sock: change socket_id to int32_t
These changes were added since last release, so it's safe to just change them now without breaking API/ABI.
I mistakenly used uint32_t for the socket_id, when in fact DPDK and our env layer really treat it as an int. This mostly shows up with the SPDK_ENV_SOCKET_ID_ANY that is defined as -1, which forces a lot of annoying casting if we're trying to pass this value through a uint32_t interface.
This socket_id concept will be getting plumbed into nvme, bdev and nvmf layers, so instead of proliferating the uint32_t socket_id there, just change sock layer to use int32_t instead.
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: I573ba7f9aeed3b216bef72f39fc671c9c6688d3a Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24577 Reviewed-by: Ben Walker <ben@nvidia.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
| 227b8322 | 09-Aug-2024 |
Marcin Spiewak <marcin.spiewak@intel.com> |
module/sock: free addr info before return
Fixed issue found by Coverity. In some cases, uring_sock_create() and posix_sock_create() functions were not freeing resources before exiting. Now it is fix
module/sock: free addr info before return
Fixed issue found by Coverity. In some cases, uring_sock_create() and posix_sock_create() functions were not freeing resources before exiting. Now it is fixed.
Change-Id: I2fef551a1384335597e12aa57a9cd2ce3686c4c2 Signed-off-by: Marcin Spiewak <marcin.spiewak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24486 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com>
show more ...
|
| 22744fa0 | 05-Jul-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
sock/uring: support src_{addr,port} in connect()
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I29283d79abb186467e2489d5cef99ec3c8768742 Reviewed-on: https://review.spdk.io/ger
sock/uring: support src_{addr,port} in connect()
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I29283d79abb186467e2489d5cef99ec3c8768742 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24067 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Tomasz Zawadzki <tomasz@tzawadzki.com>
show more ...
|
| 72c41fdd | 10-Jul-2024 |
Jim Harris <jim.harris@samsung.com> |
sock: add spdk_sock_get_numa_socket_id
Elsewhere in SPDK/DPDK, we always just say "socket_id", but since this is the sock module for TCP sockets, we clarify the name with "numa".
Signed-off-by: Jim
sock: add spdk_sock_get_numa_socket_id
Elsewhere in SPDK/DPDK, we always just say "socket_id", but since this is the sock module for TCP sockets, we clarify the name with "numa".
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: I792c29e4db5094c88b6ced5f5e017cdee31c8fb8 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24124 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
| dfef7970 | 05-Jul-2024 |
Jim Harris <jim.harris@samsung.com> |
sock: add spdk_sock_get_interface_name
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: Ibccfac4350afba1e687edca40139316809ec1d00 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/
sock: add spdk_sock_get_interface_name
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: Ibccfac4350afba1e687edca40139316809ec1d00 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24081 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 7923828f | 11-Jul-2024 |
Jim Harris <jim.harris@samsung.com> |
util: add spdk_net_getaddr
This enables reuse of the vast bulk of the getaddr code that was previously duplicated in the posix and uring sock modules.
Signed-off-by: Jim Harris <jim.harris@samsung.
util: add spdk_net_getaddr
This enables reuse of the vast bulk of the getaddr code that was previously duplicated in the posix and uring sock modules.
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: I0d8d1de9907850013736b1766019649c0eddb5f5 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24143 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
| 9645ea13 | 03-Jul-2024 |
Jim Harris <jim.harris@samsung.com> |
util: move module/sock/sock_kernel.h contents to net.c
This header file contained helper functions that both posix and uring sock modules were using to operate on file descriptors. Promote them to t
util: move module/sock/sock_kernel.h contents to net.c
This header file contained helper functions that both posix and uring sock modules were using to operate on file descriptors. Promote them to the util library in the newly formed net.c file, and rename the functions as follows:
get_addr_str => spdk_net_get_address_string sock_is_loopback => spdk_net_is_loopback
Signed-off-by: Jim Harris <jim.harris@samsung.com> Change-Id: I89b6e766ad8cf272afcefb8bf5a87ad3b1db2d6b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24047 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
| 719d03c6 | 02-Jul-2024 |
Karl Bonde Torp <k.torp@samsung.com> |
sock/uring: only register net impl if supported
Change-Id: Iaadf37caae0dc03f88b52ac8c1c124372784c736 Signed-off-by: Karl Bonde Torp <k.torp@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/
sock/uring: only register net impl if supported
Change-Id: Iaadf37caae0dc03f88b52ac8c1c124372784c736 Signed-off-by: Karl Bonde Torp <k.torp@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23978 Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
| 29f02008 | 11-Apr-2024 |
Krzysztof Goreczny <krzysztof.goreczny@dell.com> |
sock: Add interrupt APIs for socket groups
Add spdk_sock_group_register_interrupt() API for setting interrupts on the socket groups. Only POSIX and SSL sockets are supported.
Change-Id: I4201de7650
sock: Add interrupt APIs for socket groups
Add spdk_sock_group_register_interrupt() API for setting interrupts on the socket groups. Only POSIX and SSL sockets are supported.
Change-Id: I4201de7650cff8f97e0cb7d4ad4d2c31a7e9a79f Signed-off-by: Krzysztof Goreczny <krzysztof.goreczny@dell.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22744 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Boris Glimcher <Boris.Glimcher@emc.com> Community-CI: Mellanox Build Bot
show more ...
|
| 7521dc6f | 09-Apr-2024 |
Krzysztof Karas <krzysztof.karas@intel.com> |
sock: replace sock impl priorities
Instead of using socket priorities, use a macro to set the default implementation. POSIX is the simplest impl compared to other sockets in SPDK and it should be us
sock: replace sock impl priorities
Instead of using socket priorities, use a macro to set the default implementation. POSIX is the simplest impl compared to other sockets in SPDK and it should be used as default (POSIX is most likely to be present at all times and has the minimal requirements to work).
Change-Id: I8f0dc47777183154773ac6905aa9306418657e06 Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22139 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 4fabc831 | 11-Apr-2024 |
Marcin Spiewak <marcin.spiewak@intel.com> |
sock/uring: make 'buf' 8-byte aligned
In spdk_uring_sock structure, a padding is needed to keep 'buf' aligned to 8-bytes. 'struct cmsghdr' is mapped to this buffer in uring_sock_group_impl_add_sock(
sock/uring: make 'buf' 8-byte aligned
In spdk_uring_sock structure, a padding is needed to keep 'buf' aligned to 8-bytes. 'struct cmsghdr' is mapped to this buffer in uring_sock_group_impl_add_sock() function, and while first element of this control message header has a size of 8 bytes, the whole structure must be 8-byte aligned, otherwise UBSAN will generate runtime error.
This patch is part of the fix for github issue 3090.
Change-Id: I4f0b99b4cec23f541024f2d4bbcc2ab49db6ed05 Signed-off-by: Marcin Spiewak <marcin.spiewak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22741 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
| 355312bf | 26-Jan-2024 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
so_ver: increase all major versions after SPDK 24.01
To allow SO_MINOR updates on LTS for the whole year it is supported, the major version for all components needs to be increased. This is to preve
so_ver: increase all major versions after SPDK 24.01
To allow SO_MINOR updates on LTS for the whole year it is supported, the major version for all components needs to be increased. This is to prevent scenario where two versions exists with matching versions, but conflicting ABI. Ex. Next SPDK release adds an API call increasing the minor version, then LTS needs just a subset of those additions.
Increasing major so version after LTS, allows the future releases to update versions as needed. Yet allowing LTS to increase minor version separately.
This patch: - increases SO_VER by 1 for all components - resets SO_MINOR to 0 for all components
Due to patch below being introduced after v24.01 code freeze, bdev lib version should not be increased: 61623c5ca9 bdev: add spdk_bdev_io padding in place of io_submit_ch
Short reference to how the versions were changed: MAX=$(git grep "SO_VER := " | cut -d" " -f 3 | sort -ubnr | head -1) for((i=$MAX;i>0;i-=1)); do find . -name "Makefile" -exec \ sed -i -e "s/SO_VER := $i\$/SO_VER := $(($i+1))/g" {} +; done find . -name "Makefile" -exec \ sed -i -e "s/SO_MINOR := .*/SO_MINOR := 0/g" {} +
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: I0166d6be6c17bbbf985d48b5dfcb36f1d4af1b48 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21582 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| b0f7294d | 04-Aug-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
sock/uring: move connection fail code to a function
This code was duplicated in three places, so it makes sense to extract it to a function.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
sock/uring: move connection fail code to a function
This code was duplicated in three places, so it makes sense to extract it to a function.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I5b03afbea6786cdb7f92b11def55619104f6bd96 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19339 Reviewed-by: Jim Harris <jim.harris@gmail.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
| 908ad5ff | 06-Jul-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
sock/uring: check if socket was closed after sock_abort_requests()
It's possible that user closed the socket in response to a callback from one of the aborted requests. In that case, it'd be freed,
sock/uring: check if socket was closed after sock_abort_requests()
It's possible that user closed the socket in response to a callback from one of the aborted requests. In that case, it'd be freed, so we need to take care not to touch it anymore.
Fixes #3089.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I9707b26a6232a20b2a18d09e07dd91c6874aca13 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18993 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Karol Latecki <karol.latecki@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
show more ...
|
| a14bc376 | 19-May-2023 |
Ziye Yang <ziye.yang@intel.com> |
uring: Fix the code broken when zero-copy send is enabled.
Without this fix, the program will hang in the target side. If we use SPDK NVMe-of target with tcp transport with uring,
./scripts/rpc.py
uring: Fix the code broken when zero-copy send is enabled.
Without this fix, the program will hang in the target side. If we use SPDK NVMe-of target with tcp transport with uring,
./scripts/rpc.py sock_impl_set_options --enable-zerocopy-send-server -i uring
Then we use nvmf initiator to test, the program hangs.
Signed-off-by: Ziye Yang <ziye.yang@intel.com> Change-Id: Iea12ee0396008481e8317424d1f405eb03634ed4 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18236 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 84c04174 | 27-Jan-2023 |
Ben Walker <benjamin.walker@intel.com> |
sock: Remove spdk_sock_readv_async
Prefer to use spdk_sock_recv_zcopy instead.
Change-Id: Ia1920c2cbda4f787094e939ad91e52b4b4bb4ac2 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on
sock: Remove spdk_sock_readv_async
Prefer to use spdk_sock_recv_zcopy instead.
Change-Id: Ia1920c2cbda4f787094e939ad91e52b4b4bb4ac2 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16609 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
| ece3730c | 28-Apr-2023 |
Ben Walker <benjamin.walker@intel.com> |
sock/uring: Rename spdk_sock_task_type to uring_task_type
This is an io_uring specific enum and it is not public.
Change-Id: Ia4fc8c8991afa198311c178b28fe791f2b35b441 Signed-off-by: Ben Walker <ben
sock/uring: Rename spdk_sock_task_type to uring_task_type
This is an io_uring specific enum and it is not public.
Change-Id: Ia4fc8c8991afa198311c178b28fe791f2b35b441 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17892 Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 257bd5f6 | 13-Dec-2022 |
Ben Walker <benjamin.walker@intel.com> |
sock/uring: Implement recv next
This is a moderately efficient implementation that submits asynchronous reads using pooled memory in the kernel.
Future implementations can use fixed buffers and mul
sock/uring: Implement recv next
This is a moderately efficient implementation that submits asynchronous reads using pooled memory in the kernel.
Future implementations can use fixed buffers and multi-shot recv, plus true kernel zero copy support, to improve.
Change-Id: Ife3ab3c3d7236231ba5329b0d18c6ba0e44a08f8 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16108 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
| eac5d183 | 27-Apr-2023 |
Ben Walker <benjamin.walker@intel.com> |
sock/uring: Use a recv instead of poll to detect data on the socket
If we post a recv with buffer select, but never post buffers, we get an ENOBUFS status whenever the socket has data. This can be u
sock/uring: Use a recv instead of poll to detect data on the socket
If we post a recv with buffer select, but never post buffers, we get an ENOBUFS status whenever the socket has data. This can be used instead of posting the pollin and transitions us closer to our final design.
Change-Id: I6995be1348340bc2d19c064a7e4743d56a09272f Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17834 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
| efb0a5c4 | 10-Apr-2023 |
Ben Walker <benjamin.walker@intel.com> |
sock/uring: Don't submit POLLERR at all.
Just keep a recv for the ERRQUEUE running at all times instead.
Change-Id: I6018190dd8c304f21dffe36861bd4fe503a0ad1b Signed-off-by: Ben Walker <benjamin.wal
sock/uring: Don't submit POLLERR at all.
Just keep a recv for the ERRQUEUE running at all times instead.
Change-Id: I6018190dd8c304f21dffe36861bd4fe503a0ad1b Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17596 Reviewed-by: Jim Harris <james.r.harris@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
| 1cd5bf92 | 10-Apr-2023 |
Ben Walker <benjamin.walker@intel.com> |
sock/uring: Resubmit pollin operations when they finish
Instead of looping through the whole set of sockets on every poll to see if any need a new pollin operation submitted, just resubmit one any t
sock/uring: Resubmit pollin operations when they finish
Instead of looping through the whole set of sockets on every poll to see if any need a new pollin operation submitted, just resubmit one any time one finishes.
Change-Id: Iac600a2996fc5355b53ce74eb5651c2045556604 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17595 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
| 5f1e93c8 | 24-Apr-2023 |
Ben Walker <benjamin.walker@intel.com> |
sock/uring: Improve error handling cases
Break the error handing up by opcode since it is going to be different for each. For this patch, this duplicates a lot of code. But the cases will all diverg
sock/uring: Improve error handling cases
Break the error handing up by opcode since it is going to be different for each. For this patch, this duplicates a lot of code. But the cases will all diverge later in this series.
Change-Id: I12d5d88c41ca4b7063293dd143000212fe6d2156 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17736 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|