Integrated: 8321939: [GenShen] ShenandoahOldEvacRatioPercent=100 fails with divide-by-zero

Y. Srinivas Ramakrishna ysr at openjdk.org
Thu Dec 28 01:15:11 UTC 2023


On Wed, 13 Dec 2023 00:47:44 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

> The code that used the option ShenandoahOldEvacRatioPercent (SOEP) didn't correctly deal with the extremal setting of 100. Corrected the error and did some light refactoring in the vicinity, getting rid of some dead variables, adding documentation, and assertions. Also modified slightly the documentation of the option.
> 
> **Testing:**
> - [x] GHA
> - [x] code pipeline testing (TestPeriodicGC fails without the unrelated fix in https://github.com/openjdk/shenandoah/pull/372)
> - [x] specjbb with SOEP=default,0,100 to verify that the issue is fixed

This pull request has now been integrated.

Changeset: 2c77c16e
Author:    Y. Srinivas Ramakrishna <ysr at openjdk.org>
URL:       https://git.openjdk.org/shenandoah/commit/2c77c16ef32d95f59a631e4a51836f4d6ccf4d39
Stats:     173 lines in 3 files changed: 62 ins; 68 del; 43 mod

8321939: [GenShen] ShenandoahOldEvacRatioPercent=100 fails with divide-by-zero

Reviewed-by: kdnilsen

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

PR: https://git.openjdk.org/shenandoah/pull/369


More information about the shenandoah-dev mailing list