RFR: 8318190: GenShen: Be less aggressive with triggers for old-gen has expanded [v3]

[Kelvin Nilsen]

On Wed, 18 Oct 2023 20:52:50 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> Thanks for clarifying.  I "guess" the main tradeoff is how often do we do the work of evaluating the trigger condition.  As currently implemented, we evaluate this condition at the end of each GC cycle, which is relatively rare.  If we move it to should_start_gc(), it could be revaluated every ms or so.  On the other hand, after we integrate your change, the question of should_start_gc(OLD) will only be asked at the start of a new young gc, which is also relatively rare.
>> 
>> I'll move it now, in anticipation of your improvement, if you concur.
>
> Or should we integrate as is, and let you move the test when as part of your improvement?

I'm going to leave code as is for now.  It makes sense to move the trigger evaluation as part of the fix that you (Wiliam Kemper) have under development.  Thanks.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/341#discussion_r1364692588