使用tcp参数 创建的连接未关闭

[chenfeixyz]

if config.TCP {
conn, err := tcpraw.Dial("tcp", remoteAddr)
if err != nil {
return nil, errors.Wrap(err, "tcpraw.Dial()")
}
return kcp.NewConn(remoteAddr, block, config.DataShard, config.ParityShard, conn)
}
kcp-go ownConn 属性是false 未看到关闭的地方 查看进程fd数量一直在增加

[xtaci]

在std/copy.go中，会处理关闭。

[xtaci]

另外tcp的实现并不是很完美，可以用udp_to_raw

[chenfeixyz]

在std/copy.go中，会处理关闭。

conn, err := tcpraw.Dial("tcp", remoteAddr) 是这个conn没有释放
kcp.NewConn 创建的UDPSession ownConn值是false Close的时候不会调用conn的Close

kcp-go sess.go Close函数
if s.l != nil { // belongs to listener
s.l.closeSession(s.remote)
return nil
} else if s.ownConn { // client socket close
return s.conn.Close()
} else {
return nil
}

[xtaci]

即使这里没关闭，那么garbage collection也会close掉这个链接，只是不够实时而已，不会出现fd不断增加的情况。

[xtaci]

可能问题出在tcpraw,但需要分析一下

[chenfeixyz]

xtaci/tcpraw@7b3a1df @chenfeixyz 试试这个update，看是否重现， go get -u github.com/xtaci/tcpraw@7b3a1df734aa77772ebe655c067aea6f6c044a58

还是不行 tcpraw.Dial 创建的conn内有协程在引用自己 不会被GC标记 可以在kcp-go UDPSession里加个SetOwnConn函数吗

[xtaci]

现在是只要TCPconn不被引用，自然会gc 然后关闭掉socket 。注意我的setfinalizer改动 E-mail: ***@***.*** Blog: https://github.com/xtaci

…

On Sun, Aug 18, 2024 at 13:31 chenfeixyz ***@***.***> wrote: ***@***.*** <xtaci/tcpraw@7b3a1df> @chenfeixyz <https://github.com/chenfeixyz> 试试这个update，看是否重现， go get -u ***@***.*** 还是不行 tcpraw.Dial 创建的conn内有协程在引用自己 不会被GC标记 可以在kcp-go UDPSession里加个SetOwnConn函数吗 — Reply to this email directly, view it on GitHub <#965 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAR45ZJKGZWYO5CSINH3QBDZSAWSJAVCNFSM6AAAAABMVMGBQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJVGEZDAMZSHA> . You are receiving this because you commented.Message ID: ***@***.***>

[xtaci]

这个是肯定会Close掉socket的

[xtaci]

tagged to : https://github.com/xtaci/tcpraw/releases/tag/v1.2.26

我在休假，估计要下周才能具体的debug，不过思路肯定是这个，让他GC就可以了。

kcptun的deps也更新到了v1.2.26

[chenfeixyz]

现在是只要TCPconn不被引用，自然会gc 然后关闭掉socket 。注意我的setfinalizer改动 E-mail: @.*** Blog: https://github.com/xtaci
…
On Sun, Aug 18, 2024 at 13:31 chenfeixyz @.> wrote: @. <xtaci/tcpraw@7b3a1df> @chenfeixyz https://github.com/chenfeixyz 试试这个update，看是否重现， go get -u @.*** 还是不行 tcpraw.Dial 创建的conn内有协程在引用自己 不会被GC标记 可以在kcp-go UDPSession里加个SetOwnConn函数吗 — Reply to this email directly, view it on GitHub <#965 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAR45ZJKGZWYO5CSINH3QBDZSAWSJAVCNFSM6AAAAABMVMGBQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJVGEZDAMZSHA . You are receiving this because you commented.Message ID: @.***>

func (conn *tcpConn) cleaner() 这个函数是单独协程运行的 一直在引用tcpConn本身

[xtaci]

现在是只要TCPconn不被引用，自然会gc 然后关闭掉socket 。注意我的setfinalizer改动 E-mail: @.*** Blog: https://github.com/xtaci
…
On Sun, Aug 18, 2024 at 13:31 chenfeixyz @.> wrote: _@**._ <xtaci/tcpraw@7b3a1df> @chenfeixyz https://github.com/chenfeixyz 试试这个update，看是否重现， go get -u _@_.* 还是不行 tcpraw.Dial 创建的conn内有协程在引用自己 不会被GC标记 可以在kcp-go UDPSession里加个SetOwnConn函数吗 — Reply to this email directly, view it on GitHub <#965 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAR45ZJKGZWYO5CSINH3QBDZSAWSJAVCNFSM6AAAAABMVMGBQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJVGEZDAMZSHA . You are receiving this because you commented.Message ID: @.***>

func (conn *tcpConn) cleaner() 这个函数是单独协程运行的 一直在引用tcpConn本身

这个没有关系，内部各种交叉引用tcpConn都可以，但只要TCPConn没有被外部引用了，会触发关闭操作。

[xtaci]

kcptun的tcpraw.Dial后，返回的是*TCPConn，这个一旦没有被kcptun引用了，会触发SetFinalizer中的wrapper.Close，进而关闭socket fd

[xtaci]

commit d5a4974
@chenfeixyz 试下

主动关闭和被动关闭都具有了。

[chenfeixyz]

commit d5a4974 @chenfeixyz 试下

主动关闭和被动关闭都具有了。

fd数量现在正常了

tcp_linux.go 文件 cleaner函数 time.NewTicker 没有加Stop

[xtaci]

https://github.com/xtaci/tcpraw/tree/v1.2.27

[xtaci]

ca79329
还有这个commit,之前因为有udp2raw，没有花多少心思在tcp模拟

[xtaci]

还包含了iptables的重置