Integrated: 8263044: jdk/jfr/jvm/TestDumpOnCrash.java timed out

[Erik Gahlin]

On Mon, 10 Oct 2022 09:32:45 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> Hi,
> 
> When a crash/emergency dump is created, it is sometimes not complete.
> 
> This is OK as there is no guarantee that a dump will succeed when the JVM crashes. Problem occurs if this happens before the first metadata event has been written (metadataPosition == 0) in which case the parser will loop indefinitely.
> 
> This is expected behavior in the streaming case (EventStream), but if the recording was opened with RecordingFile it is usually able to finish or throw an exception. The fix is to invoke pollWait() if a valid recording only consists of the header (68 bytes). This will result in similar behaviour as if the recording was terminated prematurely, but the metadata event had been written. 
> 
> Testing: jdk/jdk/jfr
> 
> Thanks
> Erik

This pull request has now been integrated.

Changeset: 94ec729f
Author:    Erik Gahlin <egahlin at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/94ec729faae7b45fcfe0a94dd42a90e4d08fd1ec
Stats:     12 lines in 1 file changed: 10 ins; 0 del; 2 mod

8263044: jdk/jfr/jvm/TestDumpOnCrash.java timed out

Reviewed-by: mgronlun

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

PR: https://git.openjdk.org/jdk/pull/10629