Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
acassis closed issue #16447: [BUG] TCP keep alive sockets not working URL: https://github.com/apache/nuttx/issues/16447 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
linguini1 commented on issue #16447: URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2914840431 > You can submit the PR directly. Thank you very much. Thank you! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
zhhyu7 commented on issue #16447:
URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2914823618
> > [@linguini1](https://github.com/linguini1) could you test the following
patch?
> > ```
> > tcp_timer:
> > /* Yes.. Has the retry count expired? */
> >
> > if (conn->keepretries >= conn->keepcnt)
> > {
> > /* Yes... stop the network monitor, closing the
> >* connection and all sockets associated with the
> >* connection.
> >*/
> >
> > + devif_conn_event(conn->dev, TCP_ABORT,
conn->sconn.list);
> > tcp_stop_monitor(conn, TCP_ABORT);
> > }
> > ```
>
> Thank you [@zhhyu7](https://github.com/zhhyu7), this resolved the issue! I
am very appreciative of your help. Would you be able to either submit the patch
as a PR or would you like me to submit this patch?
@linguini1 You can submit the PR directly. Thank you very much.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
linguini1 commented on issue #16447:
URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2913398083
> [@linguini1](https://github.com/linguini1) could you test the following
patch?
>
> ```
> tcp_timer:
> /* Yes.. Has the retry count expired? */
>
> if (conn->keepretries >= conn->keepcnt)
> {
> /* Yes... stop the network monitor, closing the
>* connection and all sockets associated with the
>* connection.
>*/
>
> + devif_conn_event(conn->dev, TCP_ABORT,
conn->sconn.list);
> tcp_stop_monitor(conn, TCP_ABORT);
> }
> ```
Thank you @zhhyu7, this resolved the issue! I am very appreciative of your
help. Would you be able to either submit the patch as a PR or would you like me
to submit this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
zhhyu7 commented on issue #16447:
URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2911596744
@linguini1 could you test the following patch?
```
void tcp_stop_monitor(FAR struct tcp_conn_s *conn, uint16_t flags)
{
DEBUGASSERT(conn != NULL && conn->dev != NULL);
/* Notify all registered callbacks */
devif_conn_event(conn->dev, flags, conn->sconn.list);
/* Stop the network monitor */
tcp_shutdown_monitor(conn, flags);
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
EliasJRH commented on issue #16447: URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2910860582 (Sorry for excessive mentions) @wengzhe you were mentioned on a what seems to be a similar issue [here](github.com/apache/nuttx/issues/13493). Do you have any advice? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
bskdany commented on issue #16447: URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2910651049 @zhhyu7 I saw you removed the tcp_callback when accepting conn timeout, do you happen to know if that change might have interfered with TCP Keepalive? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
linguini1 commented on issue #16447: URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2910641686 @wangyingdong I saw you wrote the zero-probe timer support and `tcp_xmit_probe()`. Do you have any advice which may help with this issue? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
linguini1 commented on issue #16447: URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2910621768 @a-lunev Do you have any ideas what this issue might be? I saw you've made some changes a long time ago to the networking logic, and thought you might now better than me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [I] [BUG] TCP keep alive sockets not working [nuttx]
linguini1 commented on issue #16447: URL: https://github.com/apache/nuttx/issues/16447#issuecomment-2910564953 Noticed that after the probes receive no reply, the TCP stack calls the abort mechanism indefinitely: ```console [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting [CPU0] [ 2] tcp_timer: aborted [CPU0] [ 2] tcp_timer: aborting ``` This is net/tcp/tcp_timer.c:704 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
