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/docs: support to unroll route loop and update troubleshooting.md #158

Merged
merged 4 commits into from
Jun 27, 2023

Conversation

mzz2017
Copy link
Contributor

@mzz2017 mzz2017 commented Jun 26, 2023

Background

dae will fail to load eBPF objects under the compilation of clang-13.

FATA[0022] load eBPF objects: field TproxyWanEgress: program tproxy_wan_egress: load program: argument list too long: 1617: (bf) r2 = r6: 1618: (85) call bpf_map_loo (truncated, 992 line(s) omitted)

Checklist

Full changelogs

  • Allow defining __UNROLL_ROUTE_LOOP to unroll the route loop for fewer program instructions and shorter loading time. However, enabling this brings more memory occupation.
  • Add a related note into troubleshooting.md.
  • Add a No network after dae suspend note into troubleshooting.md.

@mzz2017 mzz2017 requested a review from a team as a code owner June 26, 2023 17:59
@dae-prow
Copy link
Contributor

dae-prow bot commented Jun 26, 2023

❌ Your branch is currently out-of-sync to main. No worry, I will fix it for you.

@mzz2017
Copy link
Contributor Author

mzz2017 commented Jun 26, 2023

图片 图片

There is no problem with clang-11 or clang-15.

@mzz2017 mzz2017 requested a review from 1715173329 June 26, 2023 18:03
Copy link
Member

@1715173329 1715173329 left a comment

Choose a reason for hiding this comment

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

Aha nice thanks!

@mzz2017 mzz2017 merged commit d60e644 into main Jun 27, 2023
14 checks passed
@mzz2017 mzz2017 deleted the chore_clang13 branch June 27, 2023 02:04
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.

None yet

3 participants