Skip to content

refactor(net): the big network rebuild #977

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
Open
wants to merge 101 commits into
base: master
Choose a base branch
from
Open

refactor(net): the big network rebuild #977

wants to merge 101 commits into from

Conversation

Samuka007
Copy link
Member

@Samuka007 Samuka007 commented Oct 14, 2024 •
edited
Loading

  • 重构事件通知链
  • 使网络子系统更易读
  • 引入uevent/netlink socket
  • 使http_server可中断
  • 正确实现unix stream, seqpacket socket框架
  • 解决资源依赖导致的潜在死锁（中断由内核线程处理）
  • 部分网络调用待完善
  • 未能正常运行test-backlog
  • 未引入路由子系统
  • close 时 解除 Port 占用 feat(net): 实现inet socket资源释放 #990
  • Redis 未能正常使用客户端连接

update: 8/29/2025

  • 实现内核 Packet 抽象
  • Internet Family
    • TCP Connect 行为从loop更改到事件驱动
    • TCP Listening Backlog 实现优化
    • 中断线程的事件丢失（应该是因为搬数据的线程执行慢而错过第二次中断的唤醒）
    • Epoll Support (WIP)
    • 端口复用
    • UDP 接口实现
    • Shutdown 收发半双工关闭实现
    • IPv6
  • Unix Family
    • 封装Unix地址空间转换与序列化
    • 实现 Unix Socket 通信逻辑

Samuka007 and others added 14 commits October 10, 2024 17:53
@Samuka007
ready for merge in master (#964)
40d9375 
uevent should be format

Enum of smoltcp socket should be optimized.

need to add interface for routing subsys

actix is still not abled to run.

clean some casual added code to other places
@val213
updates uevent and make fmt
4e4ce68
@val213
新增CommonAttrGroup,基本支持sysfs下各种类型设备的uevent属性文件的读测试
178486f
@val213
修改net设备的uevent内容，使之与Linux语义一致
e67348c
@val213
删除无用注释
49000bc
@val213
feat: 支持 sysfs下 block, char, Net, rtc, tty 等类型设备的 uevent 文件读操作 (#973)
38d161c 
* 新增CommonAttrGroup,基本支持sysfs下各种类型设备的uevent属性文件的读测试

* 修改net设备的uevent内容，使之与Linux语义一致

* 删除无用注释
@val213
Merge remote-tracking branch 'upstream/feat-network-rebuild' into uev…
b781eaa 
…ent-network-rebuild
@Samuka007
Merge remote-tracking branch 'upstream/feat-network-rebuild' into fea…
2e98aea 
…t-network-rebuild
@Samuka007
can ctrl-c interupt from blocking accept
7cebb88
@Samuka007
make fmt
8fe49e1
@Samuka007
clean debug messages
bca536e
@Samuka007
Merge pull request #974 from Samuka007:feat-network-rebuild
9a1fe0f 
clean format, enable ctrl-c in accept blocking
@Samuka007
merge master
63ccb53
@Samuka007
Merge pull request #976 from Samuka007/feat-network-rebuild
91f24d4 
feat: merge master and ready for pr in master
@Samuka007 Samuka007 requested a review from fslongjin October 14, 2024 12:45
@dragonosbot
Copy link
Contributor

@Samuka007: no appropriate reviewer found, use r? to override

@github-actions github-actions bot added the enhancement New feature or request label Oct 14, 2024
@dragonosbot dragonosbot added A-driver Area: 驱动程序 A-fs Area: 文件系统 O-x86_64 Target: x86_64 S-等待审查 Status: 等待assignee以及相关方的审查。 labels Oct 14, 2024
@Samuka007 Samuka007 added the A-network Area: 网络子系统 label Oct 14, 2024
@Samuka007
Copy link
Member Author

r? @fslongjin @yuyi2439

@fslongjin fslongjin requested a review from smallcjy October 15, 2024 05:25
fslongjin
fslongjin requested changes Oct 15, 2024
View reviewed changes
Copy link
Member

@fslongjin fslongjin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

需要添加文档

@dragonosbot dragonosbot added S-等待作者修改 Status: 这正在等待作者的一些操作（例如代码更改或更多信息）。 and removed S-等待审查 Status: 等待assignee以及相关方的审查。 labels Oct 15, 2024
@Samuka007 Samuka007 marked this pull request as draft June 10, 2025 03:52
@Samuka007 Samuka007 marked this pull request as ready for review August 4, 2025 07:40
@Samuka007
Copy link
Member Author

@fslongjin cc? 虽然 unix socket 还用不了，但是原来的也只是充当给 Novashell 通信的内部实现，目前把之前逻辑错误（即会因找不到unix路径所属socket而错误创建socket的错误）去掉之后可以无问题进入 busybox。

@fslongjin
Copy link
Member

@fslongjin cc? 虽然 unix socket 还用不了，但是原来的也只是充当给 Novashell 通信的内部实现，目前把之前逻辑错误（即会因找不到unix路径所属socket而错误创建socket的错误）去掉之后可以无问题进入 busybox。

ok,后面的pr再实现这个呗

@sparkzky sparkzky force-pushed the feat-network-rebuild branch from 8e44263 to d419148 Compare August 18, 2025 04:44
@sparkzky
Copy link
Collaborator

解决了一下冲突，合进主线？
@fslongjin cc

@Samuka007 Samuka007 force-pushed the feat-network-rebuild branch from 8fb7cc2 to 2040ee3 Compare August 28, 2025 17:53
@sparkzky
Copy link
Collaborator

sparkzky commented Sep 2, 2025 •
edited
Loading

@Samuka007 @fslongjin cc
支持epoll了,但是测试程序在我那个分支，可以先把这个分支合进主线~
图片

@sparkzky sparkzky mentioned this pull request Sep 2, 2025
Open
18 tasks
@sparkzky
Copy link
Collaborator

sparkzky commented Sep 8, 2025

unix stream socket test

图片

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fslongjin fslongjin fslongjin requested changes

@smallcjy smallcjy Awaiting requested review from smallcjy

@val213 val213 Awaiting requested review from val213

Requested changes must be addressed to merge this pull request.

Assignees

@fslongjin fslongjin

Labels
A-driver Area: 驱动程序 A-fs Area: 文件系统 A-network Area: 网络子系统 O-x86_64 Target: x86_64 S-等待审查 Status: 等待assignee以及相关方的审查。 T-doc Relevant to the document team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] latest refactor中网络从阻塞中唤醒很慢
10 participants
@Samuka007 @dragonosbot @fslongjin @smallcjy @chiichen @sparkzky @val213 @1037827920 @xiaolin2004 @BrahmaMantra