RFR: 8352994: ZGC: Fix regression introduced in JDK-8350572 [v3]

[Stefan Karlsson]

On Tue, 8 Apr 2025 15:22:49 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> We have seen a bunch of timeouts that all points towards the introduction of a check against VMError::is_error_reported_in_current_thread() in the ZGC verification code. I propose this workaround to first check if there's really an error reporting event that is going on by checking VMError::is_error_reported().
>> 
>> The underlying performance issue (or hang(?)) when calling os::current_thread_id() is being investigated as a separate bug. This fix just tries to clean up issues we see when running ZGC testing.
>> 
>> Thanks to @plummercj for digging into this and proposing the same workaround.
>> 
>> Testing: GHA is clean, I'll run this through a few tiers of our CI pipeline
>
> Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8352994_is_error_reported
>  - Remove test from ProblemList
>  - 8352994: ZGC: Fix regression introduced in JDK-8350572

Thanks for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/24349#issuecomment-2788390540