Skip to content

GTK: move audio bell processing to the application - #13657

Merged
jcollie merged 1 commit into
ghostty-org:mainfrom
jcollie:gtk-bell-threads
Aug 6, 2026
Merged

jcollie merged 1 commit into
ghostty-org:mainfrom
jcollie:gtk-bell-threads

Conversation

@jcollie

@jcollie jcollie commented Aug 6, 2026

Copy link
Copy Markdown
Member

This fixes #13647 by using at most one GStreamer thread per application. This was previously addressed in #12815 which used at most one GStreamer thread per surface. Originally discussed in #12808.

This fixes ghostty-org#13647 by using at most one GStreamer thread per application.
This was previously addressed in ghostty-org#12815 which used at most one GStreamer
thread per surface. Originally discussed in ghostty-org#12808.
@jcollie
jcollie requested a review from a team as a code owner August 6, 2026 01:13
@ghostty-bot ghostty-bot Bot added the gtk Issues exclusive to GTK which are not OS-specific label Aug 6, 2026
@tristan957

Copy link
Copy Markdown
Member

Merge at will. This makes a ton of sense.

@jcollie
jcollie enabled auto-merge August 6, 2026 03:23
@jcollie
jcollie merged commit e1fc390 into ghostty-org:main Aug 6, 2026
195 of 199 checks passed
@jcollie
jcollie deleted the gtk-bell-threads branch August 6, 2026 03:34
@github-actions github-actions Bot added this to the 1.4.0 milestone Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gtk Issues exclusive to GTK which are not OS-specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GTK: audio bell leaks a GStreamer pipeline + GL-context threads per ring (idle CPU climbs over time)

2 participants