summaryrefslogtreecommitdiffstats
path: root/wifi/1.0/vts
diff options
context:
space:
mode:
authorRoshan Pius <rpius@google.com>2017-08-08 08:43:50 -0700
committerRoshan Pius <rpius@google.com>2017-08-09 04:58:29 +0000
commit0d1bb97447eef57aa7a76def4dddbd4242a42af2 (patch)
tree1ab5622ebe9f05a3d08b3a2a695c8138429588ce /wifi/1.0/vts
parent5d77efddbda689b0ad1bf11b810bb8130d9b3282 (diff)
downloadandroid_hardware_interfaces-0d1bb97447eef57aa7a76def4dddbd4242a42af2.tar.gz
android_hardware_interfaces-0d1bb97447eef57aa7a76def4dddbd4242a42af2.tar.bz2
android_hardware_interfaces-0d1bb97447eef57aa7a76def4dddbd4242a42af2.zip
wifi(vts): Remove a tight loop of toggling wifi state
The tight loop of HAL start/stop exposes a race condition within the HAL. Adding a fix for preventing this race would need a fix which would be pretty risky at this point of the release. The issue itself is unlikely to happen in real use cases because wifi toggles (user initiated or test scripts initiated) via framework goes through a series of processing before it invokes the HAL start/stop. Bug: 64195190 Test: `make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 && vts-tradefed run commandAndExit vts --skip-all-system-status-check --primary-abi-only --skip-preconditions --module VtsHalWifiV1_0Target -l INFO` Change-Id: I4e4d65f8b6e2e423a3a5f26e5a97a78b7e99c3e8
Diffstat (limited to 'wifi/1.0/vts')
-rw-r--r--wifi/1.0/vts/functional/wifi_chip_hidl_test.cpp2
1 files changed, 2 insertions, 0 deletions
diff --git a/wifi/1.0/vts/functional/wifi_chip_hidl_test.cpp b/wifi/1.0/vts/functional/wifi_chip_hidl_test.cpp
index 06e21ff42..3c2313592 100644
--- a/wifi/1.0/vts/functional/wifi_chip_hidl_test.cpp
+++ b/wifi/1.0/vts/functional/wifi_chip_hidl_test.cpp
@@ -182,6 +182,8 @@ TEST_F(WifiChipHidlTest, ConfigureChip) {
EXPECT_EQ(WifiStatusCode::SUCCESS,
HIDL_INVOKE(wifi_chip_, configureChip, mode.id).code);
stopWifi();
+ // Sleep for 5 milliseconds between each wifi state toggle.
+ usleep(5000);
}
}