Skip to content
This repository has been archived by the owner on Sep 19, 2023. It is now read-only.

8289093: BlockLocationPrinter fails to decode addresses with G1 #68

Closed
wants to merge 2 commits into from
Closed

8289093: BlockLocationPrinter fails to decode addresses with G1 #68

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e008106
Return the original value for addresses > top in G1CollectedHeap::blo…
tschatzl Jun 24, 2022
aa029e4
Fix return value to just return null
tschatzl Jun 27, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 6 additions & 0 deletions src/hotspot/share/gc/g1/g1CollectedHeap.cpp
View file
Open in desktop
Expand Up @@ -2348,6 +2348,12 @@ void G1CollectedHeap::par_iterate_regions_array(HeapRegionClosure* cl,

HeapWord* G1CollectedHeap::block_start(const void* addr) const {
HeapRegion* hr = heap_region_containing(addr);
// The CollectedHeap API requires us to not fail for any given address within
// the heap. HeapRegion::block_start() has been optimized to not accept addresses
// outside of the allocated area.
if (addr >= hr->top()) {
return nullptr;
}
return hr->block_start(addr);
}

Expand Down