From: shike liu <liushike123@gmail.com>
To: netdev-bot+sinfo@kernel.org
Cc: netdev@vger.kernel.org, jmaloy@redhat.com,
tung.quang.nguyen@est.tech, davem@davemloft.net,
edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
horms@kernel.org, shuah@kernel.org,
tipc-discussion@lists.sourceforge.net,
linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC net-next 0/2] tipc: notify pending connects of peer node loss
Date: Wed, 30 Sep 2026 10:08:34 +0800 [thread overview]
Message-ID: <CAL9mPHuyOrK4WN4ZSWRFSYWpeTQBGjqXLDiy_Wr9AtEahL9KYw@mail.gmail.com> (raw)
In-Reply-To: <CAL9mPHsf_10zX4Z_2WFHNJob7Tkfnaw6sp03q2+ujnTygn1Ucg@mail.gmail.com>
The issue was first observed on actual network devices operating in a
cluster. The client was waiting in connect() for the server to accept the
connection. When contact with the peer node was lost, the pending
connect did not return promptly and instead continued waiting for its
connection timeout.
I subsequently investigated the TIPC connection setup and node-loss
paths with LLM assistance. The analysis showed that pending connects
were not registered in the peer node's conn_sks list, so
node_lost_contact() did not notify them. The initial report came from
the observed device failure, rather than an LLM or static-analysis scan.
I then reproduced the behavior using the same final selftest on the
unmodified base kernel and on that base with this series applied, in
x86_64 QEMU/KVM guests. The tests use two network namespaces connected
by veth pairs and disable TIPC Ethernet bearers to simulate loss of
connectivity.
On the unmodified kernel, 8 tests passed and 10 failed. With the series
applied, all 18 tests passed, covering SOCK_STREAM and SOCK_SEQPACKET.
The node-loss cases check completion within two seconds and the
expected EHOSTUNREACH error.
I will include this discovery background in the commit message if a
revised series is needed. I am not reposting the series solely for this
clarification.
Regards,
liushike
shike liu <liushike123@gmail.com> 于2026年9月30日周三 10:01写道:
>
> Hi,
>
> > - How the issue was discovered, e.g. hit in production, hit during
> > development, syzbot report, manual code inspection, LLM or static
> > analysis tool scan.
>
> The issue was first observed on actual network devices operating in a
> cluster. The client was waiting in connect() for the server to accept the
> connection. When contact with the peer node was lost, the pending
> connect did not return promptly and instead continued waiting for its
> connection timeout.
>
> I subsequently investigated the TIPC connection setup and node-loss
> paths with LLM assistance. The analysis showed that pending connects
> were not registered in the peer node's conn_sks list, so
> node_lost_contact() did not notify them. The initial report came from
> the observed device failure, rather than an LLM or static-analysis scan.
>
> I then reproduced the behavior using the same final selftest on the
> unmodified base kernel and on that base with this series applied, in
> x86_64 QEMU/KVM guests. The tests use two network namespaces connected
> by veth pairs and disable TIPC Ethernet bearers to simulate loss of
> connectivity.
>
> On the unmodified kernel, 8 tests passed and 10 failed. With the series
> applied, all 18 tests passed, covering SOCK_STREAM and SOCK_SEQPACKET.
> The node-loss cases check completion within two seconds and the
> expected EHOSTUNREACH error.
>
> I will include this discovery background in the commit message if a
> revised series is needed. I am not reposting the series solely for this
> clarification.
>
> Regards,
> liushike
>
> <netdev-bot+sinfo@kernel.org> 于2026年9月29日周二 14:54写道:
>>
>> Hi!
>>
>> This is an automated message. This series looks like a fix, but its
>> commit messages seem to be missing some information:
>>
>> - How the issue was discovered, e.g. hit in production, hit during
>> development, syzbot report, manual code inspection, LLM or static
>> analysis tool scan.
>>
>> Please do not repost the series just to address the above. Instead,
>> reply to this email with the missing information, so that reviewers
>> can take it into account. If the series needs another revision for
>> other reasons, please include the information in the commit messages
>> then.
>>
>> The evaluation is done by an LLM so it may be wrong, if you think
>> that is the case please reply and explain.
next prev parent reply other threads:[~2026-09-30 2:08 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-29 6:47 [RFC net-next 0/2] tipc: notify pending connects of peer node loss liushike
2026-09-29 6:47 ` [RFC net-next 1/2] tipc: abort pending connects when the peer node is lost liushike
2026-10-08 6:46 ` Tung Quang Nguyen
2026-10-09 2:38 ` shike liu
2026-10-09 12:10 ` Tung Quang Nguyen
2026-09-29 6:47 ` [RFC net-next 2/2] selftests: net: cover pending TIPC connects on peer node loss liushike
2026-09-29 6:54 ` [RFC net-next 0/2] tipc: notify pending connects of " netdev-bot+sinfo
[not found] ` <CAL9mPHsf_10zX4Z_2WFHNJob7Tkfnaw6sp03q2+ujnTygn1Ucg@mail.gmail.com>
2026-09-30 2:08 ` shike liu [this message]
2026-10-08 1:51 ` shike liu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAL9mPHuyOrK4WN4ZSWRFSYWpeTQBGjqXLDiy_Wr9AtEahL9KYw@mail.gmail.com \
--to=liushike123@gmail.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=horms@kernel.org \
--cc=jmaloy@redhat.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=netdev-bot+sinfo@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=shuah@kernel.org \
--cc=tipc-discussion@lists.sourceforge.net \
--cc=tung.quang.nguyen@est.tech \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox