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.

Sign up for GitHub

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

Jump to bottom

feat(net): 实现inet socket端口资源释放 #990

Open
1 of 2 tasks
Samuka007 opened this issue Oct 17, 2024 · 5 comments
Open
1 of 2 tasks

feat(net): 实现inet socket端口资源释放 #990

Samuka007 opened this issue Oct 17, 2024 · 5 comments
Assignees
@Samuka007
Labels
A-network Area: 网络子系统 enhancement New feature or request good first issue Good for newcomers

Comments

@Samuka007
Copy link
Member

Samuka007 commented Oct 17, 2024 •
edited
Loading

目前仅实现了inet socket的port分配，但没有很好利用raii释放port资源。考虑引入结构实现inet port资源的获取与释放。

源码位于重构后的网络子系统：kernel/src/net/socket/inet/common/port.rs

  • 实现TCP Port释放
  • 实现UDP Port释放
@Samuka007 Samuka007 added enhancement New feature or request good first issue Good for newcomers A-network Area: 网络子系统 labels Oct 17, 2024
@Samuka007 Samuka007 self-assigned this Oct 17, 2024
@dragonosbot dragonosbot added the needs-triage 这个问题可能需要分类处理。如果已经完成分类，请移除它。 label Oct 17, 2024
@Samuka007 Samuka007 removed the needs-triage 这个问题可能需要分类处理。如果已经完成分类，请移除它。 label Oct 17, 2024
@Samuka007 Samuka007 mentioned this issue Oct 17, 2024
Draft
13 tasks
@ywh555hhh
Copy link

Can I try?

@fslongjin
Copy link
Member

Can I try?

@dragonos-community/sig-network

这个目前新的重构解决了吗？还是说，仍然存在这个问题？

@Samuka007
Copy link
Member Author

Can I try?

@DragonOS-Community/sig-network

这个目前新的重构解决了吗？还是说，仍然存在这个问题？

未解决的，是在新的重构中可以完善的地方！非常欢迎贡献~

@ywh555hhh
Copy link

Can I try?

@dragonos-community/sig-network

这个目前新的重构解决了吗？还是说，仍然存在这个问题？

准备演讲前还在 github 上回复我太 respect 了 @fslongjin

@Samuka007 Samuka007 linked a pull request Nov 18, 2024 that will close this issue
fix(net): TCP Resources and add syscall print, change to use latest smoltcp #1032
Merged
@Samuka007
Copy link
Member Author

Fix TCP in #1032

@Samuka007 Samuka007 removed a link to a pull request Nov 18, 2024
fix(net): TCP Resources and add syscall print, change to use latest smoltcp #1032
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Samuka007 Samuka007

Labels
A-network Area: 网络子系统 enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Samuka007 @fslongjin @ywh555hhh @dragonosbot