Integrated: 8287113: JFR: Periodic task thread uses period for method sampling events

[Erik Gahlin]

On Sat, 21 May 2022 14:22:52 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> Could I have a review of a change that excludes the native method sampling event type from the JFR Periodic Task thread. Sampling is done in native code, not in the periodic task thread. 
> 
> The method sampling event (jdk.ExecutionSample) was already excluded, but jdk.NativeMethodSample was missing. This lead to the periodic task thread waking up 50 times per second instead of once (when using the default configuration). Each time the thread waits an unnecessary JavaMonitorWait event was emitted.
> 
> Testing: jdk/jdk/jfr

This pull request has now been integrated.

Changeset: 8122466f
Author:    Erik Gahlin <egahlin at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/8122466fbb8e3f3450131895551ec5b832845938
Stats:     5 lines in 2 files changed: 4 ins; 0 del; 1 mod

8287113: JFR: Periodic task thread uses period for method sampling events

Reviewed-by: jbachorik

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

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