| SHA1 | ||
|---|---|---|
Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I7320cf60a8994a6c8da99293f068265f54a9ddc6 
Maciej Żenczykowski
commited |
1e487177f1 | |
(since this map relies on percpu access semantics, and we don't even have userspace helpers to access PERCPU_ maps) This will also be used by the upcoming satellite egress limiting logic. See aosp/3235142 With this change we see: $ adbz shell ls -lZ /sys/fs/bpf/net_shared/*_dscpPolicy_* -rw-rw---- 1 root system u:object_r:fs_bpf_net_shared:s0 0 2024-09-04 17:01 /sys/fs/bpf/net_shared/map_dscpPolicy_ipv4_dscp_policies_map -rw-rw---- 1 root system u:object_r:fs_bpf_net_shared:s0 0 2024-09-04 17:01 /sys/fs/bpf/net_shared/map_dscpPolicy_ipv6_dscp_policies_map ---------- 1 root root u:object_r:fs_bpf_loader:s0 0 2024-09-04 17:01 /sys/fs/bpf/net_shared/map_dscpPolicy_socket_policy_cache_map -r--r----- 1 root system u:object_r:fs_bpf_net_shared:s0 0 2024-09-04 17:01 /sys/fs/bpf/net_shared/prog_dscpPolicy_schedcls_set_dscp_ether Test: TreeHugger, atest CtsNetTestCases:android.net.cts.DscpPolicyTest Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I50a12ca4cd5587342f8aaf9df8cb28917f555252
Maciej Żenczykowski
commited |
1ec8d7df58 | |
To shorten lines. Now: egrep 'base:' NetBpfLoad.cpp only finds the 'using' statements. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I8b125c420c106e4b9c2af0a4fa2e4892a08bb152
Maciej Żenczykowski
commited |
8a7672822d | |
Bug: 329269188 Test: Treehugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Iba87f62743f76f532e51f4dbb156054e50d6dd5d
Maciej Żenczykowski
commited |
1c2187ace9 | |
Note that this is a no-op for everything before sdk=36+, since V/sdk=35 only supports up to 6.6 LTS anyway. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I701045074dfb7b0c586536634e210318b5af364b
Maciej Żenczykowski
commited |
9b6a994984 | |
This duplicates the previous check, but without the exceptions. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Id83695689268baacd1112821fa3fd212c3aa0bb6
Maciej Żenczykowski
commited |
21b4a05c73 | |
Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Ib94ed340712b662072eee50d8f1deb9f6fe20b55
Maciej Żenczykowski
commited |
de7374d850 | |
(prep work to make next diff easier on the eyes) Test: N/A Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I2a6a10cbdeace7e7b43d64bc1e0743216e4321e1
Maciej Żenczykowski
commited |
726b58fd19 | |
(ALOGD can get entirely stripped out of the logs) Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I513d10acb9f1cc330b9cd5a4245f3af7e790a4d6
Maciej Żenczykowski
commited |
8597e3ac16 | |
BPF_PROG_LOAD can fail with various permission related errors, and then it doesn't provide any extra debug details. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I9f6f0bf30158828fe31045a11a1f4c015feeb61e
Maciej Żenczykowski
commited |
9f8b17ee5b | |
We're reduced logging enough that it is no longer clear which program failed to load. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I8919a055f82d0c0e6e6818799463c606e853f70b
Maciej Żenczykowski
commited |
bfc0b6187c | |
Maciej Żenczykowski
authored and
 Gerrit Code Review
commited |
461b408129 | |
This is entirely expected due to lack of implementation in <4.14. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Ifebe6843dc77b1b9a506c61239dcebe7262d0869
Maciej Żenczykowski
commited |
508c8f7857 | |
Longer term we should fix the bpf_map_def.h to actually set map_flags correctly. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I4e7954ccd0afad4ca8d9cec179ad6efa20592609
Maciej Żenczykowski
commited |
dcb2e7496c | |
This allows changing the location of bpf programs easily when building for the host to run on UML as a test. Bug: 205608149 Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I24816b6660581e74f277df39495a43027b77d2cf
Maciej Żenczykowski
commited |
78fa86107b | |
Some compilers warn/error and fail to build if it is not in proper order. (specifically: triggers during build for host) Bug: 205608149 Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I84ddec9a2443939616fd78f522e8620680e1289e
Maciej Żenczykowski
commited |
52be6a822d | |
(particularly problematic on host builds where host headers are ancient...) Bug: 205608149 Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Iae64b900464e7584ad657f18d8101993fd8c8e33
Maciej Żenczykowski
commited |
020eaf4dd5 | |
These messages are quite spammy. Since this is now all mainline code built/shipped together, they're basically all deterministic. We print the map/program ids for all maps & programs we load which means it's pretty obvious what was skipped anyway. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Id46d9be8f77bef0b4c86e3a76503570c43e43af4
Maciej Żenczykowski
commited |
dbdd90fb31 | |
Generated via: git mv staticlibs/native/bpf_headers bpf/headers cd bpf/headers/include git mv bpf/bpf_helpers.h . git mv bpf/bpf_map_def.h . manually fix up bpf/bpf_map_def.h include path bpf_helpers.h is only used by bpf programs so the 'bpf/' prefix is spurious. bpf_map_def.h is only used by bpf programs and the two bpfloaders, which effectively means the 'bpf/' prefix isn't useful. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: Ic3fc4e354a15821c27c23b7dbc019daad9f6ad85
Maciej Żenczykowski
commited |
d602835b4f | |
Step 2 of moving bpf related stuff into a common directory, will ease development. Test: TreeHugger Signed-off-by: Maciej Żenczykowski <maze@google.com> Change-Id: I95cab49ae03eb275982f43d2246085b06e66dc6d
Maciej Żenczykowski
commited |
a207ac743c |
Renamed from netbpfload/NetBpfLoad.cpp (Browse further)