On Thu, 14 Apr 2022 17:08:32 GMT, Markus Grönlund <mgron...@openjdk.org> wrote:

> For additional context, I should add that the CrashProtection mechanism was 
> mainly put in place as a result of having to deliver JFR from JRockit into 
> Hotspot under a deadline, upholding feature-parity. The stack walking code 
> was in really bad shape back then. Over the years, it has been hardened and 
> improved much, and I have not seen any reported issues about JFR crashes in 
> many years (we log when crashing in production).
> 
> An important difference is that AGCT allows more thread states compared to 
> JFR, so there can be issues in that area that are not seen in JFR.

Thanks for the JFR insight, Marcus. I can see the CrashProtection being a 
stop-gap measure under time constraints. Maybe worth rethinking now. From your 
description, the way it is used in JFR differ significantly from how the 
async-profiler would use it.

Privately, I start wondering whether the benefits of async-profiler over JFR 
are worth the risk of corrupting the VM or the work needed to harden out AGCT. 
Their feature-set seems to overlap a lot. I may not see the whole picture 
though, I'm not a profiler expert.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8225

Reply via email to