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

early spring cleanup helper prep for xdp #10344

Merged
merged 4 commits into from Feb 26, 2020
Merged

early spring cleanup helper prep for xdp #10344

merged 4 commits into from Feb 26, 2020

Conversation

borkmann
Copy link
Member

@borkmann borkmann commented Feb 26, 2020

Given we've been refactoring the whole BPF code base via #10333, perform some more cleanup including rebase of #10017 while at it, so others only need to rebase once. See commit msg @brb


This change is Reviewable

zhiyuan0x and others added 3 commits February 26, 2020 14:01
Also remove __inline__ treewide and add a compiler.h library where
we can place misc compiler related things in a central place.

Signed-off-by: Zhiyuan Hou <zhiyuan2048@linux.alibaba.com>
[ Daniel: picking up and extending Zhiyuan's PR #10017 after rebase ]
Co-developed-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Get a bit of order in here.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
It's not used yet, but to make comments consistent to CTX_ACT_ codes.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann borkmann added pending-review sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. kind/cleanup This includes no functional changes. labels Feb 26, 2020
@borkmann borkmann requested review from brb and a team February 26, 2020 14:04
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

3 similar comments
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Feb 26, 2020
@borkmann borkmann added area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. release-note/misc This PR makes changes that have no direct user impact. labels Feb 26, 2020
@borkmann
Copy link
Member Author

test-me-please

@borkmann borkmann force-pushed the pr/header-cleanup branch 2 times, most recently from 7ee1ca9 to 2c0b3fb Compare February 26, 2020 14:13
@borkmann
Copy link
Member Author

test-me-please

@borkmann
Copy link
Member Author

test-me-please

Early spring cleanup to make them more useful and later better
extensible for XDP. Also remove iproute2 header which is not needed
anymore.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann
Copy link
Member Author

test-me-please

@borkmann borkmann merged commit 66ca0bd into master Feb 26, 2020
1.8.0 automation moved this from In progress to Merged Feb 26, 2020
@borkmann borkmann deleted the pr/header-cleanup branch February 26, 2020 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
No open projects
1.8.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

5 participants