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.

Sign up for GitHub

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

Jump to bottom

Small bpf cleanups #11688

Merged
merged 2 commits into from
May 27, 2020
Merged

Small bpf cleanups #11688

merged 2 commits into from
May 27, 2020

Conversation

tklauser
Copy link
Member

Please see individual commit messages.

@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label May 26, 2020
@tklauser tklauser requested a review from a team May 26, 2020 09:53
@tklauser tklauser requested a review from a team as a code owner May 26, 2020 09:53
@tklauser
Copy link
Member Author

test-me-please

pchaigno
pchaigno approved these changes May 26, 2020
View reviewed changes
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

Nice!

bpf/lib/ipv6.h Outdated Show resolved Hide resolved
@tklauser
bpf: use macro argument in GET_PREFIX
478d3aa 
Rename `GET_PREFIX`s argument to `prefix` so the macro argument is used
instead of relying on the caller declaring a variable `prefix`.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser
test/bpf: add missing backslash in check-complexity.sh sed command
c81f03e 
This will include IPV4_FROM_HOST/IPV6_FROM_HOST sections.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser
Copy link
Member Author

test-me-please

@coveralls
Copy link

coveralls commented May 26, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.03%) to 36.896% when pulling c81f03e on pr/tklauser/bpf-cleanup into 24fcbe6 on master.

@tklauser
Copy link
Member Author

tklauser commented May 26, 2020 •
edited
Loading

retest-4.19 (previous failure with #11235: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Kernel/1708/)

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 26, 2020
@joestringer joestringer merged commit 2c03d31 into master May 27, 2020
@joestringer joestringer deleted the pr/tklauser/bpf-cleanup branch May 27, 2020 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brb brb brb approved these changes

@borkmann borkmann borkmann approved these changes

@joestringer joestringer joestringer approved these changes

@pchaigno pchaigno pchaigno approved these changes

Assignees
No one assigned
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@tklauser @coveralls @brb @borkmann @joestringer @pchaigno