summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaciej Żenczykowski <maze@google.com>2022-12-20 08:16:15 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2022-12-20 08:16:15 +0000
commitc9c66f71ea69d42e41533a930ce5fb26095e1b2e (patch)
tree68308cfaf809205398c86f1a325af49b29625387
parentf7f841024a42d511cd87260ff7e0d01dbde6d83b (diff)
parent978b847fff1a97477a79ae3a65530a39d972f85a (diff)
downloadnetd-c9c66f71ea69d42e41533a930ce5fb26095e1b2e.tar.gz
cleanup a bunch of bpf related things (header files mostly) am: 4d913bbcf8 am: feca8e61ca am: 978b847fff
Original change: https://android-review.googlesource.com/c/platform/system/netd/+/2363881 Change-Id: I5a518b89ea718440d614e196e75e4a3780b215b8 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--server/Android.bp3
-rw-r--r--server/BandwidthController.cpp1
-rw-r--r--server/FirewallController.cpp1
-rw-r--r--server/FirewallController.h1
-rw-r--r--server/IptablesRestoreControllerTest.cpp1
-rw-r--r--server/MDnsSdListener.cpp1
-rw-r--r--server/NetworkController.cpp1
-rw-r--r--server/TcUtils.h5
-rw-r--r--server/TetherController.h1
-rw-r--r--tests/Android.bp1
-rw-r--r--tests/benchmarks/bpf_benchmark.cpp1
-rw-r--r--tests/binder_test.cpp2
12 files changed, 2 insertions, 17 deletions
diff --git a/server/Android.bp b/server/Android.bp
index 615e740f..1f893749 100644
--- a/server/Android.bp
+++ b/server/Android.bp
@@ -45,7 +45,6 @@ cc_library_static {
"system/netd/include",
"system/netd/server/binder",
],
- header_libs: ["bpf_headers"],
srcs: [
"BandwidthController.cpp",
"Controllers.cpp",
@@ -101,7 +100,6 @@ cc_binary {
required: [
"bpfloader",
],
- header_libs: ["bpf_headers"],
shared_libs: [
"android.system.net.netd@1.0",
"android.system.net.netd@1.1",
@@ -208,7 +206,6 @@ cc_test {
"system/netd/server/binder",
"system/netd/tests",
],
- header_libs: ["bpf_headers"],
tidy_timeout_srcs: [
"BandwidthControllerTest.cpp",
"InterfaceControllerTest.cpp",
diff --git a/server/BandwidthController.cpp b/server/BandwidthController.cpp
index 8dba75a5..438dbb8d 100644
--- a/server/BandwidthController.cpp
+++ b/server/BandwidthController.cpp
@@ -56,7 +56,6 @@
#include "Fwmark.h"
#include "NetdConstants.h"
#include "android/net/INetd.h"
-#include "bpf/BpfUtils.h"
/* Alphabetical */
#define ALERT_IPT_TEMPLATE "%s %s -m quota2 ! --quota %" PRId64" --name %s\n"
diff --git a/server/FirewallController.cpp b/server/FirewallController.cpp
index b3c6b025..47975dd3 100644
--- a/server/FirewallController.cpp
+++ b/server/FirewallController.cpp
@@ -34,7 +34,6 @@
#include "Controllers.h"
#include "FirewallController.h"
#include "NetdConstants.h"
-#include "bpf/BpfUtils.h"
using android::base::Join;
using android::base::StringAppendF;
diff --git a/server/FirewallController.h b/server/FirewallController.h
index 1bff064c..45ae0823 100644
--- a/server/FirewallController.h
+++ b/server/FirewallController.h
@@ -24,7 +24,6 @@
#include <vector>
#include "NetdConstants.h"
-#include "bpf/BpfUtils.h"
namespace android {
namespace net {
diff --git a/server/IptablesRestoreControllerTest.cpp b/server/IptablesRestoreControllerTest.cpp
index cecdf4d9..1999fbf9 100644
--- a/server/IptablesRestoreControllerTest.cpp
+++ b/server/IptablesRestoreControllerTest.cpp
@@ -36,7 +36,6 @@
#include <netdutils/Stopwatch.h>
#include "NetdConstants.h"
-#include "bpf/BpfUtils.h"
#define XT_LOCK_NAME "/system/etc/xtables.lock"
#define XT_LOCK_ATTEMPTS 10
diff --git a/server/MDnsSdListener.cpp b/server/MDnsSdListener.cpp
index 4bd5f0d4..d2422f6b 100644
--- a/server/MDnsSdListener.cpp
+++ b/server/MDnsSdListener.cpp
@@ -21,6 +21,7 @@
#include <errno.h>
#include <inttypes.h>
#include <linux/if.h>
+#include <net/if.h>
#include <netdb.h>
#include <netinet/in.h>
#include <pthread.h>
diff --git a/server/NetworkController.cpp b/server/NetworkController.cpp
index 5233a1ea..f1aaeca8 100644
--- a/server/NetworkController.cpp
+++ b/server/NetworkController.cpp
@@ -30,6 +30,7 @@
#include <android-base/strings.h>
#include <cutils/misc.h> // FIRST_APPLICATION_UID
#include <netd_resolv/resolv.h>
+#include <net/if.h>
#include "log/log.h"
#include "Controllers.h"
diff --git a/server/TcUtils.h b/server/TcUtils.h
index 4b1b2b9f..9b157191 100644
--- a/server/TcUtils.h
+++ b/server/TcUtils.h
@@ -16,17 +16,12 @@
#pragma once
-#include <android-base/result.h>
#include <errno.h>
#include <linux/if_ether.h>
#include <linux/if_link.h>
#include <linux/rtnetlink.h>
#include <tcutils/tcutils.h>
-#include <string>
-
-#include "bpf/BpfUtils.h"
-
namespace android {
namespace net {
diff --git a/server/TetherController.h b/server/TetherController.h
index d2195f23..4aa2a2d6 100644
--- a/server/TetherController.h
+++ b/server/TetherController.h
@@ -27,7 +27,6 @@
#include "NetdConstants.h"
#include "android-base/result.h"
-#include "bpf/BpfMap.h"
#include "android/net/TetherOffloadRuleParcel.h"
diff --git a/tests/Android.bp b/tests/Android.bp
index f590f768..e0426a29 100644
--- a/tests/Android.bp
+++ b/tests/Android.bp
@@ -90,7 +90,6 @@ cc_test {
"netd_test.cpp",
],
include_dirs: ["system/netd/server"],
- header_libs: ["bpf_connectivity_headers"],
shared_libs: [
"libbase",
"libbinder",
diff --git a/tests/benchmarks/bpf_benchmark.cpp b/tests/benchmarks/bpf_benchmark.cpp
index 4de80661..4614142f 100644
--- a/tests/benchmarks/bpf_benchmark.cpp
+++ b/tests/benchmarks/bpf_benchmark.cpp
@@ -20,7 +20,6 @@
#define BPF_MAP_MAKE_VISIBLE_FOR_TESTING
#include "bpf/BpfMap.h"
-#include "bpf/BpfUtils.h"
constexpr uint32_t TEST_MAP_SIZE = 10000;
diff --git a/tests/binder_test.cpp b/tests/binder_test.cpp
index 33a9d69f..f1d34945 100644
--- a/tests/binder_test.cpp
+++ b/tests/binder_test.cpp
@@ -53,8 +53,6 @@
#include <android-base/test_utils.h>
#include <android/multinetwork.h>
#include <binder/IPCThreadState.h>
-#include <bpf/BpfMap.h>
-#include <bpf/BpfUtils.h>
#include <com/android/internal/net/BnOemNetdUnsolicitedEventListener.h>
#include <com/android/internal/net/IOemNetd.h>
#include <cutils/multiuser.h>