pop-os / system76-scheduler

Auto-configure CFS and process priorities for improved desktop responsiveness
Mozilla Public License 2.0
545 stars 33 forks source link

fix(pw): insert to active_sessions, avoid creating duplicated threads #85

Closed taoky closed 1 year ago

taoky commented 1 year ago

In daemon/src/pw.rs, pipewire_service() with pw_tx and pw_rx:

BTreeSet<PathBuf> is defined to "deduplicate" socket names. However, it seems that this BTreeSet is always empty, as no one is inserting anything into it. Thus this daemon will actually create a thread every 60s, probably an unexpected behavior.

mmstick commented 1 year ago

I'll merge your PR into the scheduler branch so that they can be reviewed alongside other PRs and QA in one pass.