RFR: 8256640: assert(!m->is_old() || ik()->is_being_redefined()) failed: old methods should not be in vtable

[Daniel D.Daugherty]

On Thu, 19 Nov 2020 20:48:53 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> I added an assert with https://bugs.openjdk.java.net/browse/JDK-8256365 to catch adding redefined methods into the vtable where they don't belong (except while redefining that class).  The assert found that CDS was restoring classes with methods in the default_methods array which can be from a redefined class.  So these methods need to be adjusted before recreating the vtable.
>> Also fixed is a potential bug where a Method in a methodHandle in the default_methods array can be redefined in the safepoint caused by loader constraint checking, then added to the vtable.  The window for this bug is very small so I couldn't write a test for it. This change was in my v1 patch for JDK-8256365, so reintroduced here.
>> 
>> The test java/lang/instrument/IsModifiableClassAgent.java now passes with this patch.  Rerunning tier1-6 tests in progress.  Built with minimal VM to verify #if INCLUDE_JVMTIs were in the right places.
>
> Marked as reviewed by dcubed (Reviewer).

Normally when fixing a regression, I would want one of the original
reviewers to chime in on the thread, but it's the wrong time of day
for @dholmes-ora or @fisk.

I'm good with this fix. Your call on whether to address my one nit.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1323