Closed Bug 493103 Opened 15 years ago Closed 15 years ago

add deadlock asserts to Win32 VMPI spinlock

Categories

(Tamarin Graveyard :: Virtual Machine, defect, P3)

All
Windows Vista
defect

Tracking

(Not tracked)

VERIFIED FIXED
flash10.1

People

(Reporter: cpeterson, Assigned: lhansen)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.0.10) Gecko/2009042316 Firefox/3.0.10 (.NET CLR 3.5.30729)
Build Identifier: 

The Win32 VMPI spinlock implementation (SpinLockWin) does not allow for recursive locking. These asserts would have been helpful when debugging a recent deadlock.

This patch also combines the Win32 and Win64 code paths.

Reproducible: Always




P4 Change 523793 by chrispet@chrispet-vista on 2009/05/14 15:16:44
Attached patch patchSplinter Review
Attachment #377548 - Attachment is patch: true
Attachment #377548 - Attachment mime type: application/octet-stream → text/plain
Attachment #377548 - Flags: review?(stejohns)
Attachment #377548 - Flags: review?(treilly)
Attachment #377548 - Flags: review?(stejohns)
Attachment #377548 - Flags: review?(lhansen)
Comment on attachment 377548 [details] [diff] [review]
patch

reassigning review to Lars + Tommy
Comment on attachment 377548 [details] [diff] [review]
patch

This looks OK to me, and I appreciate the #ifdef removal as well.

If Tommy approves as well I can land it (I recently refactored the Windows spinlocks so the patch won't apply cleanly but there should be no other complications).
Attachment #377548 - Flags: review?(lhansen) → review+
Attachment #377548 - Flags: review?(treilly) → review+
Assignee: nobody → lhansen
Flags: flashplayer-qrb+
Priority: -- → P3
Target Milestone: --- → flash10.x
This patch didn't land but I did land one that added these assert for mac and win platforms.
Status: UNCONFIRMED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: