refactor(net): the big network rebuild #977
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
* 新增CommonAttrGroup,基本支持sysfs下各种类型设备的uevent属性文件的读测试 * 修改net设备的uevent内容，使之与Linux语义一致 * 删除无用注释
…ent-network-rebuild
…t-network-rebuild
clean format, enable ctrl-c in accept blocking
feat: merge master and ready for pr in master
|
@Samuka007: no appropriate reviewer found, use |
dragonosbot
added
A-driver
dragonosbot
added
S-等待作者修改
命名为 |
rust仓库里面写的是types 然后rust圣经（R门！/滑稽）写的类型本身也是叫做Types 之前看过一些库，没见过用 |
然后，那个文件夹里面的类型的命名也是需要修改，如果某个类型是要跟Posix规范定义的值保持一致的（对外接口的），请全部加上 |
|
tcp udp port释放未实现 |
* 重新组织代码分布 * remove unused imports * fix it back to old syscall printing style * regulate naming to posix * todo: socket close 及 port 接触占用 * fix: format check
fslongjin
reviewed
Oct 17, 2024
kernel/src/driver/net/loopback.rs
Outdated
Comment on lines
509
to
515
| //TODO: 现在先不用初始化宏进行注册，使virtonet排在网卡列表头，待网络子系统重构后再使用初始化宏并修复该bug | ||
| // #[unified_init(INITCALL_DEVICE)] | ||
| pub fn loopback_init() -> Result<(), SystemError> { | ||
| loopback_probe(); | ||
| log::debug!("Successfully init loopback device"); | ||
| return Ok(()); | ||
| } |
into feat-network-rebuild
remove unnecessary comments, rename module to posix.
去除virtio_blk, virtio_net expect错误处理，直接返回错误
* fix: 修复网卡顺序的bug * make fmt * feat: 添加默认网卡字段 * make fmt * 删掉奇怪加入的文件
…t-network-rebuild
fix(net): address family错误返回错误导致的redis跑不起来
|
重构的 PR 命名为 refactor 似乎比 feat 更合适？ |
fslongjin
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
一次自认为性价比并不十分高的网络子系统重构，但是辛苦 @val213 @smallcjy @1037827920 @Saga1718 的大力支持，愿意在三天两头构建不成功的网络重构分支下协助我一同完成重构。现网络重构分支已准备好合入主线！