patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 8 Packages Projects Releases Wiki Activity

tests: bsim: Bluetooth: Reduce multiple id test execution iterations

Browse source 
Reduce the iterations in multiple id multiple simultaneous
connections test, and tune the buffer counts to catch any
buffer leak related regressions.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
This commit is contained in:
Vinayak Kariappa Chettimada 2024-04-26 15:22:39 +02:00 committed by Anas Nashif
parent c878e3fcad
commit 5258b528f7
3 changed files with 16 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
tests/bsim/bluetooth/ll/multiple_id/prj.conf
View file

@ -34,7 +34,13 @@ CONFIG_BT_CTLR_DATA_LENGTH_MAX=251
# Each PHY update can pause connections for 6 interval hence to let other # Each PHY update can pause connections for 6 interval hence to let other
# parallel connection establishment to succeed increase Rx buffer count. # parallel connection establishment to succeed increase Rx buffer count.
CONFIG_BT_CTLR_RX_BUFFERS=6 # A minimum of 1 Rx buffer is required to receive data PDU, during control
# procedures Rx buffer could be held, preventing new connections to be
# established, a value of 3 is tuned based on this test case execution for 2
# iterations. If there is buffer leak, simulated by using value of 2 here, the
# test is failing.
# If there is buffer leak, this test now should catch it.
CONFIG_BT_CTLR_RX_BUFFERS=3
# Provide enough spacing between connections so that multiple peripheral roles # Provide enough spacing between connections so that multiple peripheral roles
# when connected to a single peer device (peripheral_identity sample) have # when connected to a single peer device (peripheral_identity sample) have

9
tests/bsim/bluetooth/ll/multiple_id/src/main.c
View file

@ -18,7 +18,12 @@
#include "time_machine.h" #include "time_machine.h"
#include "bstests.h" #include "bstests.h"
#define ITERATIONS 10 /* The test case is performing 250 simultaneous connections and managing
* parallel control procedures utilizing the available/configured minimum
* buffer counts. Hence, two iterations of connect-disconnect should be
* sufficient to catch any regressions/buffer leaks.
*/
#define ITERATIONS 2
int init_central(uint8_t iterations); int init_central(uint8_t iterations);
int init_peripheral(uint8_t iterations); int init_peripheral(uint8_t iterations);
@ -81,7 +86,7 @@ exit:
static void test_multiple_init(void) static void test_multiple_init(void)
{ {
bst_ticker_set_next_tick_absolute(4500e6); bst_ticker_set_next_tick_absolute(1500e6);
bst_result = In_progress; bst_result = In_progress;
} }

4
tests/bsim/bluetooth/ll/multiple_id/tests_scripts/multiple.sh
View file

@ -7,7 +7,7 @@ source ${ZEPHYR_BASE}/tests/bsim/sh_common.source
# Multiple connection between two devices with multiple peripheral identity # Multiple connection between two devices with multiple peripheral identity
simulation_id="multiple" simulation_id="multiple"
verbosity_level=2 verbosity_level=2
EXECUTE_TIMEOUT=2200 EXECUTE_TIMEOUT=1600
cd ${BSIM_OUT_PATH}/bin cd ${BSIM_OUT_PATH}/bin
@ -18,6 +18,6 @@ Execute ./bs_${BOARD_TS}_tests_bsim_bluetooth_ll_multiple_id_prj_conf\
-v=${verbosity_level} -s=${simulation_id} -d=1 -testid=peripheral -v=${verbosity_level} -s=${simulation_id} -d=1 -testid=peripheral
Execute ./bs_2G4_phy_v1 -v=${verbosity_level} -s=${simulation_id} \ Execute ./bs_2G4_phy_v1 -v=${verbosity_level} -s=${simulation_id} \
-D=2 -sim_length=4500e6 $@ -argschannel -at=40 -D=2 -sim_length=1800e6 $@ -argschannel -at=40
wait_for_background_jobs wait_for_background_jobs