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

Shenandoah: Traversal weak references misses GC phase scope

    Details

    • Type: Enhancement
    • Status: Closed
    • Priority: P4
    • Resolution: Won't Fix
    • Affects Version/s: 8-shenandoah, 11-shenandoah, 14, 15
    • Fix Version/s: None
    • Component/s: hotspot
    • Subcomponent:
      gc

      Description

      Static analysis reveals "phase_process" local variable is unused here:

      void ShenandoahTraversalGC::weak_refs_work_doit() {
        ...

        ShenandoahPhaseTimings::Phase phase_process = ShenandoahPhaseTimings::weakrefs_process;

      Indeed, we are missing ShenandoahGCPhase scope, like ShenandoahConcurrentMark::weak_refs_work_doit() has. New Traversal-specific counter needs to be added, and used with ShenandoahGCPhase here.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                shade Aleksey Shipilev
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: