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

Thomas Schatzl tschatzl at openjdk.java.net
Mon Mar 29 15:41:27 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.

Seem good.

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

Marked as reviewed by tschatzl (Reviewer).

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



More information about the hotspot-gc-dev mailing list