Skip to content

[PW_SID:1183466] bass: wait for PAST listener readiness - #2662

Open
BluezTestBot wants to merge 5 commits into
masterfrom
1183466
Open

[PW_SID:1183466] bass: wait for PAST listener readiness#2662
BluezTestBot wants to merge 5 commits into
masterfrom
1183466

Conversation

@BluezTestBot

@BluezTestBot BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown

Drop watches / timers in test teardown, not after all tests have run.
Avoids leaking them to following tests.

Assisted-by: opencode:gpt-6.1-sol

tools/iso-tester.c | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)

pv added 5 commits October 11, 2026 22:41
@pv @Vudentz
iso-tester: remove IO sources during teardown
f6a3055 
Drop watches / timers in test teardown, not after all tests have run.
Avoids leaking them to following tests.

Assisted-by: opencode:gpt-6.1-sol
@pv @Vudentz
lib: add BT_ISO_PAST_POLLOUT socket option
e7ba17f 
Add kernel uAPI definition for PAST listener readiness option.

Assisted-by: opencode:qwen3.8-flash-next
@pv @Vudentz
tools: iso-tester: test PAST listener readiness
ad56b9f 
Test POLLOUT notification after LE Set PAST Parameters completes, with
and without deferred setup. Query BT_ISO_PAST_POLLOUT and skip the tests
when listener readiness notifications are unavailable.

Add tests:

ISO Broadcaster PAST Listen Ready Receive - Success
ISO Broadcaster PAST Listen Ready Defer - Success

Assisted-by: opencode:gpt-6.1-sol
@pv @Vudentz
doc: iso: document ISO PAST listener readiness
670dc23 
Document PAST receiver setup and waiting for POLLOUT before requesting
PAST. Describe the read-only BT_ISO_PAST_POLLOUT capability query and
handling kernels without readiness notifications.

Assisted-by: LLM
@pv @Vudentz
bass: wait for PAST listener readiness
d08111b 
Defer SYNC_INFO_RE until the PA sync listener signals POLLOUT, so the
Assistant cannot start PAST before receiver setup completes.

Check kernel support via BT_ISO_PAST_POLLOUT, otherwise skip waiting.

Fixes test_bass_past_transport_* test failures.

Assisted-by: LLM
@codacy-production

codacy-production Bot commented Oct 11, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 32 complexity · 0 duplication

Metric Results
Complexity 32
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

CheckPatch
Desc: Run checkpatch.pl script
Duration: 2.00 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

GitLint
Desc: Run gitlint
Duration: 1.20 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

BuildEll
Desc: Build and Install ELL
Duration: 19.26 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

BluezMake
Desc: Build BlueZ
Duration: 382.69 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

MakeCheck
Desc: Run Bluez Make Check
Duration: 14.01 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

MakeDistcheck
Desc: Run Bluez Make Distcheck
Duration: 138.19 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

CheckValgrind
Desc: Run Bluez Make Check with Valgrind
Duration: 228.68 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

CheckSmatch
Desc: Run smatch tool with source
Duration: 289.47 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

bluezmakeextell
Desc: Build Bluez with External ELL
Duration: 93.09 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

TestFunctional
Desc: Run test-functional
Duration: 1235.95 seconds
Result: FAIL
Output:

FAIL functional.test_bluetoothctl::test_bluetoothctl_pair_bredr[hosts15-vm2]: pytest_bluezenv.rpc.RemoteError: <pexpect.popen_spawn.PopenSpawn object at 0x7fbeb5906510>
    searcher: searcher_re:
        0: re.compile(b'Changing pairable on succeeded')
    <pexpect.popen_spawn.PopenSpawn object at 0x7fbeb5906510>
    searcher: searcher_re:
        0: re.compile(b'Changing pairable on succeeded')
    Remote traceback:
        Traceback (most recent call last):
          File "/usr/local/lib/python3.14/dist-packages/pytest_bluezenv/../pytest_bluezenv/rpc.py", line 102, in server_stream
            result = method(*msg["a"], **msg["kw"])
          File "/usr/local/lib/python3.14/dist-packages/pytest_bluezenv/../pytest_bluezenv/env.py", line 256, in call_plugin
            return getattr(self.plugins[name], method)(*a, **kw)
                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^
          File "/usr/local/lib/python3.14/dist-packages/pytest_bluezenv/../pytest_bluezenv/host_plugins.py", line 502, in expect
            ret = self.ctl.expect(*a, **kw)
          File "/usr/lib/python3/dist-packages/pexpect/spawnbase.py", line 354, in expect
            return self.expect_list(compiled_pattern_list,
                   ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
                    timeout, searchwindowsize, async_)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
          File "/usr/lib/python3/dist-packages/pexpect/spawnbase.py", line 383, in expect_list
            return exp.expect_loop(timeout)
                   ~~~~~~~~~~~~~~~^^^^^^^^^
          File "/usr/lib/python3/dist-packages/pexpect/expect.py", line 181, in expect_loop
            return self.timeout(e)
                   ~~~~~~~~~~~~^^^
          File "/usr/lib/python3/dist-packages/pexpect/expect.py", line 144, in timeout
            raise exc
        pexpect.exceptions.TIMEOUT: <pexpect.popen_spawn.PopenSpawn object at 0x7fbeb5906510>
        searcher: searcher_re:
            0: re.compile(b'Changing pairable on succeeded')
        <pexpect.popen_spawn.PopenSpawn object at 0x7fbeb5906510>
        searcher: searcher_re:
            0: re.compile(b'Changing pairable on succeeded')

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

IncrementalBuild
Desc: Incremental build with the patches in the series
Duration: 749.51 seconds
Result: PASS

@BluezTestBot

BluezTestBot commented Oct 11, 2026

Copy link
Copy Markdown
Author

ScanBuild
Desc: Run Scan Build
Duration: 1116.57 seconds
Result: PASS

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

area:lib area:profiles-audio area:tools

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@BluezTestBot @pv