RFR: 8259774: samplethreads option does not work for jfr thread sampler

[Erik Gahlin]

On Thu, 14 Jan 2021 11:50:09 GMT, Yang Yi <github.com+5010047+kelthuzadx at openjdk.org> wrote:

> Hi,
> 
> Please help review this minor change that lets the JFR option
> `samplethreads` works.
> 
> In the current implementation, no matter whether `samplethreads`
> option is turned on or off, it will not affect thread sampling. 
> The sampler completely ignores it. This patch will address this 
> problem and let the sampler aware of this option.

The samplethreads option was added in 7u40 as an easy way to disable method profiling. 

This was when the feature was new and we had spurious crashes when walking thread stacks. It allowed us to disable the features in our test infrastructure. We later fixed the issue. If we get a crash, we have a crash handlar that will allow us to continue. We kept the flag for a few releases, as a workaround for customers if they would run into issues. AFAIK, this has never been the case?

The flag should really be removed. Could be it is still there because it has not yet gone through deprecation process.

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

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