RFR (XS) JDK-8026041: JVM crashes with assert "assert(is_updated()) failed: must not be clear", with -XX:+PrintGCApplicationConcurrentTime in -Xcomp mode

[Coleen Phillimore]

Looks good - thanks for the quick fix and testing.
Coleen

On 10/10/2013 9:08 PM, Christian Tornqvist wrote:
>
> Hi Lois,
>
> Change looks good, thanks for fixing this.
>
> Thanks,
>
> Christian
>
> *From:*hotspot-runtime-dev-bounces at openjdk.java.net 
> [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] *On Behalf Of 
> *Lois Foltan
> *Sent:* Thursday, October 10, 2013 7:12 PM
> *To:* hotspot-runtime-dev at openjdk.java.net
> *Subject:* RFR (XS) JDK-8026041: JVM crashes with assert 
> "assert(is_updated()) failed: must not be clear", with 
> -XX:+PrintGCApplicationConcurrentTime in -Xcomp mode
>
> Please review the following fix:
>
> Webrev:
> http://cr.openjdk.java.net/~hseigel/bug_jdk8026041/ 
> <http://cr.openjdk.java.net/%7Ehseigel/bug_jdk8026041/>
>
> Bug: JVM crashes with assert "assert(is_updated()) failed: must not be 
> clear"
>          with -XX:+PrintGCApplicationConcurrentTime in -Xcomp mode
> https://bugs.openjdk.java.net/browse/JDK-8026041
>
> Summary of fix:
>     Prior to printing the time interval for 
> -XX:+PrintGCApplicationConcurrentTime in
>     RuntimeService::record_safepoint_begin(), check first to verify 
> that the VM initialization
>     is complete.  The recent addition of pre-initialization of Method* 
> classes during VM startup,
> JDK-8022595 <https://bugs.openjdk.java.net/browse/JDK-8022595>, caused 
> enough code to be generated that GuaranteedSafepointInterval timer
>     in VMThread::loop() timed out resulting in an early safepoint.  
> Thank you to Staffan Larsen
>     and David Holmes for troubleshooting this for runtime.
>
> Tests:
>     JTREG, vm.quick.testlist with "-server -Xcomp 
> -XX:+PrintGCApplicationConcurrentTime" specified (in progress)
>
> Thank you, Lois
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20131010/0dab8cf0/attachment.html