Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

期待该项目有更好的发展,提一些建议 #91

Closed
OuOumm opened this issue May 8, 2024 · 6 comments · Fixed by #128
Closed

期待该项目有更好的发展,提一些建议 #91

OuOumm opened this issue May 8, 2024 · 6 comments · Fixed by #128

Comments

@OuOumm
Copy link

OuOumm commented May 8, 2024

开机自启动安排一下,主要是这个。

gui的话,内存占用可以少一点,或者说,分离出来,可以退出gui之后,服务依旧运行

还有就是多平台的支持。

其次就是链接会存在不稳定的情况,gui页面显示链接正常,但实际上通过远程桌面连接是连接不上的,提示无法连接服务器

日志也建议完善一下,以便更好的反馈内容。

@KKRainbow
Copy link
Member

gui的话,内存占用可以少一点,或者说,分离出来,可以退出gui之后,服务依旧运行

这个可能比较困难,windows 上的内存占用主要来源于 WinTun

其次就是链接会存在不稳定的情况,gui页面显示链接正常,但实际上通过远程桌面连接是连接不上的,提示无法连接服务器

这个可能跟 MTU 有关系,我已经做了修复,后续可以测试下

其他的建议都非常合理,我会尽快完成

@OuOumm
Copy link
Author

OuOumm commented May 8, 2024

gui的话,内存占用可以少一点,或者说,分离出来,可以退出gui之后,服务依旧运行

这个可能比较困难,windows 上的内存占用主要来源于 WinTun

其次就是链接会存在不稳定的情况,gui页面显示链接正常,但实际上通过远程桌面连接是连接不上的,提示无法连接服务器

这个可能跟 MTU 有关系,我已经做了修复,后续可以测试下

其他的建议都非常合理,我会尽快完成

gui的话,内存占用可以少一点,或者说,分离出来,可以退出gui之后,服务依旧运行

这个可能比较困难,windows 上的内存占用主要来源于 WinTun

其次就是链接会存在不稳定的情况,gui页面显示链接正常,但实际上通过远程桌面连接是连接不上的,提示无法连接服务器

这个可能跟 MTU 有关系,我已经做了修复,后续可以测试下

其他的建议都非常合理,我会尽快完成

幸苦!

@lanthora
Copy link

lanthora commented May 8, 2024

windows 上的内存占用主要来源于 WinTun

Wintun 内存占用多可能是由于 WintunStartSession 启动时给的 Ring Buffer 太大了,可以考虑设置成 WINTUN_MIN_RING_CAPACITY.

@KKRainbow

@KKRainbow
Copy link
Member

windows 上的内存占用主要来源于 WinTun

Wintun 内存占用多可能是由于 WintunStartSession 启动时给的 Ring Buffer 太大了,可以考虑设置成 WINTUN_MIN_RING_CAPACITY.

@KKRainbow

我看看。不知道好不好改,tun 是直接用的 rust-tun 库弄得,不知道有没有暴露接口出来。

感谢提示哈

@m1m1sha
Copy link
Contributor

m1m1sha commented May 8, 2024

rust-tun

https://github.com/meh/rust-tun/blob/master/src/platform/windows/device.rs#L52

用的 MAX_RING_CAPACITY ,是写死的

@lanthora
Copy link

lanthora commented May 8, 2024

rust-tun

https://github.com/meh/rust-tun/blob/master/src/platform/windows/device.rs#L52

用的 MAX_RING_CAPACITY ,是写死的

不怎么了解 Rust,不过看到这行代码附近的几行写死的变量人都麻了

@KKRainbow KKRainbow linked a pull request Jun 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants