Skip to content
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

chore: add tcx info in BPF_PROG_TYPE_SCHED_CLS.md #104

Merged
merged 2 commits into from
Jan 3, 2025

Conversation

ianchen0119
Copy link
Contributor

No description provided.

@ianchen0119
Copy link
Contributor Author

This pull request includes an addition to the docs/linux/program-type/BPF_PROG_TYPE_SCHED_CLS.md file to document the new tcx BPF fast path with BPF link support introduced in kernel v6.6. The changes provide detailed information on the new features and relevant return codes.

Documentation updates:

  • Added a section on tcx support, including a table of program types, attach types, ELF section names, and sleepable status.
  • Included the definition of return codes for tcx programs with a code snippet.
  • Provided a link to further details on tcx from the LSFMM+BPF Summit Recap and Video.

@ianchen0119
Copy link
Contributor Author

@dylandreimerink

Could you please help to review this PR? Many Thanks!

Copy link
Collaborator

@dylandreimerink dylandreimerink left a comment

Choose a reason for hiding this comment

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

Changes look good. I have fixed CI on master, could you rebase your changes to fix CI?

@ianchen0119
Copy link
Contributor Author

@dylandreimerink done!

@dylandreimerink
Copy link
Collaborator

dylandreimerink commented Jan 2, 2025

Thanks, sorry for the slow replies, holidays and all. Now that CI can actually run, the spellcheck isn't happy. I suggest adding tc, tcx and Netkit to the dictionary. LSFMM is not really a common word, so I suggest wrapping it in <nospell></nospell> to ignore it.

@ianchen0119
Copy link
Contributor Author

Hi @dylandreimerink

I just updated the PR.
Would you please help to approve the CI workflow?

Thanks!

@dylandreimerink dylandreimerink merged commit bd784f6 into isovalent:master Jan 3, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants