New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 773302 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Implement GetPreferredOutputStreamParameters for PulseAudio

Project Member Reported by maxmorin@chromium.org, Oct 10 2017

Issue description

GetPreferredOutputStreamParameters doesn't currently query PulseAudio for the native parameters for the device, so we may end up using suboptimal parameters, forcing PulseAudio to resample the data we deliver. Instead, GetPreferredOutputStreamParameters should return parameters such that resampling isn't necessary. Looks like pa_context_get_sink_info_by_name could be used to query for the parameters.
 
Right, I should rather say that it queries the same parameters regardless of which device is used. (It's also not really clear to me from the PA documentation that the "server default" is the same as the native parameters of the default sink, though that would be very resonable).
Cc: julien.isorce@chromium.org
Project Member

Comment 4 by sheriffbot@chromium.org, Oct 22

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by dbbrooks@chromium.org, Today (19 hours ago)

Owner: maxmorin@chromium.org
Status: Assigned (was: Untriaged)
maxmorin@ for triage, routing, or an update. Thanks. 

Max, given the age, would it be OK to close the issue as "Won't fix"?

Comment 6 by maxmorin@chromium.org, Today (8 hours ago)

Owner: ossu@chromium.org
Oskar: Is this something your team would like to do? We'd get rid of some resampling and there's also a fifo in our code which could be eliminated. Otherwise, please won'tfix this one.

Comment 7 by ossu@chromium.org, Today (8 hours ago)

I can take a look and see if the format in PulseAudio is actually device-specific, or if it's a single, shared format regardless of device used. My hunch is that it's the latter.  

Comment 8 by maxmorin@chromium.org, Today (7 hours ago)

As mentioned in #1, there's pa_context_get_sink_info_by_name to get information by sink. You can also check it in the terminal by "pacmd list-sinks" (under "sample spec").

Sign in to add a comment