[master] Integrated: 8368755: [Lilliput] Fix CompressedClassSpaceSize handling

[Roman Kennke]

On Fri, 26 Sep 2025 09:35:38 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> We currently set the default CompressedClassSpaceSize to 128M - much lower than the current upstream default of 1G, and still much lower than the possible encoding range with 4-byte-headers of 512M. I think I did this earlier to make some tests pass and to address a problem with CDS which also wants a smallish share of it, but I have in-fact only hidden a bug.
> 
> The way it should work is to set the CompressedClassSpaceSize to 512M and let CDS take its share and use the rest for class-space. This is infact implemented by [JDK-8332514](https://bugs.openjdk.org/browse/JDK-8332514) since a while. However, there is a bug there that prevents it from working with 4-byte-headers: it has 4G encoding range hard-coded.
> 
> This change fixes:
> - It reverts the default CCSS to 1G
> - It sets CCSS to 512M when running with COH
> - It makes the auto-sizing work by setting the correct encoding range
> - It fixes a test that tries to use CCSS*2. I changed it to use CCSS/2, I think this is still in the spirit of the test ('use a different CCSS and see if it still works')
> 
> Testing:
> - [x] tier1
> - [x] tier2
> - [x] gc/arguments/TestUseCompressedOopsErgoTools.java (affected by the change)
> - [x] runtime/cds/appcds/dynamicArchive/DynamicLotsOfClasses.java (failed before, fixed by this change)
> - [x] runtime/cds/appcds/LotsOfJRTClasses.java (failed before, fixed by this change)

This pull request has now been integrated.

Changeset: 56c3c3b3
Author:    Roman Kennke <rkennke at openjdk.org>
URL:       https://git.openjdk.org/lilliput/commit/56c3c3b35fcf376325aaf46218cc186b411afe6f
Stats:     24 lines in 6 files changed: 14 ins; 0 del; 10 mod

8368755: [Lilliput] Fix CompressedClassSpaceSize handling

Reviewed-by: stuefe

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

PR: https://git.openjdk.org/lilliput/pull/203