RFR [jdk11]: 8212200: assert(on_stack()) failed when shared java.lang.object is redefined by JVMTI agent

[Lindenmaier, Goetz]

Hi Matthias,

I had a look at your change. 

classFile.cpp:
You skipped one chunk, as the fixed code is not in 11. OK.

systemDictionary.hpp
You removed comment
" and a flag word
// that makes some minor distinctions, like whether the klass
// is preloaded, optional, release-specific, etc."
I think this should be kept, as this is still relevant in 11.
Your change does not touch this flag.

Other changes are ok.

heapShared.cpp
Just indentation, ok.

For the fixes to the test you included in your change. 
Please downport them as changes of its own, and 
push the three changes together.

Best regards,
 Goetz.

> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> On Behalf
> Of Baesken, Matthias
> Sent: Tuesday, June 30, 2020 9:36 AM
> To: 'jdk-updates-dev at openjdk.java.net' <jdk-updates-dev at openjdk.java.net>
> Subject: [CAUTION] RFR [jdk11]: 8212200: assert(on_stack()) failed when
> shared java.lang.object is redefined by JVMTI agent
> 
> Hello, please review  the jdk11 backport of  8212200 .
> 
> I had to do a few slight adjustments to adjust the src  changes of  8212200   to
> jdk11 .
> 
> The test
> ( test/hotspot/jtreg/runtime/SharedArchiveFile/serviceability/ReplaceCritical
> Classes.java.)  from the original jdk/jdk webrev  of 8212200   did not work.
> So I had to include adjustments from 8221918 and 8213275  to fix  the test .
> 
> 
> 
> Bug/webrev :
> 
> https://bugs.openjdk.java.net/browse/JDK-8212200
> 
> http://cr.openjdk.java.net/~mbaesken/webrevs/8212200.0/
> 
> 
> Original jdk/jdk webrev :
> 
> http://hg.openjdk.java.net/jdk/jdk/rev/625f6c742392
> 
> 
> Thanks, Matthias