Skip to content

Commit be0de41

Browse files
committed
8308006: Missing NMT memory tagging in CMS
Reviewed-by: phh
1 parent 2f6a287 commit be0de41

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/hotspot/share/gc/cms/concurrentMarkSweepGeneration.cpp

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@
8080
#include "runtime/timer.hpp"
8181
#include "runtime/vmThread.hpp"
8282
#include "services/memoryService.hpp"
83+
#include "services/memTracker.hpp"
8384
#include "services/runtimeService.hpp"
8485
#include "utilities/align.hpp"
8586
#include "utilities/stack.inline.hpp"
@@ -5655,6 +5656,9 @@ bool CMSBitMap::allocate(MemRegion mr) {
56555656
log_warning(gc)("CMS bit map backing store failure");
56565657
return false;
56575658
}
5659+
5660+
// Record NMT memory type
5661+
MemTracker::record_virtual_memory_type(brs.base(), mtGC);
56585662
assert(_virtual_space.committed_size() == brs.size(),
56595663
"didn't reserve backing store for all of CMS bit map?");
56605664
assert(_virtual_space.committed_size() << (_shifter + LogBitsPerByte) >=
@@ -5743,6 +5747,9 @@ bool CMSMarkStack::allocate(size_t size) {
57435747
log_warning(gc)("CMSMarkStack backing store failure");
57445748
return false;
57455749
}
5750+
5751+
// Record NMT memory type
5752+
MemTracker::record_virtual_memory_type(rs.base(), mtGC);
57465753
assert(_virtual_space.committed_size() == rs.size(),
57475754
"didn't reserve backing store for all of CMS stack?");
57485755
_base = (oop*)(_virtual_space.low());

0 commit comments

Comments
 (0)