Request for review: 6922246: Deadlock during intensive class loading in parallel

[Tom Rodriguez]

On Oct 19, 2010, at 3:11 PM, coleen phillimore wrote:

> Summary: allocate methodData before taking out the MethodData_lock
> open webrev at http://cr.openjdk.java.net/~coleenp/6922246/
> bug link at http://bugs.sun.com/view_bug.do?bug_id=6922246
> 
> Thanks to Ramki for the diagnosis and suggestion for the fix.

That looks ok, though if the only purpose of MethodData_lock is setting that field we could just as well use a cmpxchg_ptr to set the method data field and no lock.  One less heavyweight lock seems good.

tom

> 
> Coleen