RFR (M) 8221183: Avoid code cache walk in MetadataOnStackMark

[Erik Österlund]

Hi Coleen,

Seems like the recreate_old_table() should be called reset_old_table() 
as it does not really create anything.

The CodeCache::mark_all_nmethods_for_deoptimization function seems like 
a general facility. I don't think I like passing in a boolean if class 
redefinition-specific stuff should be done in there or not. It's a very 
small method that just iterates over the nmethods. I'd rather duplicate 
that while loop in a new function and have the class redefinition logic 
be separate.

Otherwise this looks good!

Thanks,
/Erik

On 2019-03-27 14:09, coleen.phillimore at oracle.com wrote:
> Summary: Note nmethods with "old" Methods in them in table to walk instead.
> 
> See RFE for more details.  Tested with RedefineClasses tests 
> with/without -Xcomp, mach5 tier1-5.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/2019/8221183.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8221183
> 
> Thanks,
> Coleen