[PATCH 3/3] tests: permit multiple GO negotiation failures

Benjamin Berg benjamin at sipsolutions.net
Thu Jun 12 01:10:21 PDT 2025


From: Benjamin Berg <benjamin.berg at intel.com>

In this test the station will do a P2P find and then send a GO
negotiation request. However, the other station may not be on the listen
channel at the time when the action frame is transmitted. In this case,
the find/GO neg request sequence is restarted.

The test currently times out if this sequence fails twice in a row. Be
more tolerant to this type of failure and increase the timeout.

It may also be possible to improve the retransmit strategy.

Signed-off-by: Benjamin Berg <benjamin.berg at intel.com>
Reviewed-by: Andrei Otcheretianski <andrei.otcheretianski at intel.com>
---
 tests/hwsim/test_p2p_grpform.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/hwsim/test_p2p_grpform.py b/tests/hwsim/test_p2p_grpform.py
index 6b18c42e79..178bd73817 100644
--- a/tests/hwsim/test_p2p_grpform.py
+++ b/tests/hwsim/test_p2p_grpform.py
@@ -958,7 +958,7 @@ def test_grpform_not_ready2(dev):
     if not dev[1].discover_peer(addr0):
         raise Exception("Could not discover peer")
     dev[1].global_request("P2P_CONNECT " + addr0 + " pbc")
-    ev = dev[0].wait_global_event(["P2P-GO-NEG-REQUEST"], timeout=5)
+    ev = dev[0].wait_global_event(["P2P-GO-NEG-REQUEST"], timeout=10)
     if ev is None:
         raise Exception("No P2P-GO-NEG-REQUEST event")
     dev[0].dump_monitor()
-- 
2.49.0




More information about the Hostap mailing list