On Wed, 11 Sep 2024 07:00:25 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> If these are just methods that might be seen in a stackdump then we still >> need `ThreadSleepEvent.isEnabled` and `ThreadSleepEvent.<init>`. > >> If these are just methods that might be seen in a stackdump then we still >> need `ThreadSleepEvent.isEnabled` and `ThreadSleepEvent.<init>`. > > I think minimal here is to just drop isTurnedOn from lists in these tests. > From what I can tell, the tests are sampling in a loop and can tolerate up to > a limit. Hopefully someone will step up some day and replace them. @AlanBateman @dholmes-ora Ok, that's fair enough. I have replaced the commit which dropped all three in the trace to just dropping the isTurnedOn frame. ------------- PR Comment: https://git.openjdk.org/jdk/pull/20923#issuecomment-2343431165
