RFR: 8265450: Merge PreservedMarksSet::restore code paths
Thomas Schatzl
tschatzl at openjdk.java.net
Tue Apr 20 08:46:23 UTC 2021
Hi all,
can I have reviews for this refactoring that merges parallel and serial preserved marks restoration during evacuation failure into a single code path, simply by having the serial path call the parallel path in the VM thread.
This just reduces code and avoids the potential bugs when updating one path but not the other (which already happened).
The execution difference should not really matter for Serial GC where the serial path is executed:
- we are talking about the evacuation failure path
- there are likely a lot of preserved marks (random writes to the heap) that offset these two additional calls and some tiny superfluous member initialization.
Testing: tier1
Thanks,
Thomas
-------------
Commit messages:
- Changes
- Fix compilation
- Initial change
- Initial version
Changes: https://git.openjdk.java.net/jdk/pull/3584/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3584&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8265450
Stats: 49 lines in 1 file changed: 11 ins; 14 del; 24 mod
Patch: https://git.openjdk.java.net/jdk/pull/3584.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3584/head:pull/3584
PR: https://git.openjdk.java.net/jdk/pull/3584
More information about the hotspot-gc-dev
mailing list