RFR: Use PLABs for old gen allocations, including promotions.

Bernd Mathiske bmathiske at openjdk.java.net
Tue Apr 13 21:10:41 UTC 2021


On Mon, 12 Apr 2021 19:50:45 GMT, Bernd Mathiske <bmathiske at openjdk.org> wrote:

> Introducing a 3rd kind of LAB for generational mode: "PLAB".
> 
> (This PR is in draft form as testing is still ongoing.
> Our smoke testing till flags a few unexpected crashes in generational mode.)
> 
> Shenandoah already has TLABs and GCLABs. The latter pertain firmly to young gen and allocations in old gen are so far always shared allocations, without LAB. With this PR, old gen allocations go through LABs as well and we separate those from the other two kinds. In other words, each generation has its separate GC-decicated LAB. 
> 
> When not in generational mode, GCLABs work as before and PLABs are never used.
> 
> The new diagnostic flag -XX:-ShenandoahUsePLAB can be used to turn off PLAB use, so that all old gen allocations are shared as before this PR. By default PLABs will be used.
> 
> Note that In source code, the type of both GCLABs and PLABs is "PLAB*", sonce GCLABs already had that type, but variable references and other value-level identifiers are consistently named "gclab" and "plab".
> 
> In ShenandoahFreeSet::allocate_single() there is a notable additional change that can easily be reverted if needed. Instead of fitting the intended allocation into any free or occupied region that can hold it, we now search for an occupied region in the same generation first, before considering free regions. The idea is to pack regions more densely as long as they have space left. This provides more opportunity for region borrowing between generations, should the need arise. For single-generational Shenandoah, it should not make a huge difference, right? Or is it crucial to find any free slot hit in that search loop ASAP?
> 
> Full collections need to set empty/trash regions that they compact into to non-FREE. For now, they will be made "YOUNG". (See the second commit.)

Every time I edit the description text to remove the part about being draft and testing, the GitHub bot edits it back. So, I declare here that the bugs in question at the time of opening the PR as draft have been fixed and that it is now ready for review, not in draft.

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

PR: https://git.openjdk.java.net/shenandoah/pull/30


More information about the shenandoah-dev mailing list