[jdk21u-dev] RFR: 8261242: [Linux] OSContainer::is_containerized() returns true when run outside a container [v4]
Dmitry Cherepanov
dcherepanov at openjdk.org
Mon Jun 23 11:41:30 UTC 2025
On Fri, 20 Jun 2025 12:06:22 GMT, Jan Kratochvil <jkratochvil at openjdk.org> wrote:
>> A patch 3 of 7 for: [[21u] Backport intention of 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected](https://mail.openjdk.org/pipermail/jdk-updates-dev/2025-April/043206.html)
>>
>> It has a patch dependency on PR 2 of 7: https://github.com/openjdk/jdk21u-dev/pull/1660
>>
>> This backport is not clean as noticed by @MBaesken - `test/hotspot/jtreg/ProblemList.txt` was removing `containers/cgroup/PlainRead.java` but that testcase is missing in JDK21 so there is nothing to remove.
>>
>> There was also a trivial conflict with backported JDK-8313083 in `src/hotspot/os/linux/cgroupSubsystem_linux.hpp`.
>>
>> The backport also contains an additional oneliner (by Yuri Nesterenko):
>>
>> make/data/hotspot-symbols/symbols-linux:
>> +JVM_IsContainerized
>>
>> as otherwise the build would fail with:
>>
>> src/java.base/linux/native/libjava/CgroupMetrics.c:42:(.text+0x11): undefined reference to `JVM_IsContainerized'
>>
>> This oneliner was not needed in JDK24 as there is [JDK-8017234: Hotspot should stop using mapfiles](https://bugs.openjdk.org/browse/JDK-8017234).
>>
>> The whole patchset has been tested on CentOS-7.9 (for cgroup v1) and on Fedora 40 (for cgroup v2) for test/hotspot/jtreg/containers.
>
> Jan Kratochvil has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>
> - Merge branch 'devmaster' into devmaster-cgroup2reduced123-conflict
> - Merge branch 'devmaster' into devmaster-cgroup2reduced123-conflict
> - 8261242: [Linux] OSContainer::is_containerized() returns true when run outside a container
>
> Reviewed-by: stuefe, iklam
> - src/java.base/linux/native/libjava/CgroupMetrics.c:42:(.text+0x11): undefined reference to `JVM_IsContainerized'
> - a patch by Yuri Nesterenko
> - 8333326: Linux Alpine build fails after 8302744
>
> Reviewed-by: sgehwolf, clanger, stuefe
> - 8302744: Refactor Hotspot container detection code
>
> Reviewed-by: jsjolen, stuefe
Marked as reviewed by dcherepanov (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk21u-dev/pull/1661#pullrequestreview-2949748053
More information about the jdk-updates-dev
mailing list