Add pipewire sound driver #2166
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
Author
|
@akrram09 Feel free to check this out. I did start working on this over a year ago, but had totally missed how to get the default metadata. Still not sure which properties should be used though: const DEFAULT_SINK_KEY: &str = "default.configured.audio.sink";
const DEFAULT_SOURCE_KEY: &str = "default.configured.audio.source";or const DEFAULT_SINK_KEY: &str = "default.audio.sink";
const DEFAULT_SOURCE_KEY: &str = "default.audio.source"; |
bim9262
force-pushed
the
pipewire_sound
branch
2 times, most recently
from
4ec751b to
c71710f
Compare
Collaborator
Author
|
@MaxVerevkin, I'm curious if you have any thoughts on removing the idle loop. I thought about spawning an async runtime so that we can select from all of the |
This comment was marked as resolved.
This comment was marked as resolved.
MaxVerevkin
reviewed
Jun 10, 2025
Collaborator
Author
This is a good question, that I do not have a good answer to 😓 |
Collaborator
Author
|
@MaxVerevkin, I found a solution to having the idle loop, there is a pw channel that you can attach to the mainloop: https://pipewire.pages.freedesktop.org/pipewire-rs/pipewire/channel/index.html |
Collaborator
|
This doesn't always work when I (un)plug my wired headphones. For example I click to toggle mute, but nothing happens. It usually starts working again after I open pavucontrol and manipulate the volume from there. |
Collaborator
Author
|
Can you capture the output of When I do that I see something like: |
bim9262
force-pushed
the
pipewire_sound
branch
3 times, most recently
from
23c0731 to
e4db55f
Compare
Collaborator
|
Sometimes it just doesn't work even without headphones, and restarting the bar does not help. But after manually toggling mute from pavucontrol it works again. But then I still haven't found a way to get back to the state in which it doesn't work. Anyway, below is the full log of plugging/unplugging. Plug: Unplug: |
Collaborator
Author
|
Alright, so I hadn't tested it using a headphone jack on the computer before. I had been using a USB DAC which is a Also added in the equivalent of the |
Collaborator
Author
|
@MaxVerevkin did that last push resolve the issue you were seeing? |
Collaborator
|
Still have the same issue. It sometimes doesn't work even without headphones, just the laptop speakers. |
Collaborator
|
I'll try to debug it in more depth when I gen some time. |
bim9262
commented
Jun 16, 2025
src/pipewire.rs
Outdated
| } else if data.links.remove(&uid).is_some() { | ||
| update_copy2.replace_with(|v| *v | EventKind::LINK_REMOVED); | ||
| } else if data.ports.remove(&uid) { | ||
| update_copy2.replace_with(|v| *v | EventKind::PORT_ADDED); |
Collaborator
Author
There was a problem hiding this comment.
This should have been PORT_REMOVED 🤦♂️
Collaborator
Author
|
I found that when I use the headphone jack with the Pro Audio profile that setting the volume doesn't work, but updates to the volume (made in another tool) do show up in the status bar. |
Collaborator
Author
|
I found that some outputs update the Node+Device while others only require the Node to be updated. Please let me know if you see any issue after this change 🙏 Node+DeviceNode only. |
Collaborator
|
Still the same 😢 |
Collaborator
Author
|
@MaxVerevkin assuming that at the very least #!/bin/python
import sys
import re
section = None
id = None
type_ = None
param = ""
section_re = re.compile(r"^(added|removed|changed):$")
param_id = re.compile(r"^[\* ]\t id:")
for line in sys.stdin:
section_match = section_re.match(line)
if section_match:
section = section_match.group(1)
id = None
type_ = None
param = ""
elif line.startswith("\tid:"):
id = line.rstrip()
elif line.startswith("\ttype:"):
type_ = line.rstrip()
if section == "changed":
if param_id.match(line):
if "channelVolumes" in param:
print(id)
print(type_)
print(param)
param = ""
param += lineIf you could run In the case that |
Collaborator
Author
|
I just restarted my computer and noticed that the mute/setting the volume using the pipewire block didn't work (nor did changes made with the pulse block cause updates to the pipewire block). I ran the script I previously shared while setting the volume with the pulse block and got nothing. Upon opening pavucontrol (end even after exiting) both blocks started working again... I checked the relevant services and the all seem to be enabled and running 😕 |
Contributor
I can confirm that this is still an issue (not fixed by a recent pipewire release or anything). |
bim9262
force-pushed
the
pipewire_sound
branch
2 times, most recently
from
ca2f282 to
917d3e1
Compare
Collaborator
Author
|
@MaxVerevkin @Stebalien I've just updated the pipewire dependency to the latest version. I'm having trouble reproducing the issue (consistently anyways). Can you see if the behavior improved now or see if looking the log output with the script shared here: #2166 (comment) helps? |
Contributor
|
Note: I get nothing when filtering through your script Also note:
|
Contributor
|
I believe I've found the upstream issue: https://gitlab.freedesktop.org/pipewire/wireplumber/-/issues/822. |
Collaborator
Author
|
Thanks for your research @Stebalien. It does seem like an upstream issue 🫤 |
Contributor
|
In case you didn't see it, wireplumber pointed me at https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4949 which is almost certainly my issue, at least. |
bim9262
force-pushed
the
pipewire_sound
branch
from
917d3e1 to
303c5ea
Compare
Collaborator
Author
|
I did @Stebalien , thanks! I just added in a workaround for the loopback issue that opens a stream to input/output for just a moment when the bar is starting up. That seems to fix it on my end. Could you verify the fix? @Stebalien @MaxVerevkin ? |
Contributor
|
The workaround fixes the issue (unless I restart pipewire while i3bar is running, but I'm unlikely to do that in practice). |
Collaborator
Author
|
To be fair, restarting pipewire will break most things that use it (for example I tested spotify and chrome) and they'll need to be restarted as well. |
Collaborator
Sorry I cannot test right now since I don't have a linux machine with me :( |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #2164