Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature][Pipewire]Pipewire Backend #363

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

bradc6
Copy link

@bradc6 bradc6 commented Jun 22, 2023

This includes both the changes to the cmake files and a new Pipewire backend. Please lmk if there is interest in this PR and I'll clean it up more cause this is a rough cut for sure

@bradc6 bradc6 changed the title Pipewire Backend [Feature]Pipewire Backend Jun 22, 2023
@bradc6 bradc6 changed the title [Feature]Pipewire Backend [Feature][Pipewire]Pipewire Backend Jun 22, 2023
@Catty2014
Copy link

Kinda good work, however I've found some issues when stopping playback:

*** pw_stream_destroy called from wrong context, check thread and locking: Operation not permitted
*** impl_ext_end_proxy called from wrong context, check thread and locking: Operation not permitted
*** impl_ext_end_proxy called from wrong context, check thread and locking: Operation not permitted
double free or corruption (!prev)

I'm using pipewire1.2.3.

@Catty2014
Copy link

Also, the buffer size must be set very high(8192 on my machine) to have a fluent playback (512 is sufficient for SDL2 backend). Maybe performance issue?

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