[lilliput-jdk17u:lilliput] Integrated: 8309197: [Lilliput/JDK17] Fix ObjectSampleMarker without Lilliput

[Aleksey Shipilev]

On Wed, 31 May 2023 11:43:31 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> The change in ObjectSampleMarker, a part of JFR, is incorrect when Lilliput is off. It would reach to `set_narrow_klass` and `narrow_klass` unconditionally, but that is only safe to do when `UseCompactObjectHeader` is on.
> 
> This would be caught by assert in `set_narrow_klass` and `narrow_klass` with `-UCOH`. AFAICS, this has a chance to corrupt the header in release bits, as it would copy only the parts of the mark word, and we might be just lucky those are the same bits. 
> 
> Additional testing:
>  - [x] macos-aarch64-server-fastdebug, `jdk/jfr/event/oldobject` with `-XX:-UseCompactObjectHeaders` (passes now)
>  - [x] macos-aarch64-server-fastdebug, `jdk/jfr/event/oldobject` with `-XX:+UseCompactObjectHeaders` (still fails, with unrelated problems)

This pull request has now been integrated.

Changeset: b3f04097
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/lilliput-jdk17u/commit/b3f04097c215fbc84687f1327a5a6e5065ad2264
Stats:     9 lines in 1 file changed: 5 ins; 0 del; 4 mod

8309197: [Lilliput/JDK17] Fix ObjectSampleMarker without Lilliput

Reviewed-by: rkennke

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

PR: https://git.openjdk.org/lilliput-jdk17u/pull/30