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

asm: update eBPF built-in functions #1011

Merged
merged 2 commits into from
Apr 17, 2023
Merged

Conversation

florianl
Copy link
Contributor

@florianl florianl commented Apr 12, 2023

Sync eBPF built-in functions with Linux kernel v6.2.

The format of ___BPF_FUNC_MAPPER in include/uapi/linux/bpf.h change slightly and I used a modified version of the gawk script from the comment, to generate the output:

/FN\(.+)/ {
  match($1, /\(([a-z_0-9]+),/, r)
  split(r[1], p, "_")
  printf "Fn"
  for (i in p) {
    printf "%s%s", toupper(substr(p[i], 1, 1)), substr(p[i], 2)
  }
  print ""
}

Sync eBPF built-in functions with Linux kernel v6.2.

Signed-off-by: Florian Lehner <dev@der-flo.net>
@florianl florianl requested a review from lmb April 12, 2023 19:34
@lmb
Copy link
Collaborator

lmb commented Apr 13, 2023

Should we update the scriptlet in the doc comment as well?

Commit 8a76145a2ec2a81dfe34d7ac42e8c242f095e8c8 changed the format of
___BPF_FUNC_MAPPER. Adapt matching regex to these changes.

Signed-off-by: Florian Lehner <dev@der-flo.net>
@florianl
Copy link
Contributor Author

updated the PR to include the changes to extract the data.

@lmb lmb merged commit 7e65d1d into cilium:master Apr 17, 2023
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