-
Notifications
You must be signed in to change notification settings - Fork 9
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
Code cleaning #18
Code cleaning #18
Conversation
This field is only needed for IP fragmentation and we don't support it for now. Even if IP fragmentation was supported, reading RFC6864 I understand we should not modify this field. More information about that: https://crnetpackets.com/2015/08/29/a-short-story-about-the-ip-id-field/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -123,7 +118,7 @@ class Data(ct.Structure): | |||
|
|||
# Compile & attach bpf program | |||
print("\nCompiling & attaching bpf code ...") | |||
b = BPF(src_file ="ulb.c", debug=debug, cflags=["-Wno-incompatible-pointer-types", "-Wno-compare-distinct-pointer-types"]) | |||
b = BPF(src_file ="ulb.c", debug=debug) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feature request: let the user to specify cflags using a CFLAGS environment variable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I create a new issue about that : #19.
Do not hesitate to create new issue for feature request.
See commit names/comments for details.
I advice to review commit by commit.
(This tasks is related to #6)