-
Notifications
You must be signed in to change notification settings - Fork 173
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
fs_watcher:libbpf_fs_watcher(添加了一个write系统调用) #714
Conversation
Signed-off-by: shangfan <[email protected]>
@@ -0,0 +1,63 @@ | |||
#include "vmlinux.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
文件头要加一下
@@ -0,0 +1,99 @@ | |||
#include <stdio.h> | |||
#include <unistd.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里也是文件头加一下
Signed-off-by: shangfan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
代码现在基本没有问题了,希望下个pr可以将fs_watcher项目的流水线完成
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Pull Request Template
Description/描述
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
此 pr 解决了什么问题
这个pr写了比较简易版的write的系统调用,这个礼拜会把这个代码再继续优化,之前写的代码有点问题,等修改后,和这个代码合并
Fixes # (issue)
Type of change /PR 类型
Please delete options that are not relevant.
How Has This Been Tested?/ 测试方式
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist/检查列表