summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaciej Żenczykowski <maze@google.com>2021-07-01 02:22:30 +0000
committerMaciej Zenczykowski <maze@google.com>2021-07-01 03:10:30 +0000
commitaee2cbe30f5097ec85daa2cfabfe7377dfd5e542 (patch)
treeec36741078d8f126a549803ffa7a4a85e17f30e5
parent2d4975f0c055ed9ee6ef9050c03fb19214d648b5 (diff)
downloadnetd-aee2cbe30f5097ec85daa2cfabfe7377dfd5e542.tar.gz
Revert "Disable IdletimerAddRemoveInterface test on >=5.10"
This reverts commit ac67f9348b78ad6a9876e72cf57c1818177a9bfd. The Android Common Kernels [5.10+] now supports this. Test: atest NetdBinderTest IdletimerControllerTest Bug: 175745224 Signed-off-by: Maciej Żenczykowski <maze@google.com> Original-Change: https://android-review.googlesource.com/1673846 Merged-In: I4ff791cd455dc14106e329e7c29811495ce7bae7 Change-Id: I4ff791cd455dc14106e329e7c29811495ce7bae7
-rw-r--r--tests/binder_test.cpp11
1 files changed, 0 insertions, 11 deletions
diff --git a/tests/binder_test.cpp b/tests/binder_test.cpp
index 69d1f9b5..1658abcb 100644
--- a/tests/binder_test.cpp
+++ b/tests/binder_test.cpp
@@ -43,7 +43,6 @@
#include <openssl/base64.h>
#include <sys/socket.h>
#include <sys/types.h>
-#include <sys/utsname.h>
#include <android-base/file.h>
#include <android-base/format.h>
@@ -1229,16 +1228,6 @@ void expectIdletimerInterfaceRuleNotExists(const std::string& ifname, int timeou
} // namespace
TEST_F(NetdBinderTest, IdletimerAddRemoveInterface) {
- // TODO(b/175745224): Temporarily disable idletimer test on >5.10 kernels
- utsname u;
- if (!uname(&u)) {
- unsigned long major, minor;
- char *p;
- major = strtoul(u.release, &p, 10);
- minor = strtoul(++p, NULL, 10);
- if (major > 5 || (major == 5 && minor >= 10)) return;
- }
-
// TODO: We will get error in if expectIdletimerInterfaceRuleNotExists if there are the same
// rule in the table. Because we only check the result after calling remove function. We might
// check the actual rule which is removed by our function (maybe compare the results between