Thanks JC!
Coleen

On 5/15/19 1:04 PM, Jean Christophe Beyler wrote:
Hi Coleen,

Looks good to me (not an official reviewer),
Jc

*From: *Vladimir Kozlov <[email protected] <mailto:[email protected]>>
*Date: *Wed, May 15, 2019 at 9:34 AM
*To: * <[email protected] <mailto:[email protected]>>, hotspot-dev developers, [email protected] <mailto:[email protected]>

    Looks good.

    Thanks,
    Vladimir

    On 5/15/19 6:46 AM, [email protected]
    <mailto:[email protected]> wrote:
    > Summary: adjust old method table by only one thread.
    >
    > Entries were removed from the redefinition old method table by
    G1 by multiple threads at a safepoint, so didn't take the
    > CodeCache_lock. Moved the removal to nmethod->flush().  The bug
    is confidential because there's a confidential comment
    > not marked confidential but we'll open it up once that's fixed.
    >
    > Passed redefinition tests in repository with "VM_OPTIONS=-Xcomp
    -XX:TieredStopAtLevel=1", including the PCL tests that
    > in the oracle closed repo.
    >
    > Also ran hs-tier1-3.
    >
    > open webrev at
    http://cr.openjdk.java.net/~coleenp/2019/8223585.01/webrev
    > bug link https://bugs.openjdk.java.net/browse/JDK-8223585
    >
    > Thanks,
    > Coleen



--

Thanks,
Jc

Reply via email to