Integrated: Loan from old should align on region size
Kelvin Nilsen
kdnilsen at openjdk.org
Wed Feb 1 16:58:45 UTC 2023
On Wed, 1 Feb 2023 02:57:07 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:
> This fixes an error detected by a recently added assertion on a different branch. When loaning memory from old-gen to young-gen, it is important the total amount of the loan be an integral number of ShenandoahHeapRegions. The total loan represents a sum of memory loaned to support young evacuations and memory loaned to support mutator allocations made while GC is active. The sum of these two values is a multiple of region size. The previous version of the code accidentally loaned only the memory dedicated to supporting mutator allocations, which is not necessarily a multiple of the region size.
This pull request has now been integrated.
Changeset: 998ea70c
Author: Kelvin Nilsen <kdnilsen at openjdk.org>
URL: https://git.openjdk.org/shenandoah/commit/998ea70c079ac4b2498178aade9d84eb6b2853ea
Stats: 7 lines in 1 file changed: 6 ins; 0 del; 1 mod
Loan from old should align on region size
Reviewed-by: ysr, wkemper
-------------
PR: https://git.openjdk.org/shenandoah/pull/213
More information about the shenandoah-dev
mailing list