webrev review request for 6880029
Tom Rodriguez
Thomas.Rodriguez at Sun.COM
Wed Sep 23 15:00:24 PDT 2009
Doesn't that introduce a race in the initialization?
inc_page_trap_count is called from the safepointing thread so multiple
threads could perform the initialization. Why can't being_statistics
be moved up above the guarding of the polling page? BTW, you used the
wrong bug id in the comment.
tom
On Sep 23, 2009, at 2:35 PM, Xiaobin Lu wrote:
> Webrev: http://javaweb.sfbay/~xl116366/webrev/6880029/
>
> Details:
>
> The bug was uncovered by CBOE when their application crashed pretty
> early during startup when -XX:+PrintSafepointStatistics was
> specified. The root cause is that the polling page exception used by
> JIT could occur without safepoint being involved and the array which
> is used to record the page trap count hasn't been initialized at
> that point. The fix is to do initialization at the point where we
> try to record the page trap count.
>
> Verified by: CBOE
>
> Thanks,
>
> -Xiaobin
More information about the hotspot-runtime-dev
mailing list