https://issues.apache.org/bugzilla/show_bug.cgi?id=47744

--- Comment #4 from Greg Vanore <gvan...@solers.com> 2009-10-26 09:33:27 UTC ---
(In reply to comment #3)
> I'm not a big fan of trying to work around JVM bugs, but this patch looks like
> it is OK (once the missing synchronizing is added).  Synchronizing will add a
> small hit, but this part of the code is well off the critical path.

Yes, I agree with everything you said, except Sun has closed and concluded it
is *not* a problem with the VM:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6877740 

A bit after posting this I revised my local copy of the patch to add
synchronization - should I repost the patch to include it?

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to