RFR: 8253694: Remove Thread::muxAcquire() from ThreadCrashProtection() [v2]
Patricio Chilano Mateo
pchilanomate at openjdk.java.net
Mon Oct 5 14:12:55 UTC 2020
> Hi all,
>
> Please review the following patch. Current ThreadCrashProtection() implementation uses static members which requires
> the use of Thread::muxAcquire() to allow only one user at a time. We can avoid this synchronization requirement if each
> thread has its own ThreadCrashProtection *data. I tested it builds on Linux, macOS and Windows. Since the
> JfrThreadSampler is the only one using this I run all the tests from test/jdk/jdk/jfr/. I also run some tests with JFR
> enabled while forcing a crash in OSThreadSampler::protected_task() and tests passed with several "Thread method sampler
> crashed" UL output. Also run tiers1-3. Thanks, Patricio
Patricio Chilano Mateo has updated the pull request incrementally with one additional commit since the last revision:
Remove _crash_mux
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/376/files
- new: https://git.openjdk.java.net/jdk/pull/376/files/c3168cc7..00539d6f
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=376&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=376&range=00-01
Stats: 101 lines in 14 files changed: 45 ins; 35 del; 21 mod
Patch: https://git.openjdk.java.net/jdk/pull/376.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/376/head:pull/376
PR: https://git.openjdk.java.net/jdk/pull/376
More information about the hotspot-jfr-dev
mailing list