[RFC mptcp-next v14 00/10] NVME over MPTCP

Geliang Tang posted 10 patches 6 days, 14 hours ago
Only 9 patches received!
There is a newer version of this series
drivers/nvme/host/tcp.c                       | 107 ++++-
drivers/nvme/target/configfs.c                |   1 +
drivers/nvme/target/tcp.c                     | 152 ++++++-
include/linux/nvme.h                          |   1 +
include/net/mptcp.h                           |  31 ++
net/mptcp/sockopt.c                           | 151 +++++++
tools/testing/selftests/net/mptcp/Makefile    |   1 +
tools/testing/selftests/net/mptcp/config      |   8 +
.../testing/selftests/net/mptcp/mptcp_lib.sh  |  12 +
.../testing/selftests/net/mptcp/mptcp_nvme.sh | 399 ++++++++++++++++++
10 files changed, 840 insertions(+), 23 deletions(-)
create mode 100755 tools/testing/selftests/net/mptcp/mptcp_nvme.sh
[RFC mptcp-next v14 00/10] NVME over MPTCP
Posted by Geliang Tang 6 days, 14 hours ago
From: Geliang Tang <tanggeliang@kylinos.cn>

v14:
- Patch 1:
  - Add const struct nvmet_fabrics_ops *ops back to struct nvmet_tcp_proto,
    instead of using queue->port->nport->tr_ops
  - Add protocol validation
	"if (port->proto->protocol != newsock->sk->sk_protocol)"
    when allocating a new queue
  - Add out_put_port label for error path cleanup
- Patch 3:
  - Drop all "if (sk->sk_protocol != IPPROTO_MPTCP)" checks in MPTCP helpers
- Patch 6:
  - Drop all "if (sk->sk_protocol != IPPROTO_MPTCP)" checks in MPTCP helpers
- Patch 7:
  - Drop "/dev/nvme*cn1" from device discovery loop, only check
    "/dev/nvme*n1" to ensure fio tests multipath head device
  - Per sashiko's comment, "../../../subsystems/${nqn}" does not work.
- Drop patch 12 in v13.

v13:
 - Patch 1
   - adds kref reference counting to struct nvmet_tcp_port
 - Patch 2
   - add nvmet_tcp_done_recv_pdu to use tr_ops from nport structure
     instead of hardcoded nvmet_tcp_ops (moved from v12 Patch 1)
 - Patch 4
   - split mptcp_sock_set_tos into __mptcp_sock_set_tos (internal) and
     mptcp_sock_set_tos (get rcv_tos from first subflow)
   - add protocol validation to all MPTCP helpers
 - Patch 7
   - export __mptcp_sock_set_tos
   - add protocol validation to mptcp_sock_set_syncnt
 - Patch 12 (new)
   - fix module unload race with concurrent sysfs controller deletion
 Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1779104752.git.tanggeliang@kylinos.cn/

v12:
 - Patch 1 (new):
   - Add nvmet_tcp_done_recv_pdu to use tr_ops from nport structure instead
     of hardcoded nvmet_tcp_ops
 - Patch 2:
   - Remove RCU protection for proto access
   - Store proto pointer directly in struct nvmet_tcp_queue instead of
     struct nvmet_tcp_port
   - Determine proto based on port->sock->sk->sk_protocol during queue
     allocation
   - Delete port->proto field from struct nvmet_tcp_port
   - Remove RCU annotations and rcu_dereference() for proto access
 - Patch 3:
   - Add nvmet_mptcp_registered flag to track successful MPTCP transport
     registration
   - Only unregister MPTCP transport during module exit if registration
     succeeded
   - Guard MODULE_ALIAS("nvmet-transport-4") with #ifdef CONFIG_MPTCP
 - Patch 4:
   - Remove unnecessary sock_hold()/sock_put() pairs in MPTCP helpers
   - Remove redundant priority >= 0 check in sync_socket_options()
 - Patch 8:
   - Move trap cleanup EXIT before init() to ensure cleanup runs on early
     failure
   - Export variables (nqn, path, port, etc.) immediately upon definition
 - Patch 9:
   - Skip iopolicy setting gracefully when iopolicy sysfs file does not
     exist (kernel without NVMe multipath support)
 Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1779087443.git.tanggeliang@kylinos.cn/

v11:
 Patch 2 (new):
 - Add RCU protection for host_iface validation to fix a pre-existing
   use-after-free issue when validating network interface names.
 Patch 3:
 - Add RCU protection for queue->port access in nvmet_tcp_alloc_cmd
   (previously missing).
 - Cache proto pointer in nvmet_tcp_done_recv_pdu before releasing
   RCU lock.
 Patch 4:
 - Remove nvmet_unregister_transport(&nvmet_tcp_ops) on MPTCP registration
   failure (MPTCP is optional, TCP continues to work).
 Patch 5:
 - Update MPTCP helper functions to iterate over all subflows using
   mptcp_for_each_subflow().
 - Add sock_hold() with explanatory comment for concurrent subflow closure
   protection.
 - Fix priority synchronization in sync_socket_options: change condition
   from priority > 0 to priority >= 0 to allow priority 0.
 Patch 9:
 - Simplify validate_params: use regex ^[1-4]$ for path validation.
 - Remove tc_args quotes in init() to allow proper word splitting for netem
   parameters.
 Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1778997507.git.tanggeliang@kylinos.cn/

v10:
 Patch 1 (new):
 - Add Fixes tag to the commit that checks return value of
   nvmet_tcp_set_queue_sock.
 Patch 2:
 - Fix RCU read lock release issue in nvmet_tcp_done_recv_pdu:
   move rcu_read_unlock() after nvmet_req_init().
 - Fix RCU read lock release issue in nvmet_tcp_set_queue_sock:
   cache proto pointer before releasing RCU lock.
 - Add missing NULL checks for queue->port in nvmet_tcp_alloc_cmd,
   nvmet_tcp_try_peek_pdu and nvmet_tcp_tls_handshake.
 - Add __rcu annotation to queue->port in struct nvmet_tcp_queue.
 - Use rcu_access_pointer() instead of rcu_dereference() in
   nvmet_tcp_destroy_port_queues.
 - Remove redundant kfree_rcu() in nvmet_tcp_remove_port, use kfree()
   since synchronize_rcu() already guarantees safety.
 Patch 4:
 - Add lock_sock_nested(ssk, SINGLE_DEPTH_NESTING) to all MPTCP helpers
   to avoid lockdep warnings.
 - Fix mptcp_sock_no_linger to properly set linger on subflow inside the
   lock.
 Patch 8:
 - Move init before trap cleanup to prevent cleanup errors when early
   exit occurs.
 - Fix usage text: change default path value from 4 to 1 to match actual
   behavior.
 - Fix break 2 to break (only one loop level).
 Patch 9:
 - Change grep -B 5 to grep (without -B) to avoid matching host NVMe
   devices.
 Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1778919284.git.tanggeliang@kylinos.cn/

v9:
 Patch 1:
  - add NULL pointer checks for RCU dereference in nvmet_tcp_done_recv_pdu
  and nvmet_tcp_set_queue_sock.
  - clear queue->port using rcu_assign_pointer and add synchronize_rcu in
  nvmet_tcp_destroy_port_queues.
  - use kfree_rcu for port structure in nvmet_tcp_remove_port.
 Patch 2:
  - change module init order, make MPTCP registration optional to prevent
  UAF.
 Patch 3:
  - fix mptcp_sock_set_priority to save config on main socket first, use
  READ_ONCE and sock_hold.
  - fix mptcp_sock_no_linger to use READ_ONCE and sock_hold, call
  sock_no_linger on ssk.
  - fix mptcp_sock_set_tos to use READ_ONCE and sock_hold.
 Patch 4:
  - remove unnecessary RCU protection for ctrl->proto (points to static
  data).
  - remove rcu_head from nvme_tcp_ctrl, use kfree instead of kfree_rcu.
 Patch 6:
  - add msk->icsk_syn_retries check before calling tcp_sock_set_syncnt in
  sync_socket_options.
  - fix mptcp_sock_set_syncnt to always return 0 after saving config.
 Patch 7:
  - split selftests into two patches.
  - fix tool check order (call mptcp_lib_check_tools before temp_file
  creation).
  - add unshare -m in cleanup to prevent configfs mount leakage.
  - improve device name parsing from nvme connect output.
 Patch 8:
  - add iopolicy tests with set_io_policy function and error checking.
  - add loss parameter for packet loss simulation (delay 5ms loss 0.5%).
 Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1778837549.git.tanggeliang@kylinos.cn/

v8:
 - address comments reported by ai-review for v7.
 - add RCU protection for queue->port on target side.
 - add RCU protection ctrl->proto on host side.
 - check !msk->first instead of "IS_ERR(msk->first)".
 - fix return value of mptcp_sock_set_syncnt.
 - update selftest.
 - fix CI error: "[SKIP] Could not run all tests without nvme".
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1775047736.git.tanggeliang@kylinos.cn/

v7:
 - address comments reported by ai-review.
 - change sockops in nvmet_tcp_port and nvme_tcp_ctrl as a pointer.
 - add null checks for queue->port->sockops in nvmet_tcp_set_queue_sock.
 - add inline for mptcp_sock_set_priority and mptcp_sock_set_tos in
   mptcp.h
 - use "ssk = msk->first" instead of "ssk = __mptcp_nmpc_sk(msk)" in
   mptcp_sock_set_priority, mptcp_sock_no_linger and mptcp_sock_set_tos.
 - drop sk_is_tcp in nvmet_tcp_done_recv_pdu
 - move ctrl->sockops setting before nvme_init_ctrl in
   nvme_tcp_alloc_ctrl
 - define nvme_mptcp_ctrl_ops
 - add MODULE_ALIAS("nvme-mptcp")
 - add more CONFIG_MPTCP checks
 - update selftest
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1774952107.git.tanggeliang@kylinos.cn/

v6:
 - introduce nvmet_tcp_sockops and nvme_tcp_sockops structures
 - fix set_reuseaddr, set_nodelay and set_syncnt, add sockopt_seq_inc
 calls, only set the first subflow, and synchronize to other subflows in
 sync_socket_options
 - Add implementations for no_linger, set_priority and set_tos
 - This version no longer depends on the "mptcp: fix stall because of
 data_ready" series of fixes
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1774862875.git.tanggeliang@kylinos.cn/

v5:
 - address comments reported by ai-review: set msk->nodelay to true in
   mptcp_sock_set_nodelay, set sk->sk_reuse to ssk->sk_reuse in
   mptcp_sock_set_reuseaddr, add mptcp_nvme.sh to TEST_PROGS, and adjust
   the order of patches.
 - remove TLS-related options from .allowed_opts of
   nvme_mptcp_transport.
 - some cleanups for selftest.
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1773374342.git.tanggeliang@kylinos.cn/

v4:
 - a new patch to set nvme iopolicy as Nilay suggested.
 - resend all set to trigger AI review.
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1772683110.git.tanggeliang@kylinos.cn/

v3:
 - update the implementation of sock_set_nodelay: originally it only set
the first subflow, but now it sets every subflow.
 - use sk_is_msk helper in this set.
 - update the selftest to perform testing under a multi-interface
environment.
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1770627071.git.tanggeliang@kylinos.cn/

v2:
 - Patch 1 fixes the timeout issue reported in v1, thanks to Paolo and Gang
Yan for their help.
 - Patch 5 implements an MPTCP-specific sock_set_syncnt helper.
 - Link: https://patchwork.kernel.org/project/mptcp/cover/cover.1764152990.git.tanggeliang@kylinos.cn/

This series (previously named "MPTCP support to 'NVME over TCP'") had three
RFC versions sent to Hannes in May, with subsequent revisions based on his
input. Following that, I initiated the process of upstreaming the dependent
"implement mptcp read_sock" series to the main MPTCP repository, which has
been merged into net-next recently.

Cc: zhenwei pi <zhenwei.pi@linux.dev>
Cc: Hui Zhu <zhuhui@kylinos.cn>
Cc: Gang Yan <yangang@kylinos.cn>

Geliang Tang (10):
  nvmet-tcp: define target tcp_proto struct
  nvmet-tcp: register target mptcp transport
  nvmet-tcp: implement target mptcp proto
  nvme-tcp: define host tcp_proto struct
  nvme-tcp: register host mptcp transport
  nvme-tcp: implement host mptcp proto
  selftests: mptcp: add nvme over mptcp test
  selftests: mptcp: nvme: add iopolicy tests
  nvmet-tcp: check return value of set_queue_sock
  nvme-tcp: add RCU protection for host_iface validation

 drivers/nvme/host/tcp.c                       | 107 ++++-
 drivers/nvme/target/configfs.c                |   1 +
 drivers/nvme/target/tcp.c                     | 152 ++++++-
 include/linux/nvme.h                          |   1 +
 include/net/mptcp.h                           |  31 ++
 net/mptcp/sockopt.c                           | 151 +++++++
 tools/testing/selftests/net/mptcp/Makefile    |   1 +
 tools/testing/selftests/net/mptcp/config      |   8 +
 .../testing/selftests/net/mptcp/mptcp_lib.sh  |  12 +
 .../testing/selftests/net/mptcp/mptcp_nvme.sh | 399 ++++++++++++++++++
 10 files changed, 840 insertions(+), 23 deletions(-)
 create mode 100755 tools/testing/selftests/net/mptcp/mptcp_nvme.sh

-- 
2.53.0
Re: [RFC mptcp-next v14 00/10] NVME over MPTCP
Posted by MPTCP CI 5 days, 20 hours ago
Hi Geliang,

Thank you for your modifications, that's great!

Our CI did some validations and here is its report:

- KVM Validation: normal (except selftest_mptcp_join): Success! ✅
- KVM Validation: normal (only selftest_mptcp_join): Success! ✅
- KVM Validation: debug (except selftest_mptcp_join): Unstable: 1 failed test(s): packetdrill_dss ⚠️ 
- KVM Validation: debug (only selftest_mptcp_join): Success! ✅
- KVM Validation: btf-normal (only bpftest_all): Success! ✅
- KVM Validation: btf-debug (only bpftest_all): Success! ✅
- Task: https://github.com/multipath-tcp/mptcp_net-next/actions/runs/26123437144

Initiator: Matthieu Baerts (NGI0)
Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/c19be69b7661
Patchwork: https://patchwork.kernel.org/project/mptcp/list/?series=1097007


If there are some issues, you can reproduce them using the same environment as
the one used by the CI thanks to a docker image, e.g.:

    $ cd [kernel source code]
    $ docker run -v "${PWD}:${PWD}:rw" -w "${PWD}" --privileged --rm -it \
        --pull always mptcp/mptcp-upstream-virtme-docker:latest \
        auto-normal

For more details:

    https://github.com/multipath-tcp/mptcp-upstream-virtme-docker


Please note that despite all the efforts that have been already done to have a
stable tests suite when executed on a public CI like here, it is possible some
reported issues are not due to your modifications. Still, do not hesitate to
help us improve that ;-)

Cheers,
MPTCP GH Action bot
Bot operated by Matthieu Baerts (NGI0 Core)
Re: [RFC mptcp-next v14 00/10] NVME over MPTCP
Posted by Geliang Tang 6 days, 9 hours ago
Hi Matt,

Not sure why, but this version didn't trigger CI and sashiko. Could you
help trigger them for me?

Sorry, the version has been iterated many times, since sashiko keeps
complaining.

Thanks,
-Geliang

