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

HeapRegionManager::find_(un)available_from_idx(_reverse) should return the index as return values

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Duplicate
    • Icon: P4 P4
    • tbd
    • 9
    • hotspot
    • gc

      From the review discussion for JDK-8133456 the following suggestions for improving the interface for two HeapRegionManager methods came up:

      I'd also prefer it slightly if HeapRegionManager::find_available_from_idx_reverse had the index as return value, with the count as the output parameter. Same for HeapRegionManager::find_unavailable_from_idx. Then they would be consistent with find_contiguous and find_highest_free, and there wouldn't be 2 index params to look odd.

      Review thread starts at http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2015-August/014488.html for reference.

            Unassigned Unassigned
            tschatzl Thomas Schatzl
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: