-
Notifications
You must be signed in to change notification settings - Fork 185
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
pkg/gadgets: Add ntfs3 to trace fsslower. #1879
Conversation
Note that, there is a bug when both NTFS modules are loaded or compiled, as francis@pwmachine:/mnt$ grep ntfs_file_open /proc/kallsyms
0000000000000000 t ntfs_file_open [ntfs]
0000000000000000 t ntfs_file_open [ntfs3]
francis@pwmachine:/mnt$ mount | grep /mnt
/home/francis/Codes/kinvolk/foo.img on /mnt type ntfs3 (rw,relatime,uid=0,gid=0,iocharset=utf8)
...
RUNTIME.CONTAINERNAME RUNTIME.CONTAINERIMAG… PID COMM T BYTES OFFSET LAT FILE
235883 dd R 512 0 4 foo
235883 dd W 512 0 9 bar
235883 dd R 512 512 0 foo
235883 dd W 512 512 1 bar
235883 dd R 512 1024 0 foo
235883 dd W 512 1024 0 bar
# Events for open are missing as the probe is installed for ntfs_file_open for ntfs module and not ntfs3. I will discuss this with |
668803b
to
128da15
Compare
Signed-off-by: Francis Laniel <flaniel@linux.microsoft.com>
Signed-off-by: Francis Laniel <flaniel@linux.microsoft.com>
128da15
to
03985cb
Compare
This pull request has been automatically marked as stale because it has not had recent activity. |
@alban @mauriciovasquezbernal This PR is pretty straightforward to review and the upstream problem should now have been addressed. Can I get reviews? |
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 from code inspection. I don't have any ntfs3 partition available to test it now.
Thank you for the review! |
Hi.
This PR enables
trace fsslower
to tracentfs3
:Best regards.