RFR: Clean up CondCardMark code: remove excess barriers, enable it by default [v2]
Aleksey Shipilev
shade at openjdk.java.net
Wed Mar 31 14:24:02 UTC 2021
> The `StoreLoad`, `StoreStore` barriers in that code are only needed for CMS, and does not relate to Shenandoah. Also, `UseCondCardMark` is a recommended option for most machines today, and therefore we better just enable it ergonomically.
>
> Additional testing:
> - [x] hotspot_gc_shenandoah, x86_64, {fastdebug,release}
Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
More cleanups of scanned_concurrently
-------------
Changes:
- all: https://git.openjdk.java.net/shenandoah/pull/26/files
- new: https://git.openjdk.java.net/shenandoah/pull/26/files/6714df04..2668e01f
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=shenandoah&pr=26&range=01
- incr: https://webrevs.openjdk.java.net/?repo=shenandoah&pr=26&range=00-01
Stats: 15 lines in 3 files changed: 0 ins; 14 del; 1 mod
Patch: https://git.openjdk.java.net/shenandoah/pull/26.diff
Fetch: git fetch https://git.openjdk.java.net/shenandoah pull/26/head:pull/26
PR: https://git.openjdk.java.net/shenandoah/pull/26
More information about the shenandoah-dev
mailing list