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

Rename CollectedHeap::is_gc_active to be explicitly about STW GCs

    XMLWordPrintable

Details

    • gc
    • b22

    Backports

      Description

        `CollectedHeap::is_gc_active()` is confusing, since its name implies any GC phase is running, while it actually only covers the STW GCs. It would be good to rename it for clarity. The freed-up name, `is_gc_active`, could then be repurposed to track any (concurrent or STW) GC phase running. That would be useful to resolve JDK-8331572.

        Doing this rename separately guarantees we have caught and renamed all current uses.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8333330 Rename CollectedHeap::is_gc_active to be explicitly about STW GCs

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

            Enhancement - null JDK-8331572 Allow using OopMapCache outside of STW GC phases

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

            Enhancement - null JDK-8331719 Cleanup asserts/code after introducing IsSTWGCActiveMark

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

            Enhancement - null JDK-8331721 Introduce CollectedHeap helper methods for any GC in progress

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

            Commit Commit openjdk/jdk21u-dev/2a210449

            Commit Commit openjdk/jdk/1eec30a6

            Review Review openjdk/jdk21u-dev/609

            Review Review openjdk/jdk/19064

            Activity

              People

                shade Aleksey Shipilev
                shade Aleksey Shipilev
                Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: