[PATCH mptcp-next] Squash to "mptcp: implement tls_mptcp_ops"

Geliang Tang posted 1 patch 2 days, 20 hours ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/multipath-tcp/mptcp_net-next tags/patchew/867d350fcac80ce446ae2a756f1eec09d4cc54fc.1767522515.git.tanggeliang@kylinos.cn
include/net/mptcp.h | 2 ++
net/tls/tls_main.c  | 4 ----
2 files changed, 2 insertions(+), 4 deletions(-)
[PATCH mptcp-next] Squash to "mptcp: implement tls_mptcp_ops"
Posted by Geliang Tang 2 days, 20 hours ago
From: Geliang Tang <tanggeliang@kylinos.cn>

Fix the error reported by CI:

 CC      net/mptcp/protocol.o
  warning: net/mptcp/protocol.c:4754:21: warning: symbol 'tls_mptcp_ops' \
           was not declared. Should it be static?

Based-on: <cover.1767518836.git.tanggeliang@kylinos.cn>

Signed-off-by: Geliang Tang <tanggeliang@kylinos.cn>
---
 include/net/mptcp.h | 2 ++
 net/tls/tls_main.c  | 4 ----
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/include/net/mptcp.h b/include/net/mptcp.h
index 4cf59e83c1c5..02564eceeb7e 100644
--- a/include/net/mptcp.h
+++ b/include/net/mptcp.h
@@ -132,6 +132,8 @@ struct mptcp_pm_ops {
 	void (*release)(struct mptcp_sock *msk);
 } ____cacheline_aligned_in_smp;
 
+extern struct tls_prot_ops tls_mptcp_ops;
+
 #ifdef CONFIG_MPTCP
 void mptcp_init(void);
 
diff --git a/net/tls/tls_main.c b/net/tls/tls_main.c
index db3c9ffabd76..70fd533d5c6c 100644
--- a/net/tls/tls_main.c
+++ b/net/tls/tls_main.c
@@ -1117,10 +1117,6 @@ static struct tls_prot_ops tls_tcp_ops = {
 	.epollin_ready	= tcp_epollin_ready,
 };
 
-#ifdef CONFIG_MPTCP
-extern struct tls_prot_ops tls_mptcp_ops;
-#endif
-
 static int tls_init(struct sock *sk)
 {
 	struct tls_context *ctx;
-- 
2.51.0
Re: [PATCH mptcp-next] Squash to "mptcp: implement tls_mptcp_ops"
Posted by MPTCP CI 2 days, 19 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): Unstable: 1 failed test(s): selftest_mptcp_join 🔴
- KVM Validation: debug (except selftest_mptcp_join): Unstable: 1 failed test(s): packetdrill_mp_join 🔴
- KVM Validation: debug (only selftest_mptcp_join): Unstable: 1 failed test(s): selftest_mptcp_join 🔴
- 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/20691737340

Initiator: Patchew Applier
Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/c170b3ef73e0
Patchwork: https://patchwork.kernel.org/project/mptcp/list/?series=1038182


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: [PATCH mptcp-next] Squash to "mptcp: implement tls_mptcp_ops"
Posted by MPTCP CI 2 days, 20 hours ago
Hi Geliang,

Thank you for your modifications, that's great!

But sadly, our CI spotted some issues with it when trying to build it.

You can find more details there:

  https://github.com/multipath-tcp/mptcp_net-next/actions/runs/20691737339

Status: failure
Initiator: Patchew Applier
Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/c170b3ef73e0
Patchwork: https://patchwork.kernel.org/project/mptcp/list/?series=1038182

Feel free to reply to this email if you cannot access logs, if you need
some support to fix the error, if this doesn't seem to be caused by your
modifications or if the error is a false positive one.

Cheers,
MPTCP GH Action bot
Bot operated by Matthieu Baerts (NGI0 Core)