bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

races with ReverbInputBuffer::m_writeIndex from ReverbConvolver

NEW
Unassigned

Status

()

Core
Web Audio
P3
normal
3 years ago
10 months ago

People

(Reporter: karlt, Unassigned)

Tracking

(Depends on: 1 bug)

24 Branch
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
ReverbInputBuffer::m_writeIndex is read and written on different threads with locks on neither.  It could be atomic, but probably better to have a separate multi-thread variable to replace m_moreInputBuffered accessed within m_backgroundThreadLock because that lock will be used anyway.

Similarly setting m_moreInputBuffered = false after processing and outside of
the lock looks unhelpful, forcing a wait even if more has been buffered.
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.