339 Commits

Author SHA1 Message Date
Erik Helin
ae6fc5e37b 8142494: Add extension point to G1EvacuationRootClosures
Reviewed-by: jmasa, mgerdin
2015-11-25 21:54:05 +01:00
Thomas Schatzl
00eb98fb06 Merge 2015-11-25 19:27:38 +00:00
Erik Helin
2623434709 Merge 2015-11-25 17:32:44 +00:00
Thomas Schatzl
986ada18e1 Merge 2015-11-25 13:52:26 +00:00
Thomas Schatzl
352c2060f2 8136679: JFR event for adaptive IHOP
Reviewed-by: tbenson, mgerdin, sangheki, ehelin
2015-11-25 14:43:29 +01:00
Erik Helin
402195fa55 8142495: Move G1ParEvacuateFollowersClosure to header file
Reviewed-by: sjohanss, brutisso
2015-11-24 15:46:45 +01:00
Joseph Provino
f85cf9d7be 8139922: Get rid of dead code in ConcurrentMark
ConcurrentMark contains lots of unused code which has been removed.

Reviewed-by: jmasa, tschatzl
2015-11-24 15:13:02 -05:00
David Lindholm
f6343b0f69 8143587: G1 crashes with humongous object of size that almost fills a heap region
Reviewed-by: mgerdin, brutisso
2015-11-24 14:03:42 +01:00
Thomas Schatzl
48264e4686 8136678: Implement adaptive sizing algorithm for IHOP
Add adaptive sizing for IHOP that maximizes throughput (starts marking as late as possible). This functionality is enabled using a new -XX:+G1AdaptiveIHOP switch (default: false).

Reviewed-by: mgerdin, jmasa, pliden
2015-11-24 10:35:52 +01:00
Thomas Schatzl
66a728fc27 8136681: Factor out IHOP calculation from G1CollectorPolicy
Move out existing IHOP value calculation into an implementation of a new interface called G1IHOPControl. Prepare for changes to accomodate adaptive IHOP implementation.

Reviewed-by: jmasa, mgerdin, ehelin
2015-11-24 10:22:36 +01:00
Alexander Harlap
f3ff4f52cc 8141135: Remove G1RemSet::write_ref
Remove unused memembers of G1RemSet

Reviewed-by: kbarrett, tschatzl
2015-11-23 11:43:02 -05:00
David Lindholm
c3db3bae7b Merge 2015-11-20 19:17:57 +00:00
David Lindholm
2d7e74703d 8138681: Runtime.getFreeMemory() reports wrong value after humongous allocation
Reviewed-by: tschatzl, mgerdin
2015-11-20 17:32:02 +01:00
Erik Helin
aefeb2d442 8142390: Move ScanRSClosure to header file
Reviewed-by: jmasa, tschatzl
2015-10-19 16:21:35 +02:00
Erik Helin
57776e7af1 8142399: G1ParCopyClosure does not need do_oop_work
Reviewed-by: stefank, tschatzl
2015-09-11 15:18:43 +02:00
Derek White
d84d65893b 8143252: Clean up G1CollectedHeap interface
Delete unused methods and parameters

Reviewed-by: mgerdin, tschatzl, pliden
2015-11-19 12:43:08 -05:00
Coleen Phillimore
72756888e9 8141570: Fix Zero interpreter build for --disable-precompiled-headers
Change to include atomic.inline.hpp and allocation.inline.hpp only in .cpp files and some build fixes from Kim to build on ubuntu without devkits

Reviewed-by: kbarrett, sgehwolf, erikj
2015-11-18 11:47:55 -05:00
Kim Barrett
e8c5bc2024 8143014: Access PtrQueue member offsets through derived classes
Moved accessors to derived classes and updated callers.

Reviewed-by: tschatzl, jmasa, twisti
2015-11-17 16:40:52 -05:00
Stefan Johansson
56adbd02ec 8141280: G1ResManAllocator doesn't work with _survivor_is_full/_old_is_full
Reviewed-by: jmasa, kbarrett, tschatzl
2015-11-17 11:20:27 +01:00
Kim Barrett
7badb3edfb 8143013: Remove unused DirtyCardQueue::iterate_closure_all_threads
Remove unused function.

Reviewed-by: tbenson, tschatzl, mgerdin
2015-11-16 14:11:36 -05:00
Thomas Schatzl
7f53346519 Merge 2015-11-13 12:36:17 +00:00
Stefan Johansson
6983acfb5f 8139424: SIGSEGV, Problematic frame: # V [libjvm.so+0xd0c0cc] void InstanceKlass::oop_oop_iterate_oop_maps_specialized<true,oopDesc*,MarkAndPushClosure>
The crash was caused by a faulty eager humongous reclaim. The reason for reclaiming a live object was that the call to cleanupHRRS was done after dirtying cards and clearing the remembered sets for the humongous object. This could lead to one or many cards being missed.

Reviewed-by: tbenson, kbarrett, tschatzl
2015-11-13 09:28:53 +01:00
Thomas Schatzl
4d9dfbeb1d 8138684: G1 decision about taking regions into the collection set is too aggressive
Factor in expected waste and uncertainty of our guess in the decision whether to take another region into the collection set.

Reviewed-by: mgerdin, jmasa
2015-11-13 09:12:06 +01:00
Sangheon Kim
6c10034f4b 8134631: G1DummyRegionsPerGC fires assert of assert(words <= filler_array_max_size()) failed: too big for a single object
Change filler max value temporarily for G1DummyRegionsPerGC flag.

Reviewed-by: tbenson, tschatzl
2015-11-12 09:52:04 -08:00
Erik Helin
0580d65a1c 8142403: Make G1CollectorPolicy::predictor const
Reviewed-by: stefank, tschatzl
2015-11-12 14:03:14 +01:00
Erik Helin
3ee73137fb 8139883: Add virtual destructor G1ParScanThreadState
Reviewed-by: tschatzl, mgerdin, stefank
2015-10-19 15:47:36 +02:00
Joseph Provino
539395ffbf Merge 2015-11-11 23:47:41 +00:00
Thomas Schatzl
6ed8f3c0b2 8140689: Skip last young-only gc if nothing to do in the mixed gc phase
Reviewed-by: mgerdin, drwhite
2015-11-10 11:07:15 +01:00
Thomas Schatzl
22d9b3df0f 8138740: Start initial mark right after mixed GC if needed
Instead of issuing a single young-only GC after the end of all mixed GCs, if needed immediately start an initial mark in that young-only GC.

Reviewed-by: drwhite, mgerdin
2015-11-10 11:07:15 +01:00
Thomas Schatzl
90d03cf69c 8139874: After G1 Full GC, the next GC is always a young-only GC
Allow initiation of an initial mark after Full GC

Reviewed-by: ehelin
2015-11-10 11:07:15 +01:00
Thomas Schatzl
d8534a2c6d 8140597: Forcing an initial mark causes G1 to abort mixed collections
Postpone the initial mark request until the current mixed GC phase has finished.

Reviewed-by: mgerdin
2015-11-10 11:07:15 +01:00
Nils Eliasson
3307d0f163 Merge 2015-11-06 11:34:03 +01:00
Tobias Hartmann
7af927f9c1 8141132: JEP 254: Compact Strings
Adopt a more space-efficient internal representation for strings.

Co-authored-by: Brent Christian <brent.christian@oracle.com>
Co-authored-by: Vivek Deshpande <vivek.r.deshpande@intel.com>
Co-authored-by: Charlie Hunt <charlie.hunt@oracle.com>
Co-authored-by: Vladimir Kozlov <vladimir.kozlov@oracle.com>
Co-authored-by: Roger Riggs <roger.riggs@oracle.com>
Co-authored-by: Xueming Shen <xueming.shen@oracle.com>
Co-authored-by: Aleksey Shipilev <aleksey.shipilev@oracle.com>
Co-authored-by: Sandhya Viswanathan <sandhya.viswanathan@intel.com>
Reviewed-by: alanb, bdelsart, coleenp, iklam, jiangli, jrose, kevinw, naoto, pliden, roland, smarks, twisti
2015-11-03 09:41:03 +01:00
Erik Helin
0f08933773 8140489: Introduce shorthand for average_time_ms in G1CollectorPolicy
Reviewed-by: mgerdin, tschatzl, drwhite
2015-10-29 17:32:08 +01:00
Erik Helin
d0f3d01013 8140393: Move WorkerDataArray to its own file
Reviewed-by: tschatzl, mgerdin, tbenson
2015-10-29 14:58:44 +01:00
Erik Helin
a935c706b6 8140761: Remove caching from WorkerDataArray
Reviewed-by: tschatzl, mgerdin, tbenson
2015-10-29 11:33:43 +01:00
Coleen Phillimore
b046110ab7 Merge 2015-10-28 15:03:40 +01:00
Coleen Phillimore
fc03719078 8139203: Consistent naming for klass type predicates
8138923: Remove oop coupling with InstanceKlass subclasses

Renamed oop_is_instance and friends, removed the functions in oop that dug down into InstanceKlass.

Reviewed-by: jrose, lfoltan, stefank
2015-10-28 09:47:23 -04:00
Thomas Schatzl
5a0303c749 Merge 2015-11-10 17:57:01 +01:00
Thomas Schatzl
00a7b7ded2 8140585: PLAB statistics are flushed too late
Move the call to G1EvacStats::adjust_desired_plab_sz() to after flushing the per-thread statistics.

Reviewed-by: mgerdin, jmasa
2015-11-10 09:29:40 +01:00
David Lindholm
fd19805f5b 8141946: Remove develop flag G1TraceHeapRegionRememberedSet
Reviewed-by: mgerdin, tschatzl
2015-11-09 15:09:03 +01:00
Kim Barrett
fd7b3345c8 8077571: ObjPtrQueue is poorly named
Renamed to SATBMarkQueue and renamed files.

Reviewed-by: brutisso, tschatzl, dholmes, twisti
2015-11-06 16:30:40 -05:00
David Lindholm
4bfbb83879 8141434: G1CollectedHeap::into_cset_dirty_card_queue_set should be moved to G1RemSet
Reviewed-by: tschatzl, mgerdin
2015-11-09 11:26:15 +01:00
David Lindholm
6bda88594d 8139867: Change how startsHumongous and continuesHumongous regions work in G1
Reviewed-by: tschatzl, tbenson
2015-11-09 09:19:39 +01:00
Per Lidén
6f81963972 8042209: Remove unused function G1SATBCardTableLoggingModRefBS::write_ref_field_static
Reviewed-by: mgerdin, tschatzl
2015-11-06 09:46:54 +01:00
Bengt Rutisson
4a65cc4f5b 8141530: Remove G1RecordHRRSOops and G1RecordHRRSEvents
Reviewed-by: tschatzl, mgerdin
2015-11-06 06:13:00 +01:00
Bengt Rutisson
199618fac0 8141394: Remove the instrumentation added by JDK-6898948
Reviewed-by: tschatzl, pliden
2015-11-05 08:48:45 +01:00
Kim Barrett
63f370152d Merge 2015-11-04 20:02:54 +00:00
Thomas Schatzl
35f9a7ac32 Merge 2015-11-04 18:10:18 +00:00
Kim Barrett
b256989eb3 6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
Simplify indexing, address obsolete code, improve access/type checking.

Reviewed-by: tschatzl, pliden
2015-11-04 13:09:57 -05:00