Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update README.md #134

Merged
merged 1 commit into from
Jan 5, 2023
Merged

Update README.md #134

merged 1 commit into from
Jan 5, 2023

Conversation

ur4t
Copy link
Contributor

@ur4t ur4t commented Jan 5, 2023

No description provided.

@ur4t
Copy link
Contributor Author

ur4t commented Jan 5, 2023

For further more information, /proc/kallsyms can be a supplementary source of traceable functions iovisor/bcc#1540. But that conflicts with kprobe-multi and makes kernel complain trace_kprobe: Could not probe notrace function BALABALA.

@ur4t ur4t force-pushed the pr/ur4t/update-readme branch from af14b00 to 9ef93fd Compare January 5, 2023 04:03
brb
brb requested changes Jan 5, 2023
View reviewed changes
Copy link
Member

@brb brb left a comment

Choose a reason for hiding this comment

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

Thanks! Could you sign-off your commit (git commit --amend -s && git push -f)?

@ur4t
Update README.md
6918215 
Signed-off-by: ur4t <46435411+ur4t@users.noreply.github.com>
@ur4t ur4t force-pushed the pr/ur4t/update-readme branch from 9ef93fd to 6918215 Compare January 5, 2023 11:42
@ur4t ur4t requested a review from brb January 5, 2023 11:45
brb
brb approved these changes Jan 5, 2023
View reviewed changes
Copy link
Member

@brb brb left a comment

Choose a reason for hiding this comment

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

Thanks!

@brb brb merged commit 5f42ebe into cilium:main Jan 5, 2023
@ur4t ur4t deleted the pr/ur4t/update-readme branch January 5, 2023 12:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@brb brb brb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ur4t @brb