On Tue, 2026-05-19 at 11:24 +0800, Geliang Tang wrote:
> From: Geliang Tang <tanggeliang@kylinos.cn>
> 
> v14:
> - Patch 1:
>   - Add const struct nvmet_fabrics_ops *ops back to struct
> nvmet_tcp_proto,
>     instead of using queue->port->nport->tr_ops
>   - Add protocol validation
> 	"if (port->proto->protocol != newsock->sk->sk_protocol)"
>     when allocating a new queue
>   - Add out_put_port label for error path cleanup
> - Patch 3:
>   - Drop all "if (sk->sk_protocol != IPPROTO_MPTCP)" checks in MPTCP
> helpers
> - Patch 6:
>   - Drop all "if (sk->sk_protocol != IPPROTO_MPTCP)" checks in MPTCP
> helpers
> - Patch 7:
>   - Drop "/dev/nvme*cn1" from device discovery loop, only check
>     "/dev/nvme*n1" to ensure fio tests multipath head device
>   - Per sashiko's comment, "../../../subsystems/${nqn}" does not
> work.
> - Drop patch 12 in v13.
> 
> v13:
>  - Patch 1
>    - adds kref reference counting to struct nvmet_tcp_port
>  - Patch 2
>    - add nvmet_tcp_done_recv_pdu to use tr_ops from nport structure
>      instead of hardcoded nvmet_tcp_ops (moved from v12 Patch 1)
>  - Patch 4
>    - split mptcp_sock_set_tos into __mptcp_sock_set_tos (internal)
> and
>      mptcp_sock_set_tos (get rcv_tos from first subflow)
>    - add protocol validation to all MPTCP helpers
>  - Patch 7
>    - export __mptcp_sock_set_tos
>    - add protocol validation to mptcp_sock_set_syncnt
>  - Patch 12 (new)
>    - fix module unload race with concurrent sysfs controller deletion
>  Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1779104752.git.tanggeliang@kylinos.cn/
> 
> v12:
>  - Patch 1 (new):
>    - Add nvmet_tcp_done_recv_pdu to use tr_ops from nport structure
> instead
>      of hardcoded nvmet_tcp_ops
>  - Patch 2:
>    - Remove RCU protection for proto access
>    - Store proto pointer directly in struct nvmet_tcp_queue instead
> of
>      struct nvmet_tcp_port
>    - Determine proto based on port->sock->sk->sk_protocol during
> queue
>      allocation
>    - Delete port->proto field from struct nvmet_tcp_port
>    - Remove RCU annotations and rcu_dereference() for proto access
>  - Patch 3:
>    - Add nvmet_mptcp_registered flag to track successful MPTCP
> transport
>      registration
>    - Only unregister MPTCP transport during module exit if
> registration
>      succeeded
>    - Guard MODULE_ALIAS("nvmet-transport-4") with #ifdef CONFIG_MPTCP
>  - Patch 4:
>    - Remove unnecessary sock_hold()/sock_put() pairs in MPTCP helpers
>    - Remove redundant priority >= 0 check in sync_socket_options()
>  - Patch 8:
>    - Move trap cleanup EXIT before init() to ensure cleanup runs on
> early
>      failure
>    - Export variables (nqn, path, port, etc.) immediately upon
> definition
>  - Patch 9:
>    - Skip iopolicy setting gracefully when iopolicy sysfs file does
> not
>      exist (kernel without NVMe multipath support)
>  Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1779087443.git.tanggeliang@kylinos.cn/
> 
> v11:
>  Patch 2 (new):
>  - Add RCU protection for host_iface validation to fix a pre-existing
>    use-after-free issue when validating network interface names.
>  Patch 3:
>  - Add RCU protection for queue->port access in nvmet_tcp_alloc_cmd
>    (previously missing).
>  - Cache proto pointer in nvmet_tcp_done_recv_pdu before releasing
>    RCU lock.
>  Patch 4:
>  - Remove nvmet_unregister_transport(&nvmet_tcp_ops) on MPTCP
> registration
>    failure (MPTCP is optional, TCP continues to work).
>  Patch 5:
>  - Update MPTCP helper functions to iterate over all subflows using
>    mptcp_for_each_subflow().
>  - Add sock_hold() with explanatory comment for concurrent subflow
> closure
>    protection.
>  - Fix priority synchronization in sync_socket_options: change
> condition
>    from priority > 0 to priority >= 0 to allow priority 0.
>  Patch 9:
>  - Simplify validate_params: use regex ^[1-4]$ for path validation.
>  - Remove tc_args quotes in init() to allow proper word splitting for
> netem
>    parameters.
>  Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1778997507.git.tanggeliang@kylinos.cn/
> 
> v10:
>  Patch 1 (new):
>  - Add Fixes tag to the commit that checks return value of
>    nvmet_tcp_set_queue_sock.
>  Patch 2:
>  - Fix RCU read lock release issue in nvmet_tcp_done_recv_pdu:
>    move rcu_read_unlock() after nvmet_req_init().
>  - Fix RCU read lock release issue in nvmet_tcp_set_queue_sock:
>    cache proto pointer before releasing RCU lock.
>  - Add missing NULL checks for queue->port in nvmet_tcp_alloc_cmd,
>    nvmet_tcp_try_peek_pdu and nvmet_tcp_tls_handshake.
>  - Add __rcu annotation to queue->port in struct nvmet_tcp_queue.
>  - Use rcu_access_pointer() instead of rcu_dereference() in
>    nvmet_tcp_destroy_port_queues.
>  - Remove redundant kfree_rcu() in nvmet_tcp_remove_port, use kfree()
>    since synchronize_rcu() already guarantees safety.
>  Patch 4:
>  - Add lock_sock_nested(ssk, SINGLE_DEPTH_NESTING) to all MPTCP
> helpers
>    to avoid lockdep warnings.
>  - Fix mptcp_sock_no_linger to properly set linger on subflow inside
> the
>    lock.
>  Patch 8:
>  - Move init before trap cleanup to prevent cleanup errors when early
>    exit occurs.
>  - Fix usage text: change default path value from 4 to 1 to match
> actual
>    behavior.
>  - Fix break 2 to break (only one loop level).
>  Patch 9:
>  - Change grep -B 5 to grep (without -B) to avoid matching host NVMe
>    devices.
>  Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1778919284.git.tanggeliang@kylinos.cn/
> 
> v9:
>  Patch 1:
>   - add NULL pointer checks for RCU dereference in
> nvmet_tcp_done_recv_pdu
>   and nvmet_tcp_set_queue_sock.
>   - clear queue->port using rcu_assign_pointer and add
> synchronize_rcu in
>   nvmet_tcp_destroy_port_queues.
>   - use kfree_rcu for port structure in nvmet_tcp_remove_port.
>  Patch 2:
>   - change module init order, make MPTCP registration optional to
> prevent
>   UAF.
>  Patch 3:
>   - fix mptcp_sock_set_priority to save config on main socket first,
> use
>   READ_ONCE and sock_hold.
>   - fix mptcp_sock_no_linger to use READ_ONCE and sock_hold, call
>   sock_no_linger on ssk.
>   - fix mptcp_sock_set_tos to use READ_ONCE and sock_hold.
>  Patch 4:
>   - remove unnecessary RCU protection for ctrl->proto (points to
> static
>   data).
>   - remove rcu_head from nvme_tcp_ctrl, use kfree instead of
> kfree_rcu.
>  Patch 6:
>   - add msk->icsk_syn_retries check before calling
> tcp_sock_set_syncnt in
>   sync_socket_options.
>   - fix mptcp_sock_set_syncnt to always return 0 after saving config.
>  Patch 7:
>   - split selftests into two patches.
>   - fix tool check order (call mptcp_lib_check_tools before temp_file
>   creation).
>   - add unshare -m in cleanup to prevent configfs mount leakage.
>   - improve device name parsing from nvme connect output.
>  Patch 8:
>   - add iopolicy tests with set_io_policy function and error
> checking.
>   - add loss parameter for packet loss simulation (delay 5ms loss
> 0.5%).
>  Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1778837549.git.tanggeliang@kylinos.cn/
> 
> v8:
>  - address comments reported by ai-review for v7.
>  - add RCU protection for queue->port on target side.
>  - add RCU protection ctrl->proto on host side.
>  - check !msk->first instead of "IS_ERR(msk->first)".
>  - fix return value of mptcp_sock_set_syncnt.
>  - update selftest.
>  - fix CI error: "[SKIP] Could not run all tests without nvme".
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1775047736.git.tanggeliang@kylinos.cn/
> 
> v7:
>  - address comments reported by ai-review.
>  - change sockops in nvmet_tcp_port and nvme_tcp_ctrl as a pointer.
>  - add null checks for queue->port->sockops in
> nvmet_tcp_set_queue_sock.
>  - add inline for mptcp_sock_set_priority and mptcp_sock_set_tos in
>    mptcp.h
>  - use "ssk = msk->first" instead of "ssk = __mptcp_nmpc_sk(msk)" in
>    mptcp_sock_set_priority, mptcp_sock_no_linger and
> mptcp_sock_set_tos.
>  - drop sk_is_tcp in nvmet_tcp_done_recv_pdu
>  - move ctrl->sockops setting before nvme_init_ctrl in
>    nvme_tcp_alloc_ctrl
>  - define nvme_mptcp_ctrl_ops
>  - add MODULE_ALIAS("nvme-mptcp")
>  - add more CONFIG_MPTCP checks
>  - update selftest
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1774952107.git.tanggeliang@kylinos.cn/
> 
> v6:
>  - introduce nvmet_tcp_sockops and nvme_tcp_sockops structures
>  - fix set_reuseaddr, set_nodelay and set_syncnt, add sockopt_seq_inc
>  calls, only set the first subflow, and synchronize to other subflows
> in
>  sync_socket_options
>  - Add implementations for no_linger, set_priority and set_tos
>  - This version no longer depends on the "mptcp: fix stall because of
>  data_ready" series of fixes
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1774862875.git.tanggeliang@kylinos.cn/
> 
> v5:
>  - address comments reported by ai-review: set msk->nodelay to true
> in
>    mptcp_sock_set_nodelay, set sk->sk_reuse to ssk->sk_reuse in
>    mptcp_sock_set_reuseaddr, add mptcp_nvme.sh to TEST_PROGS, and
> adjust
>    the order of patches.
>  - remove TLS-related options from .allowed_opts of
>    nvme_mptcp_transport.
>  - some cleanups for selftest.
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1773374342.git.tanggeliang@kylinos.cn/
> 
> v4:
>  - a new patch to set nvme iopolicy as Nilay suggested.
>  - resend all set to trigger AI review.
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1772683110.git.tanggeliang@kylinos.cn/
> 
> v3:
>  - update the implementation of sock_set_nodelay: originally it only
> set
> the first subflow, but now it sets every subflow.
>  - use sk_is_msk helper in this set.
>  - update the selftest to perform testing under a multi-interface
> environment.
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1770627071.git.tanggeliang@kylinos.cn/
> 
> v2:
>  - Patch 1 fixes the timeout issue reported in v1, thanks to Paolo
> and Gang
> Yan for their help.
>  - Patch 5 implements an MPTCP-specific sock_set_syncnt helper.
>  - Link:
> https://patchwork.kernel.org/project/mptcp/cover/cover.1764152990.git.tanggeliang@kylinos.cn/
> 
> This series (previously named "MPTCP support to 'NVME over TCP'") had
> three
> RFC versions sent to Hannes in May, with subsequent revisions based
> on his
> input. Following that, I initiated the process of upstreaming the
> dependent
> "implement mptcp read_sock" series to the main MPTCP repository,
> which has
> been merged into net-next recently.
> 
> Cc: zhenwei pi <zhenwei.pi@linux.dev>
> Cc: Hui Zhu <zhuhui@kylinos.cn>
> Cc: Gang Yan <yangang@kylinos.cn>
> 
> Geliang Tang (10):
>   nvmet-tcp: define target tcp_proto struct
>   nvmet-tcp: register target mptcp transport
>   nvmet-tcp: implement target mptcp proto
>   nvme-tcp: define host tcp_proto struct
>   nvme-tcp: register host mptcp transport
>   nvme-tcp: implement host mptcp proto
>   selftests: mptcp: add nvme over mptcp test
>   selftests: mptcp: nvme: add iopolicy tests
>   nvmet-tcp: check return value of set_queue_sock
>   nvme-tcp: add RCU protection for host_iface validation
> 
>  drivers/nvme/host/tcp.c                       | 107 ++++-
>  drivers/nvme/target/configfs.c                |   1 +
>  drivers/nvme/target/tcp.c                     | 152 ++++++-
>  include/linux/nvme.h                          |   1 +
>  include/net/mptcp.h                           |  31 ++
>  net/mptcp/sockopt.c                           | 151 +++++++
>  tools/testing/selftests/net/mptcp/Makefile    |   1 +
>  tools/testing/selftests/net/mptcp/config      |   8 +
>  .../testing/selftests/net/mptcp/mptcp_lib.sh  |  12 +
>  .../testing/selftests/net/mptcp/mptcp_nvme.sh | 399
> ++++++++++++++++++
>  10 files changed, 840 insertions(+), 23 deletions(-)
>  create mode 100755 tools/testing/selftests/net/mptcp/mptcp_nvme.sh
Re: [RFC mptcp-next v14 00/10] NVME over MPTCP
Posted by Matthieu Baerts 6 days, 3 hours ago
Hi Geliang,

19 May 2026 17:43:06 Geliang Tang <geliang@kernel.org>:

> Not sure why, but this version didn't trigger CI and sashiko. Could you
> help trigger them for me?

It looks like patchew is missing one patch:

https://patchew.org/MPTCP/cover.1779159524.git.tanggeliang@kylinos.cn/

Not sure why, I received it.

But sashiko did a review:

https://sashiko.dev/#/patchset/cover.1779159524.git.tanggeliang%40kylinos.cn

I can force the CI to run when I'm back on my computer, if you still need the CI.

> Sorry, the version has been iterated many times, since sashiko keeps
> complaining.

Are these complains often correct? If not, simply reply on the mailing list.
Or add a note in the commit message.

Cheers,
Matt