Skip to content

audio: kpb: stream handling fixes - #11446

Merged
lgirdwood merged 3 commits into
thesofproject:mainfrom
abonislawski:kpb_fixes
Nov 12, 2026
Merged

lgirdwood merged 3 commits into
thesofproject:mainfrom
abonislawski:kpb_fixes

Conversation

@abonislawski

Copy link
Copy Markdown
Member

Handle detection-only configurations without a drain sink and keep the realtime path running when audio is available. Handle ring-buffer wraparound when copying selected microphone channels.

Treat an unbound drain sink as a valid notification-only configuration
while preserving errors for bound but inactive sinks.

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
Return PPL_STATUS_PATH_STOP only when the realtime sink has no pending data,
allowing downstream modules to consume newly produced or queued audio.

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
Handle source ring-buffer wraparound when reading
selected microphone channels.

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
Copilot AI balanced review requested due to automatic review settings November 12, 2026 09:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues are reported for the KPB stream-handling changes.

No open issues 🎉

What changed in this PR

This PR fixes Keyword Processing Buffer (KPB) stream handling for detection-only configurations and realtime audio processing.

Changes:

  • Ignore drain requests when no drain sink is connected.
  • Allow downstream processing to continue when the realtime sink has audio available.
  • Handle input ring-buffer wraparound when copying selected microphone channels in both optimized and generic implementations.
File Description
src/​audio/​kpb.c Fixes optional drain-sink handling, realtime path continuation, and microphone-selection input wraparound.

Review effort: Balanced

@intel-sofci

Copy link
Copy Markdown

PR 11446: test results

Run date: 2026-11-12 19:17 UTC

Tested commit: c2a4781ec4e516210afc283f608104e1c00c417c

mtl pass rate lnl pass rate ptl pass rate wcl pass rate nvl pass rate

@lgirdwood
lgirdwood merged commit 5f46e45 into thesofproject:main Nov 12, 2026
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants