[OpenJDK 2D-Dev] Request for review: 6892485: Deadlock in SunGraphicsEnvironment / FontManager

[Roman Kennke]

Hi there,

this patch fixes the deadlock in FontManager for OpenJDK7 that has been
reported repeatedly on this list. I have a testcase here (attached),
which I could prepare for jtreg, but it is not totally reliable and
depends on the system configuration (which and how many fonts
installed), not sure if we want this.

The problem is that two threads run into font code, locking on two
different locks in wrong order. (Better analysis is in the bug). The fix
is to synchronize only on one lock object (the FontManager instance).

http://cr.openjdk.java.net/~rkennke/6892485/webrev.00/

Ok to commit?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: FontDeadlock.java
Type: text/x-java
Size: 9419 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20100225/bfd45bb5/FontDeadlock.java>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: runfontdeadlock.sh
Type: application/x-shellscript
Size: 61 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20100225/bfd45bb5/runfontdeadlock.sh>