mbox series

[BlueZ,0/6] Add new BAP BSRC/SCC tests

Message ID 20240527075422.18953-1-iulia.tanasescu@nxp.com
Headers show
Series Add new BAP BSRC/SCC tests | expand

Message

Iulia Tanasescu May 27, 2024, 7:54 a.m. UTC 
This patch adds BAP unit tests for Broadcast Source stream
establishment (BAP/BSRC/SCC/BV-35-C [Establishes Broadcast] - page 175),
stream disable (BAP/BSRC/SCC/BV-36-C [Disables Broadcast] - page 176) and
stream release (BAP/BSRC/SCC/BV-37-C [Releases Broadcast] - page 176).

This patch also adds shared/bap fixes regarding broadcast stream
management, required for successful test implementation.

Iulia Tanasescu (6):
  shared/bap: Remove DISABLING case from bcast state cb
  shared/bap: Fix potential stream access after free
  test-bap: Update bsrc_state to just handle CONFIG state
  test-bap: Add Broadcast Source Establish test
  test-bap: Add Broadcast Source Disable test
  test-bap: Add Broadcast Source Release test

 src/shared/bap.c |  26 ++++--
 unit/test-bap.c  | 210 ++++++++++++++++++++++++++++++++++++++---------
 2 files changed, 186 insertions(+), 50 deletions(-)


base-commit: 75893035705da57efd6f8a84bba77d596c463d34

Comments

Luiz Augusto von Dentz May 31, 2024, 5:17 p.m. UTC | #1 
Hi Iulia,

On Mon, May 27, 2024 at 3:54 AM Iulia Tanasescu <iulia.tanasescu@nxp.com> wrote:
>
> This patch adds BAP unit tests for Broadcast Source stream
> establishment (BAP/BSRC/SCC/BV-35-C [Establishes Broadcast] - page 175),
> stream disable (BAP/BSRC/SCC/BV-36-C [Disables Broadcast] - page 176) and
> stream release (BAP/BSRC/SCC/BV-37-C [Releases Broadcast] - page 176).
>
> This patch also adds shared/bap fixes regarding broadcast stream
> management, required for successful test implementation.
>
> Iulia Tanasescu (6):
>   shared/bap: Remove DISABLING case from bcast state cb
>   shared/bap: Fix potential stream access after free
>   test-bap: Update bsrc_state to just handle CONFIG state
>   test-bap: Add Broadcast Source Establish test
>   test-bap: Add Broadcast Source Disable test
>   test-bap: Add Broadcast Source Release test
>
>  src/shared/bap.c |  26 ++++--
>  unit/test-bap.c  | 210 ++++++++++++++++++++++++++++++++++++++---------
>  2 files changed, 186 insertions(+), 50 deletions(-)
>
>
> base-commit: 75893035705da57efd6f8a84bba77d596c463d34
> --
> 2.39.2

I'm off this week, will review this set next week.
patchwork-bot+bluetooth@kernel.org June 3, 2024, 1:50 p.m. UTC | #2 
Hello:

This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Mon, 27 May 2024 10:54:16 +0300 you wrote:
> This patch adds BAP unit tests for Broadcast Source stream
> establishment (BAP/BSRC/SCC/BV-35-C [Establishes Broadcast] - page 175),
> stream disable (BAP/BSRC/SCC/BV-36-C [Disables Broadcast] - page 176) and
> stream release (BAP/BSRC/SCC/BV-37-C [Releases Broadcast] - page 176).
> 
> This patch also adds shared/bap fixes regarding broadcast stream
> management, required for successful test implementation.
> 
> [...]

Here is the summary with links:
  - [BlueZ,1/6] shared/bap: Remove DISABLING case from bcast state cb
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=491e5b353700
  - [BlueZ,2/6] shared/bap: Fix potential stream access after free
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=af2873b26912
  - [BlueZ,3/6] test-bap: Update bsrc_state to just handle CONFIG state
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=6cb268f47f47
  - [BlueZ,4/6] test-bap: Add Broadcast Source Establish test
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=bb30b1990154
  - [BlueZ,5/6] test-bap: Add Broadcast Source Disable test
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=06a905d6b8c8
  - [BlueZ,6/6] test-bap: Add Broadcast Source Release test
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=3f747788c1b1

You are awesome, thank you!