Closed Bug 1370598 Opened 4 years ago Closed 4 years ago

Don't cap latency at 512 frames on Macs that are not Macbooks or Macbook Air.

Categories

(Core :: Audio/Video: cubeb, defect, P3)

Unspecified
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: TD-Linux, Assigned: padenot)

References

Details

Attachments

(1 file)

pacmd list-sink-inputs shows:

    index: 252
	driver: <protocol-native.c>
	flags: START_CORKED 
	state: RUNNING
	sink: 1 <alsa_output.pci-0000_00_1b.0.analog-stereo>
	volume: front-left: 65536 / 100% / 0.00 dB,   front-right: 65536 / 100% / 0.00 dB
	        balance 0.00
	muted: no
	current latency: 224.47 ms
	requested latency: 6.26 ms
	sample spec: float32le 2ch 44100Hz
	channel map: front-left,front-right
	             Stereo
	resample method: copy
	module: 13
	client: 57 <Nightly>
	properties:
		media.name = "AudioCallbackDriver"
		application.name = "Nightly"
		native-protocol.peer = "UNIX socket client"
		native-protocol.version = "32"
		application.process.id = "24891"
		application.process.user = "thomas"
		application.process.host = "homura"
		application.process.binary = "firefox"
		application.language = "en_US.utf8"
		window.x11.display = ":1"
		application.process.machine_id = "8e4cbfea404512ae70096c6202c9a3bf"
		application.process.session_id = "3"
		application.icon_name = "firefox"
		module-stream-restore.id = "sink-input-by-application-name:Nightly"

For some reason the current latency is much higher than the requested latency. One reason may be that Firefox is pushing frames into Pulseaudio long before it needs to. 224ms isn't horrible, but it could be much better.
Rank: 25
Priority: -- → P2
Was the load very high maybe ?
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Comment on attachment 8942922 [details]
Bug 1370598 - Don't cap latency at 512 frames on Macs that are not Macbooks or Macbook Air.

https://reviewboard.mozilla.org/r/213190/#review218890

Looks good!
Attachment #8942922 - Flags: review?(achronop) → review+
Pushed by paul@paul.cx:
https://hg.mozilla.org/integration/autoland/rev/f745765d92c4
Don't cap latency at 512 frames on Macs that are not Macbooks or Macbook Air. r=achronop
Oh no, this is the wrong bug...
Filed bug 1430827 for this.
https://hg.mozilla.org/mozilla-central/rev/f745765d92c4
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Assignee: nobody → padenot
Summary: Pulseaudio actual latency is much higher than requested latency with WebRTC → Don't cap latency at 512 frames on Macs that are not Macbooks or Macbook Air.
You need to log in before you can comment on or make changes to this bug.