Integrated: 8264324: Simplify allocation list management in OopStorage::reduce_deferred_updates

Kim Barrett kbarrett at openjdk.java.net
Tue Mar 30 08:50:08 UTC 2021


On Sat, 27 Mar 2021 04:59:08 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this cleanup of the allocation list management performed by OopStorage::reduce_deferred_updates.  Part of its job is to take a block whose allocation set has changed and update its place in the allocation list accordingly.  The old code has somewhat confusing logic and does unnecessary work in some cases.  The new code is clearer and avoids that excess work.  (I'm not sure what I was thinking when I wrote the old code.)
> 
> Testing:
> mach5 tier1
> I've also been using this patch for some time under other in-development changes that make heavy use of OopStorage.

This pull request has now been integrated.

Changeset: 4ea6abfb
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/4ea6abfb
Stats:     30 lines in 2 files changed: 10 ins; 4 del; 16 mod

8264324: Simplify allocation list management in OopStorage::reduce_deferred_updates

Reviewed-by: tschatzl, ayang

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

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



More information about the hotspot-gc-dev mailing list