logging fixes for last pre-ZBB JDK7-TL snapshot (6977677, 7016208, 7041595)
Daniel D. Daugherty
daniel.daugherty at oracle.com
Mon May 16 12:52:52 PDT 2011
On 5/16/2011 1:48 PM, Mandy Chung wrote:
> On 05/16/11 12:33, Daniel D. Daugherty wrote:
>>> Would this fix 7016208? It seems to me that addLogger returning
>>> is the cause for this bug.
>> Yes it probably would fix 7016208, but I would prefer to go
>> with what I have. Yes, addLogger() returning a boolean was
>> a bad design choice way back when.
>> We'll have to look at tweaking the Logging API in JDK8.
>> Are you okay if I go with what I have (modulo a few editorial
> Given the amount of testing it takes to verify these fixes, I'm fine with
> pushing your version to JDK 7. Can you put a comment to describe
> this proposed alternative solution so that someone may revise this
> fix in JDK 8 when working on LogManager code?
I'm filing a new bug for JDK8 that describes the results
of my lock analysis and other issues that I found when I
crawled through the code. I will include your suggestion
in that bug. You'll be on the interest list...
Thanks for the review!
More information about the hotspot-runtime-dev