Skip to content

fix: stop audio state from being enforced in the Linphone iterate loop#36

Merged
frankkohlhepp merged 1 commit intodevelopfrom
fix-high-cpu-load
Mar 31, 2026
Merged

fix: stop audio state from being enforced in the Linphone iterate loop#36
frankkohlhepp merged 1 commit intodevelopfrom
fix-high-cpu-load

Conversation

@frankkohlhepp
Copy link
Copy Markdown
Collaborator

@frankkohlhepp frankkohlhepp commented Mar 25, 2026

Fix for sp-7099:

The keepAliveTask TimerTask previously set AudioManager.MODE_IN_COMMUNICATION, isMicrophoneMute=false and isSpeakerphoneOn=false on every tick alongside core.iterate(), which fires every 20ms. This caused ~50 AudioManager system calls per second at all times, routing continuously through system_server and audioserver into the Audio HAL — saturating CPU at 60-115% on idle devices and preventing deep sleep.

The keepAliveTask TimerTask previously set AudioManager.MODE_IN_COMMUNICATION,
isMicrophoneMute=false and isSpeakerphoneOn=false on every tick alongside
core.iterate(), which fires every 20ms. This caused ~50 AudioManager system
calls per second at all times, routing continuously through system_server and
audioserver into the Audio HAL — saturating CPU at 60-115% on idle devices and preventing deep sleep.
@frankkohlhepp frankkohlhepp self-assigned this Mar 25, 2026
Copy link
Copy Markdown
Contributor

@narges-dipir narges-dipir left a comment

Choose a reason for hiding this comment

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

Why is this PR a release PR?

@frankkohlhepp
Copy link
Copy Markdown
Collaborator Author

Why is this PR a release PR?

🤔 Hmm... I just created a normal PR from a branch. What did I do wrong?

@frankkohlhepp
Copy link
Copy Markdown
Collaborator Author

I think the CI/CD pipeline is misconfigured to automatically release an app always. The "tags" prerequisite is commented out. @narges-dipir

@frankkohlhepp frankkohlhepp merged commit d70e293 into develop Mar 31, 2026
1 check failed
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.

2 participants