RFR: 8212107: VMThread issues and cleanup [v5]
Robbin Ehn
rehn at openjdk.java.net
Wed Sep 23 15:48:21 UTC 2020
> We simplify the vmThread by removing the queue and refactor the the main loop.
> This solves the issues listed:
> - It can create an extra safepoint directly after a safepoint.
> - It's not safe for a non-JavaThread to add safepoint to queue while GC do oops do.
> - The exposure of the vm operation is dangerous if it's a handshake.
> - The code is a hornets nest with the repetition of checks and branches
>
> Passes t1-8, and a benchmark run.
>
> If you want a smaller diff the commits contains the incremental progress and each commit passed t1.
Robbin Ehn has updated the pull request incrementally with one additional commit since the last revision:
Whitespace fixes, thanks to Shipilev
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/228/files
- new: https://git.openjdk.java.net/jdk/pull/228/files/3c0395bb..85cf9001
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=228&range=04
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=228&range=03-04
Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.java.net/jdk/pull/228.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/228/head:pull/228
PR: https://git.openjdk.java.net/jdk/pull/228
More information about the hotspot-dev
mailing list