diff mbox series

[1/7] tests: Wait after removing a BSS

Message ID 20240408130701.531194-1-andrei.otcheretianski@intel.com
State Accepted
Headers show
Series [1/7] tests: Wait after removing a BSS | expand

Commit Message

Andrei Otcheretianski April 8, 2024, 1:06 p.m. UTC 
From: Ilan Peer <ilan.peer@intel.com>

When a BSS is removed, the flow continues without actually
waiting for the AP to be stopped. This is racy in flows that
actually expect the AP to be stopped, e.g., test_ap_bss_add_remove().

Try to mitigate such cases by adding a short sleep after the
AP is removed.

Signed-off-by: Ilan Peer <ilan.peer@intel.com>
---
 tests/hwsim/hostapd.py | 3 +++
 1 file changed, 3 insertions(+)

Comments

Jouni Malinen April 21, 2024, 9:06 a.m. UTC | #1 
Thanks, patches 1-3 and 5-7 applied.
diff mbox series

Patch

diff --git a/tests/hwsim/hostapd.py b/tests/hwsim/hostapd.py
index 007812274b..ebb2e328af 100644
--- a/tests/hwsim/hostapd.py
+++ b/tests/hwsim/hostapd.py
@@ -766,6 +766,9 @@  def remove_bss(apdev, ifname=None):
     hapd_global = HostapdGlobal(apdev)
     hapd_global.remove(ifname)
 
+    # wait little to make sure the AP stops beaconing
+    time.sleep(0.1)
+
 def terminate(apdev):
     try:
         hostname = apdev['hostname']