RFR: 8137022: Concurrent refinement thread adjustment and (de-)activation suboptimal [v3]

[Kim Barrett]

On Tue, 27 Sep 2022 16:55:54 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

>> The current cards, predicted cards, and predicted time until GC are, I think, all interesting here.  The point is to give some indication of what the controller is doing and why.  (The activation/deactivation messages are probably less interesting now, but that's why I demoted them to trace level.)  For looking at what's going on with refinement `-Xlog:gc+refine*=debug` is now the thing to use.  I wouldn't want to demote the above to trace level, and only logging on changes to wanted wouldn't provide that additional detail that I found useful in making sure things were working properly.
>
> I agree that the other numbers are interesting, even if there is no change. Maybe the initial message could be altered depending on if we are going to update the number of threads. Or be more generic, something like:
> 
> [20,057s][debug][gc,refine      ] Concurrent refinement: wanted threads: 0, cards: 79767, predicted: 79767, time: 0,00ms

Yes, that seems better.  Change will be in my next push.

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

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