> 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: reviewer feedback ------------- Changes: - all: https://git.openjdk.org/jdk/pull/26867/files - new: https://git.openjdk.org/jdk/pull/26867/files/abf9b1f8..5557ae9b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26867&range=32 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26867&range=31-32 Stats: 6 lines in 1 file changed: 0 ins; 2 del; 4 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
