fix(audio): three 100% CPU busy loops on the _pa path

The Linux audio service in `--server` ignored the `Err` from `next_raw()`, so
once the cm-side `_pa` peer closed, every iteration re-polled a dead socket:
tokio-util's paused `Framed` issues one 0-byte read per poll and returns ready
at once, never `Pending`. The thread never parked and burned a full core for
the life of the process. Propagate instead, so `ServiceTmpl::run`'s existing
backoff ends the inner loop and reconnects.

Two sibling loops on the same audio path have the same shape:

- `ipc::start_pa` (runs in `--cm`) ignored the `Err` from
  `psimple::Simple::read`, so a dead pulse handle spins there instead.
- `start_voice_call`'s forwarding thread polls two channels with `try_recv`
  and has no blocking primitive at all: measured 99.8% of a core for the whole
  call, against 1.0% with a 1 ms pause (audio packets arrive every 10 ms).

fix https://github.com/rustdesk/rustdesk/issues/16226

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
rustdesk
2026-09-15 17:23:29 +08:00
parent 82b6fc2f3b
commit 7e1c45c370
3 changed files with 15 additions and 2 deletions

View File

@@ -124,7 +124,11 @@ mod pa_impl {
})?;
#[cfg(target_os = "linux")]
if let Ok(data) = stream.next_raw().await {
{
// The `_pa` peer closing surfaces as `Err` here. Dropping it left the loop polling
// a dead socket -- one 0-byte read per poll, ready at once and never `Pending` --
// which burned a full core for the rest of the process lifetime.
let data = stream.next_raw().await?;
if data.len() == 0 {
send_f32(&zero_audio_frame, &mut encoder, &sp);
continue;