Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8272065

jcmd cannot rely on the old core reflection implementation which will be changed after JEP 416

    XMLWordPrintable

Details

    • Enhancement
    • Resolution: Fixed
    • P3
    • 18
    • 18
    • hotspot
    • None
    • svc
    • b23

    Description

      jcmd VM.metaspace and VM.classloaders and VM.class_hierarchy implementation relies on the internals of the old core reflection implementation to find out the class name of the target reflected member.

      It will no longer work with the new core reflection implementation by JEP 416.

      test/hotspot/jtreg/serviceability/dcmd/vm/ShowReflectionTargetTest.java is temporarily changed to run with the old core reflection. This test should be changed to take out ` -Djdk.reflect.useDirectMethodHandle=false` to run with the new implementation.

      Attachments

        1. failed-stdout.txt
          100 kB
        2. stdout-org.txt
          99 kB
        3. stdout-with-prop.txt
          99 kB

        Issue Links

          duplicates

          Enhancement - null JDK-8210510 JCMD's dependency on hardcoded Constant Pool indexes should be eliminated

          • P3 - Major loss of function.
          • Closed
          relates to

          Enhancement - null JDK-8210510 JCMD's dependency on hardcoded Constant Pool indexes should be eliminated

          • P3 - Major loss of function.
          • Closed

          Enhancement - null JDK-8271820 Implementation of JEP 416: Reimplement Core Reflection with Method Handle

          • P3 - Major loss of function.
          • Resolved

          Enhancement - null JDK-8203343 VM.{metaspace|classloaders|classhierarchy...} jcmd should show invocation targets for Generated{Method|Constructor}AccessorImpl classes

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          links to

          Commit Commit openjdk/jdk/7281861e

          Review Review openjdk/jdk/6257

          Activity

            People

              stuefe Thomas Stuefe
              mchung Mandy Chung
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: