Hi Pauli, On Mon, May 22, 2023 at 1:20 PM <patchwork-bot+bluetooth@xxxxxxxxxx> wrote: > > Hello: > > This series was applied to bluetooth/bluez.git (master) > by Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>: > > On Sun, 21 May 2023 15:27:34 +0000 you wrote: > > There is a race between multiple connect() for CIS in the same CIG. > > connect() will both reconfigure the CIG and connect the CIS, but CIG > > cannot be reconfigured once one CIS has already connected. That these > > tests pass currently relies on some timing/event ordering in the > > emulator. > > > > Connecting multiple CIS in same CIG is meant to be done using > > DEFER_SETUP, so change the tests to use it, so we test the intended > > usage. > > > > [...] > > Here is the summary with links: > - [BlueZ,1/5] iso-tester: always use DEFER_SETUP for multiple CIS in same CIG > https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=a88c74e29b6e > - [BlueZ,2/5] shared/tester: retain test failure status > https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=195b9abbae0e > - [BlueZ,3/5] btdev: support multiple CIG > https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=678265f37c28 > - [BlueZ,4/5] btdev: report right reason for local Disconnect complete > https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=0c32cfdf9462 > - [BlueZ,5/5] iso-tester: add tests for multiple simultaneous CIG > (no matching commit) Ive ended up making some changes to patch 5/5 so it combines AC 1 with AC 2, we can probably extend these tests to have more combinations of different AC, etc, just to make sure the code can handle them properly. -- Luiz Augusto von Dentz
