Closed Bug 422252 Opened 16 years ago Closed 13 years ago

Avoid priority inversion deadlocks in GCHeap/FixedMalloc

Categories

(Tamarin Graveyard :: Garbage Collection (mmGC), defect)

Product:
Tamarin Graveyard
Component:
Garbage Collection (mmGC)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX
Milestone:
Future

People

(Reporter: treilly, Unassigned)

Details

We should replace these with something that avoids priority inversion deadlocks.  Ideally something that backs off to an OS lock, maybe NSPR has something we re-use.
Status: NEW → ASSIGNED
cleaning out old bugs. set triage+, any change in status?
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
Its possible this is causing a month long mysterious hang/crash on wince.
besides real locks another thing we could do is have a thread local DisableAllocation notion where the allocator will return NULL and assert when its engaged (to prevent further issues in this area).
Flags: flashplayer-qrb? → flashplayer-qrb+
Priority: -- → P2
Target Milestone: --- → flash10.x
(In reply to comment #0)
> We should replace these with something that avoids priority inversion
> deadlocks.  Ideally something that backs off to an OS lock, maybe NSPR has
> something we re-use.

I wonder if the "futex" approach would be appropriate here: http://en.wikipedia.org/wiki/Futex
Summary: Replace inifinite spin locks in GCHeap/FixedMalloc → Replace infinite spin locks in GCHeap/FixedMalloc
this isn't a super high priority since the priority deadlock issue has been fixed on win32 with a periodic sleep so this bug's priority can probably be lowered
Deferring.  I think we probably need a thorough review of locking/threading strategies in the GC, post-Argo.
Assignee: treilly → nobody
Status: ASSIGNED → NEW
Priority: P2 → --
Target Milestone: flash10.1 → Future
Priority: -- → P3
Target Milestone: Future → flash10.2
Summary: Replace infinite spin locks in GCHeap/FixedMalloc → Avoid priority inversion deadlocks in GCHeap/FixedMalloc
Assignee: nobody → siwilkin
Assignee: siwilkin → nobody
Priority: P3 → --
Target Milestone: flash10.x - Serrano → Future
Crypto-duplicate of broader threading agenda.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.