From: James Prestwood <prestwoj@gmail.com>
To: iwd@lists.linux.dev
Cc: James Prestwood <prestwoj@gmail.com>
Subject: [PATCH v2 4/4] auto-t: add test for channel switch during roam
Date: Mon, 18 Aug 2025 07:30:41 -0700 [thread overview]
Message-ID: <20250818143041.283887-4-prestwoj@gmail.com> (raw)
In-Reply-To: <20250818143041.283887-1-prestwoj@gmail.com>
In kernel 6.8 a new CMD_ASSOCIATE failure path was added which checks
if the AP has a channel switch in progress. Eariler patches update
IWD into handling this case better, and this new test exercises that.
---
autotests/testPSK-roam/chan_switch_test.py | 114 +++++++++++++++++++++
1 file changed, 114 insertions(+)
create mode 100644 autotests/testPSK-roam/chan_switch_test.py
diff --git a/autotests/testPSK-roam/chan_switch_test.py b/autotests/testPSK-roam/chan_switch_test.py
new file mode 100644
index 00000000..2e2154cf
--- /dev/null
+++ b/autotests/testPSK-roam/chan_switch_test.py
@@ -0,0 +1,114 @@
+#! /usr/bin/python3
+
+import unittest
+import sys, os
+
+sys.path.append('../util')
+from iwd import IWD
+from iwd import NetworkType
+from hostapd import HostapdCLI
+from packaging import version
+from subprocess import run
+import re
+import testutil
+
+#
+# The CSA handling was added in kernel 6.8, so for any earlier kernel this test
+# won't pass.
+#
+def kernel_is_newer(min_version="6.8"):
+ proc = run(["uname", "-r"], capture_output=True)
+
+ version_str = proc.stdout.decode("utf-8")
+ match = re.match(r"(\d+\.\d+)", version_str)
+ if not match:
+ return False
+
+ return version.parse(match.group(1)) >= version.parse(min_version)
+
+class Test(unittest.TestCase):
+ def test_channel_switch_during_roam(self):
+ wd = self.wd
+
+ device = wd.list_devices(1)[0]
+
+ ordered_network = device.get_ordered_network('TestFT', full_scan=True)
+
+ self.assertEqual(ordered_network.type, NetworkType.psk)
+
+ condition = 'not obj.connected'
+ wd.wait_for_object_condition(ordered_network.network_object, condition)
+
+ self.assertFalse(self.bss_hostapd[0].list_sta())
+ self.assertFalse(self.bss_hostapd[1].list_sta())
+
+ device.connect_bssid(self.bss_hostapd[0].bssid)
+
+ condition = 'obj.state == DeviceState.connected'
+ wd.wait_for_object_condition(device, condition)
+
+ self.bss_hostapd[0].wait_for_event('AP-STA-CONNECTED %s' % device.address)
+
+ testutil.test_iface_operstate(device.name)
+ testutil.test_ifaces_connected(self.bss_hostapd[0].ifname, device.name)
+ self.assertRaises(Exception, testutil.test_ifaces_connected,
+ (self.bss_hostapd[1].ifname, device.name, True, True))
+
+ # Start a channel switch and wait for it to begin
+ self.bss_hostapd[1].chan_switch(6, wait=False)
+ self.bss_hostapd[1].wait_for_event("CTRL-EVENT-STARTED-CHANNEL-SWITCH")
+ # Initiate a roam immediately which should get rejected by the kernel
+ device.roam(self.bss_hostapd[1].bssid)
+
+ # IWD should authenticate, then proceed to association
+ device.wait_for_event("ft-authenticating")
+ device.wait_for_event("ft-roaming")
+
+ # The kernel should reject the association, which should trigger a
+ # disconnect
+ condition = 'obj.state == DeviceState.disconnected'
+ wd.wait_for_object_condition(device, condition)
+
+ condition = 'obj.state == DeviceState.connected'
+ wd.wait_for_object_condition(device, condition)
+
+
+ def tearDown(self):
+ os.system('ip link set "' + self.bss_hostapd[0].ifname + '" down')
+ os.system('ip link set "' + self.bss_hostapd[1].ifname + '" down')
+ os.system('ip link set "' + self.bss_hostapd[0].ifname + '" up')
+ os.system('ip link set "' + self.bss_hostapd[1].ifname + '" up')
+
+ for hapd in self.bss_hostapd:
+ hapd.default()
+
+ self.wd.stop()
+ self.wd = None
+
+ def setUp(self):
+ self.wd = IWD(True)
+
+ @classmethod
+ def setUpClass(cls):
+ if not kernel_is_newer():
+ raise unittest.SkipTest()
+
+ IWD.copy_to_storage('TestFT.psk')
+
+ cls.bss_hostapd = [ HostapdCLI(config='ft-psk-ccmp-1.conf'),
+ HostapdCLI(config='ft-psk-ccmp-2.conf'),
+ HostapdCLI(config='ft-psk-ccmp-3.conf') ]
+
+ unused = HostapdCLI(config='ft-psk-ccmp-3.conf')
+ unused.disable()
+
+ cls.bss_hostapd[0].set_address('12:00:00:00:00:01')
+ cls.bss_hostapd[1].set_address('12:00:00:00:00:02')
+ cls.bss_hostapd[2].set_address('12:00:00:00:00:03')
+
+ HostapdCLI.group_neighbors(*cls.bss_hostapd)
+
+ @classmethod
+ def tearDownClass(cls):
+ IWD.clear_storage()
+ cls.bss_hostapd = None
--
2.34.1
next prev parent reply other threads:[~2025-08-18 14:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-18 14:30 [PATCH v2 1/4] netdev: check connected in channel switch event James Prestwood
2025-08-18 14:30 ` [PATCH v2 2/4] netdev: disconnect rather than deauth in FT association failure James Prestwood
2025-08-18 14:30 ` [PATCH v2 3/4] auto-t: make waiting for channel switch configurable James Prestwood
2025-08-18 14:30 ` James Prestwood [this message]
2025-08-20 16:20 ` [PATCH v2 1/4] netdev: check connected in channel switch event Denis Kenzior
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250818143041.283887-4-prestwoj@gmail.com \
--to=prestwoj@gmail.com \
--cc=iwd@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox