[jdk17u-dev] Integrated: 8265919: RunThese30M fails "assert((!(((((JfrTraceIdBits::load(value)) & ((1 << 4) << 8)) != 0))))) failed: invariant"

[Rinku Kumar Maurya]

On Tue, 28 May 2024 15:18:58 GMT, Rinku Kumar Maurya <duke at openjdk.org> wrote:

> clean backport to remove assertion which asserts artifact is not serialized and assumes only `JfrRecorderThread` can perform serialization of artifacts marked during the window between rotate() and end_recording(), which is not true because a GC thread during class unloading process will serialize the class loaders associated with the unloaded classes. We should remove this assertion and let the code fall through and explicitly clear the serialize and transient meta bits.

This pull request has now been integrated.

Changeset: 787cf27a
Author:    Rinku Kumar Maurya <rkmaurya at amazon.com>
Committer: Martin Doerr <mdoerr at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/787cf27a8bb19c58bfa5bd17bf433ddaa99117d7
Stats:     2 lines in 1 file changed: 0 ins; 2 del; 0 mod

8265919: RunThese30M fails "assert((!(((((JfrTraceIdBits::load(value)) & ((1 << 4) << 8)) != 0))))) failed: invariant"

Backport-of: ffa34ed429079827c9b0acb843288fde4b7fa53d

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

PR: https://git.openjdk.org/jdk17u-dev/pull/2504