RFR: 8264324: Simplify allocation list management in OopStorage::reduce_deferred_updates [v2]

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


> 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.

Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:

  Improve comment per ayang discussion

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3227/files
  - new: https://git.openjdk.java.net/jdk/pull/3227/files/fefc97e1..79739d63

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3227&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3227&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 1 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3227.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3227/head:pull/3227

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



More information about the hotspot-gc-dev mailing list