> This PR eliminates redundant bookkeeping that had been carried out by both > ShenandoahGeneration and ShenandoahFreeSet. In the new code, we keep a > single tally of relevant information within ShenandoahFreeSet. > Queries serviced by ShenandoahGeneration are now delegated to > ShenandoahFreeSet. > > This change eliminates rare and troublesome assertion failures that were > often raised when the ShenandoahFreeSet tallies did not match the > ShenandoahGeneration tallies. These assertion failures resulted because the > two sets of books are updated at different times, using different > synchronization mechanisms. > > The other benefit of this change is that we have less synchronization > overhead because we only have to maintain a single set of books.
Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision: Fix up vmstructs and other infrastructure for jmap heap dump ------------- Changes: - all: https://git.openjdk.org/jdk/pull/26867/files - new: https://git.openjdk.org/jdk/pull/26867/files/891e96b8..ee617c30 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26867&range=26 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26867&range=25-26 Stats: 130 lines in 5 files changed: 63 ins; 59 del; 8 mod Patch: https://git.openjdk.org/jdk/pull/26867.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/26867/head:pull/26867 PR: https://git.openjdk.org/jdk/pull/26867
