michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: test: privacy/legacy: Scan continuously

Browse source 
Some controllers, like the SoftDevice Controller can miss the first few
advertisements after the host tells it to start scanning. The DUT would
assumes the first RPA advertisements report it gets is the first that
was sent, but it was actually not the first. This would skew the
tester's judgement about the timing of RPA rotations.

To remedy this, the tester will scan continuously and switch between
expecting the identity address and the RPA without stopping the scanner.

The tester will tell the DUT to switch to RPA after it has received the
identity address advertisement. This ensures that the tester will not
miss the first RPA advertisement.

Signed-off-by: Aleksander Wasaznik <aleksander.wasaznik@nordicsemi.no>
This commit is contained in:
Aleksander Wasaznik 2024-06-21 13:26:42 +02:00 • committed by Anas Nashif
commit 8cd98b139d
2 changed files with 18 additions and 31 deletions
Download patch file Download diff file Expand all files Collapse all files

3
tests/bsim/bluetooth/host/privacy/legacy/src/dut.c
View file

@ -96,9 +96,6 @@ void dut_procedure(void)
LOG_DBG("start adv with RPA");
start_advertising(BT_LE_ADV_OPT_CONNECTABLE);
/* signal tester it can start scanning again, expecting an RPA this time */
backchannel_sync_send();
/* Test pass verdict is decided by the tester */
PASS("DUT done\n");
}

46
tests/bsim/bluetooth/host/privacy/legacy/src/tester.c
View file

@ -18,6 +18,7 @@
LOG_MODULE_REGISTER(tester, 4);
struct test_data_t {
bool id_addr_ok;
bt_addr_le_t old_addr;
int64_t old_time;
int rpa_rotations;
@ -81,8 +82,6 @@ static void cb_expect_rpa(const bt_addr_le_t *addr, int8_t rssi, uint8_t type,
static void cb_expect_id(const bt_addr_le_t *addr, int8_t rssi, uint8_t type,
struct net_buf_simple *ad)
{
int err;
LOG_DBG("expecting addr:");
print_address(&dut_addr);
LOG_DBG("got addr:");
@ -96,33 +95,24 @@ static void cb_expect_id(const bt_addr_le_t *addr, int8_t rssi, uint8_t type,
if (!bt_addr_le_eq(&dut_addr, addr)) {
FAIL("DUT not using identity address\n");
}
err = bt_le_scan_stop();
if (err) {
FAIL("Failed to stop scan: %d\n", err);
}
backchannel_sync_send();
}
void start_scanning(bool expect_rpa)
static void scan_cb(const bt_addr_le_t *addr, int8_t rssi, uint8_t type, struct net_buf_simple *ad)
{
int err;
struct bt_le_scan_param scan_param = {
.type = BT_HCI_LE_SCAN_PASSIVE,
.options = BT_LE_SCAN_OPT_FILTER_DUPLICATE,
.interval = 0x0040,
.window = 0x0020,
};
/* The DUT advertises with the identity address first, to test
* that option, but also to allow the DUT time to start its
* scanner. The scanner must be ready to capture the one of
* first RPA advertisements to accurately judge the RPA
* timeout, which is measured from the first RPA advertisement.
*/
if (!test_data.id_addr_ok) {
cb_expect_id(addr, rssi, type, ad);
if (expect_rpa) {
err = bt_le_scan_start(&scan_param, cb_expect_rpa);
/* Tell DUT to switch to RPA */
backchannel_sync_send();
test_data.id_addr_ok = true;
} else {
err = bt_le_scan_start(&scan_param, cb_expect_id);
}
if (err) {
FAIL("Failed to start scanning\n");
cb_expect_rpa(addr, rssi, type, ad);
}
}
@ -138,9 +128,9 @@ void tester_procedure(void)
FAIL("Failed to enable bluetooth (err %d\n)", err);
}
start_scanning(false);
err = bt_le_scan_start(BT_LE_SCAN_PASSIVE_CONTINUOUS, scan_cb);
backchannel_sync_wait();
start_scanning(true);
if (err) {
FAIL("Failed to start scanning\n");
}
}