comparison src/share/vm/gc_implementation/g1/g1CardCounts.cpp @ 20543:e7d0505c8a30

8059758: Footprint regressions with JDK-8038423 Summary: Changes in JDK-8038423 always initialize (zero out) virtual memory used for auxiliary data structures. This causes a footprint regression for G1 in startup benchmarks. This is because they do not touch that memory at all, so the operating system does not actually commit these pages. The fix is to, if the initialization value of the data structures matches the default value of just committed memory (=0), do not do anything. Reviewed-by: jwilhelm, brutisso
author tschatzl
date Fri, 10 Oct 2014 15:51:58 +0200
parents 1f1d373cd044
children 7848fc12602b
comparison
equal deleted inserted replaced
20542:0fcaab91d485 20543:e7d0505c8a30
31 #include "services/memTracker.hpp" 31 #include "services/memTracker.hpp"
32 #include "utilities/copy.hpp" 32 #include "utilities/copy.hpp"
33 33
34 PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC 34 PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC
35 35
36 void G1CardCountsMappingChangedListener::on_commit(uint start_idx, size_t num_regions) { 36 void G1CardCountsMappingChangedListener::on_commit(uint start_idx, size_t num_regions, bool zero_filled) {
37 if (zero_filled) {
38 return;
39 }
37 MemRegion mr(G1CollectedHeap::heap()->bottom_addr_for_region(start_idx), num_regions * HeapRegion::GrainWords); 40 MemRegion mr(G1CollectedHeap::heap()->bottom_addr_for_region(start_idx), num_regions * HeapRegion::GrainWords);
38 _counts->clear_range(mr); 41 _counts->clear_range(mr);
39 } 42 }
40 43
41 void G1CardCounts::clear_range(size_t from_card_num, size_t to_card_num) { 44 void G1CardCounts::clear_range(size_t from_card_num, size_t to_card_num) {