Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 341849 - Deadlock shutting down UI thread
Summary: Deadlock shutting down UI thread
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.3   Edit
Hardware: All All
: P3 major with 1 vote (vote)
Target Milestone: 1.5 M4   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-04 17:49 EDT by Gareth Bowen CLA
Modified: 2011-11-14 16:27 EST (History)
0 users

See Also:

Attachments
Heap dump of the deadlock (240.12 KB, text/plain)
2011-04-04 17:50 EDT, Gareth Bowen CLA 		no flags Details
Patch (1.97 KB, patch)
2011-04-04 19:08 EDT, Gareth Bowen CLA 		ruediger.herrmann: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gareth Bowen CLA 2011-04-04 17:49:37 EDT 
Build Identifier: 1.3.2.20110216-1023

Occasionally when running a deadlock occurs requiring a full restart (full heap dump attached). This seems to be due to the UI thread trying to shutdown while holding a lock. The shutdown then blocks on the Device.deviceLock.

I think this was caused by applying the fix in bug 316676 which moved the try catch block inside the synchronized block. I believe this can be fixed without regressing on the original bug by putting the outer try catch block outside the synchronized block but leaving the inner try catch block inside the synchronized block. This would mean the stateInfo attribute is still set in a synchronized way but the lock would be released before the shutdown of the thread.

Reproducible: Sometimes
Comment 1 Gareth Bowen CLA 2011-04-04 17:50:58 EDT 
Created attachment 192508 [details]
Heap dump of the deadlock
Comment 2 Gareth Bowen CLA 2011-04-04 19:08:17 EDT 
Created attachment 192510 [details]
Patch
Comment 3 Rüdiger Herrmann CLA 2011-11-14 15:54:48 EST 
This looks like a dup or at least related to bug 320182
Comment 4 Rüdiger Herrmann CLA 2011-11-14 15:59:18 EST 
(In reply to comment #3)
> This looks like a dup or at least related to bug 320182
Wrong bug, please disregard comment #3
Comment 5 Rüdiger Herrmann CLA 2011-11-14 16:27:20 EST 
Applied patch to CVS HEAD.