tests: bsim: Bluetooth: Test RTN=2, FT=2, Per skip 2 SE in Controller

Test RTN=2, FT=2 in Controller with 2 subevents dropped by
peripheral.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>

Author: cvinayak

subsys/bluetooth/controller/ll_sw/nordic/lll/lll_peripheral_iso.c

@@ -472,7 +472,14 @@ static void isr_rx(void *param)
 	cis_lll = param;
 
 	/* No Rx */
-	if (!trx_done) {
+	if (!trx_done ||
+#if defined(CONFIG_TEST_FT_PER_SKIP_SUBEVENTS)
+	    ((((cis_lll->event_count % 3U) < CONFIG_TEST_FT_PER_SKIP_EVENTS_COUNT) &&
+	      ((se_curr > cis_lll->nse) || (se_curr <= 2U))) ||
+	     (((cis_lll->event_count % 3U) < (CONFIG_TEST_FT_PER_SKIP_EVENTS_COUNT + 1U)) &&
+	      ((se_curr > cis_lll->nse) || (se_curr <= 1U)))) ||
+#endif
+	    false) {
 		payload_count_flush(cis_lll);
 
 		/* Next subevent or next CIS */

tests/bsim/bluetooth/ll/cis/Kconfig

@@ -16,6 +16,25 @@ config TEST_MULTIPLE_PERIPERAL_CIS
 	help
 	  Multiple Peripheral CIS establishment.
 
+config TEST_FT_SKIP_SUBEVENTS
+	bool
+	help
+	  Skip central and/or peripheral subevent reception to test flush
+	  timeout implementation.
+
+config TEST_FT_PER_SKIP_SUBEVENTS
+	bool "Skip peripheral role subevents to test Flush Timeout"
+	select TEST_FT_SKIP_SUBEVENTS
+	help
+	  Skip peripheral role subevent reception to test flush timeout
+	  implementation.
+
+config TEST_FT_PER_SKIP_EVENTS_COUNT
+	int "Skip peripheral ISO events count, all subevents in them"
+	depends on TEST_FT_PER_SKIP_SUBEVENTS
+	help
+	  Skip peripheral ISO events count where all subevents are skipped.
+
 config BT_CTLR_SCAN_UNRESERVED
 	default y if TEST_CONNECT_ACL_FIRST
 	help

tests/bsim/bluetooth/ll/cis/overlay-acl_first_ft_per_skip_2_se.conf

@@ -0,0 +1,9 @@
+CONFIG_TEST_USE_LEGACY_ADVERTISING=n
+CONFIG_TEST_CONNECT_ACL_FIRST=y
+CONFIG_TEST_FT_PER_SKIP_SUBEVENTS=y
+CONFIG_TEST_FT_PER_SKIP_EVENTS_COUNT=1
+CONFIG_BT_MAX_CONN=1
+CONFIG_BT_ISO_MAX_CHAN=1
+CONFIG_BT_CTLR_LLCP_LOCAL_PROC_CTX_BUF_NUM=9
+CONFIG_BT_CTLR_ADVANCED_FEATURES=y
+CONFIG_BT_CTLR_ISOAL_PSN_IGNORE=y

tests/bsim/bluetooth/ll/cis/prj.conf

@@ -13,6 +13,7 @@ CONFIG_BT_MAX_CONN=9
 CONFIG_BT_ISO_MAX_CHAN=9
 CONFIG_BT_ISO_TX_BUF_COUNT=18
 CONFIG_BT_ISO_TX_MTU=120
+CONFIG_BT_ISO_RX_MTU=120
 
 CONFIG_BT_BUF_CMD_TX_SIZE=255
 CONFIG_BT_BUF_EVT_RX_SIZE=255

tests/bsim/bluetooth/ll/cis/src/main.c

@@ -53,6 +53,7 @@ static bt_addr_le_t peer_addr;
 
 #define ISO_INTERVAL_US      10000U
 #define ISO_LATENCY_MS       DIV_ROUND_UP(ISO_INTERVAL_US, USEC_PER_MSEC)
+#define ISO_LATENCY_FT_MS    20U
 
 #define BT_CONN_US_TO_INTERVAL(t) ((uint16_t)((t) * 4U / 5U / USEC_PER_MSEC))
 
@@ -103,7 +104,7 @@ static bt_addr_le_t peer_addr;
 
 #define NAME_LEN 30
 
-#define BUF_ALLOC_TIMEOUT   (30) /* milliseconds */
+#define BUF_ALLOC_TIMEOUT   (40) /* milliseconds */
 NET_BUF_POOL_FIXED_DEFINE(tx_pool, CONFIG_BT_ISO_TX_BUF_COUNT,
 			  BT_ISO_SDU_BUF_SIZE(CONFIG_BT_ISO_TX_MTU), 8, NULL);
 
@@ -300,8 +301,11 @@ static void iso_recv(struct bt_iso_chan *chan, const struct bt_iso_recv_info *in
 			expected_seq_num[index] = seq_num;
 		}
 
-		expected_seq_num[index]++;
+		expected_seq_num[index] += 1U;
 
+#if defined(CONFIG_TEST_FT_SKIP_SUBEVENTS)
+		expected_seq_num[index] += ((CONFIG_TEST_FT_PER_SKIP_EVENTS_COUNT - 1U) * 2U);
+#endif
 	} else if (expected_seq_num[index] &&
 		   expected_seq_num[index] < SEQ_NUM_MAX) {
 		FAIL("%s: Invalid ISO data after valid ISO data reception.\n"
@@ -361,7 +365,11 @@ static void test_cis_central(void)
 	for (int i = 0; i < CONFIG_BT_ISO_MAX_CHAN; i++) {
 		iso_tx[i].sdu = CONFIG_BT_ISO_TX_MTU;
 		iso_tx[i].phy = BT_GAP_LE_PHY_2M;
-		iso_tx[i].rtn = 0U;
+		if (IS_ENABLED(CONFIG_TEST_FT_SKIP_SUBEVENTS)) {
+			iso_tx[i].rtn = 2U;
+		} else {
+			iso_tx[i].rtn = 0U;
+		}
 		iso_tx[i].path = NULL;
 
 		iso_qos[i].tx = &iso_tx[i];
@@ -381,7 +389,11 @@ static void test_cis_central(void)
 	cig_param.sca = BT_GAP_SCA_UNKNOWN;
 	cig_param.packing = 0U;
 	cig_param.framing = 0U;
-	cig_param.latency = ISO_LATENCY_MS;
+	if (IS_ENABLED(CONFIG_TEST_FT_SKIP_SUBEVENTS)) {
+		cig_param.latency = ISO_LATENCY_FT_MS;
+	} else {
+		cig_param.latency = ISO_LATENCY_MS;
+	}
 	cig_param.interval = ISO_INTERVAL_US;
 
 	printk("Create CIG...");
@@ -404,7 +416,7 @@ static void test_cis_central(void)
 		uint8_t conn_index;
 		uint8_t chan;
 
-		printk("Start scanning...");
+		printk("Start scanning (%d)...", i);
 		err = bt_le_scan_start(BT_LE_SCAN_CUSTOM, NULL);
 		if (err) {
 			FAIL("Could not start scan: %d\n", err);
@@ -510,7 +522,7 @@ static void test_cis_central(void)
 			ret = bt_iso_chan_send(&iso_chan[chan], buf,
 					       seq_num, BT_ISO_TIMESTAMP_NONE);
 			if (ret < 0) {
-				FAIL("Unable to broadcast data on channel %u"
+				FAIL("Unable to send data on channel %u"
 				     " : %d\n", chan, ret);
 				net_buf_unref(buf);
 				return;
@@ -522,7 +534,7 @@ static void test_cis_central(void)
 		}
 	}
 
-	k_sleep(K_MSEC(100));
+	k_sleep(K_MSEC(1000));
 
 	for (uint8_t chan = 0U; chan < CONFIG_BT_ISO_MAX_CHAN; chan++) {
 		printk("ISO disconnect channel %u...", chan);
@@ -587,6 +599,8 @@ static int iso_accept(const struct bt_iso_accept_info *info,
 	*chan = &iso_chan_p[chan_count];
 	chan_count++;
 
+	printk("Accepted on channel %p\n", *chan);
+
 	return 0;
 }
 

tests/bsim/bluetooth/ll/cis/tests_scripts/connected_iso_acl_first_ft_per_skip_2_se.sh

@@ -0,0 +1,24 @@
+#!/usr/bin/env bash
+# Copyright 2020 Nordic Semiconductor ASA
+# SPDX-License-Identifier: Apache-2.0
+
+source ${ZEPHYR_BASE}/tests/bsim/sh_common.source
+
+# Basic Connected ISO test: a Central connects to 1 Peripheral and tests RTN=2,
+# FT=2, skips 2 subevents in the peripheral
+simulation_id="connected_iso_acl_first_ft_per_skip_2_se"
+verbosity_level=2
+EXECUTE_TIMEOUT=60
+
+cd ${BSIM_OUT_PATH}/bin
+
+Execute ./bs_${BOARD}_tests_bsim_bluetooth_ll_cis_prj_conf_overlay-acl_first_ft_per_skip_2_se_conf \
+  -v=${verbosity_level} -s=${simulation_id} -d=0 -testid=central
+
+Execute ./bs_${BOARD}_tests_bsim_bluetooth_ll_cis_prj_conf_overlay-acl_first_ft_per_skip_2_se_conf \
+  -v=${verbosity_level} -s=${simulation_id} -d=1 -testid=peripheral
+
+Execute ./bs_2G4_phy_v1 -v=${verbosity_level} -s=${simulation_id} \
+  -D=2 -sim_length=30e6 $@
+
+wait_for_background_jobs

tests/bsim/bluetooth/ll/compile.sh

@@ -39,6 +39,7 @@ app=tests/bsim/bluetooth/ll/cis conf_overlay=overlay-legacy_adv_acl_first.conf c
 app=tests/bsim/bluetooth/ll/cis conf_overlay=overlay-acl_group.conf compile
 app=tests/bsim/bluetooth/ll/cis conf_overlay=overlay-acl_group_acl_first.conf compile
 app=tests/bsim/bluetooth/ll/cis conf_overlay=overlay-peripheral_cis.conf compile
+app=tests/bsim/bluetooth/ll/cis conf_overlay=overlay-acl_first_ft_per_skip_2_se.conf compile
 
 app=tests/bsim/bluetooth/ll/edtt/hci_test_app \
   conf_file=prj_dut_llcp.conf compile