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

Reorganize GC VM operations and implement is_gc_operation

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Unresolved
    • Icon: P4 P4
    • tbd
    • None
    • hotspot
    • None
    • gc

      JDK-8359110 introduces a need to be able to categorize VM operations to be able to track GC activity in the VM thread for CPU time tracking. This patch will reorganize the shared GC VM operations in gcVMOperations.hpp to make a distinction between GC operations and serviceability/CDS etc. operations performed for the GC.

      Additionally the ability to query is_gc_operation() is introduced in the base class VM_Operation and overridden in appropriate VM operations. This provides a clear and efficient interface to query the category of a VM operation.

      Renaming VM_GC_Sync_Operation to VM_Heap_Sync_Operation to make it clear that all sub-classes is related to GC. While VM_GC_HeapInspection is a serviceability operation it attempts to trigger a collection, hence it is put underneath VM_GC_Collect_Operation->VM_GC_Service_Operation. However only classes that inherits from VM_GC_Collect_Operation are considered to return true for is_gc_operation() as VM_GC_HeapInspection is considered to not belong to "GC activity".

      ZGC and Shenandoah do not use sub-classing of VM_Heap_Sync_Operation, but their respective base classes and special classes for GC activity are marked as returning true for is_gc_operation().

      The only is_XX_operation that is specified in VM_Operation is is_gc_operation to not clutter with cases that are not used (is_gc_operation will be used by JDK-8359110). That being said, these behavioral queries may be extended in case we want to explore CPU time tracking beyond the GC component.

            Unassigned Unassigned
            jnorlinder Jonas Norlinder
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: