Closed Bug 1503055 Opened 6 years ago Closed 6 years ago

Avoid deadlocking when rewinding to avoid a different deadlock.

Categories

(Core Graveyard :: Web Replay, enhancement)

enhancement
Not set
normal

Tracking

(firefox65 fixed)

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: bhackett1024, Assigned: bhackett1024)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
The attached patch fixes an issue in part 3 of bug 1500805, where we rewind to avoid deadlocking but while doing so are holding a spinlock which other threads might need in order to create or destroy the information associated with each platform mutex.  In order to rewind we need all other threads to enter a specific idle state, and to reach such a state they might need to take this spinlock, so we end up deadlocking.  The fix releases the spinlock before starting the rewind.
Attachment #9020956 - Flags: review?(continuation)
Attachment #9020956 - Flags: review?(continuation) → review+
Pushed by bhackett@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/62a1d3756cc2
Avoid deadlocking when rewinding to avoid a different deadlock, r=mccr8.
https://hg.mozilla.org/mozilla-central/rev/62a1d3756cc2
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: