[jdk8u-dev] RFR: 8237479: 8230305 causes slowdebug build failure [v5]

Severin Gehwolf sgehwolf at openjdk.org
Wed Dec 7 10:54:34 UTC 2022


On Mon, 5 Dec 2022 14:21:40 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

>> Jonathan Dowland has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 24 commits:
>> 
>>  - 8237479: 8230305 causes slowdebug build failure
>>    
>>    Declare methods as pure virtual.
>>    
>>    Backport-of: 4ca06995855b5c974321d7b3622d661b8d27ba76
>>  - 8253714: [cgroups v2] Soft memory limit incorrectly using memory.high
>>    
>>    The early implementation of cgroups v2 support was done with
>>    crun 0.8 and it contained a bug which set memory.high over
>>    memory.low when --memory-reservation was being used as a CLI
>>    option.
>>    
>>    This bug has been fixed in later crun versions, starting with
>>    crun 0.11. Use memory.low in OpenJDK as well.
>>    
>>    Backport-of: ff6843ca4842498791061f924c545fa9469cc1dc
>>  - Address style nit
>>  - TestCgroupSubsystemController: rework use of Files.writeString
>>  - CgroupSubsystemController: fix library paths
>>    
>>    We need the testlibrary copy of FileUtils but the test.lib.util copy of
>>    Utils (method createTempDirectory is missing from the testlib copy)
>>  - TestCgroupSubsystemController: fix jtreg @library path
>>  - Replace Arrays.compare with Arrays.equals
>>    
>>    jdk8u does not have Arrays.compare()
>>  - incorporate (part of) 8275713: TestDockerMemoryMetrics test fails on recent runc
>>    
>>    The main hunk from 8275713 was rolled up in the changes for 8231111.
>>    This line is also necessary.
>>  - update mapfile for new JNI method name
>>  - tests for the backport
>>    
>>    two files had fairly significant merge conflicts, eyeball only to resolve
>>    
>>    haven't run any of them yet -- will depend on the rest of the patch
>>  - ... and 14 more: https://git.openjdk.org/jdk8u-dev/compare/f04ad96c...463f7511
>
> I can't see what changes are actually being made here. It should just be a couple of changes to one file, not 41. Do we need to rebase or something?
> 
> It would also be good to do that so the Windows tests run.

@gnu-andrew Please have another look for 8u approval. Thank you!

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

PR: https://git.openjdk.org/jdk8u-dev/pull/128


More information about the jdk8u-dev mailing list