XPost: linux.debian.kernel
Control: tag -1 patch
On Thu, 2025-06-26 at 19:17 +0200, Ben Hutchings wrote:
On Thu, 2025-06-26 at 18:24 +0200, Ben Hutchings wrote:
[...]
Thank you for bringing this to our attention.
I don't think it makes sense for us to re-enable cgroups v1, given that
it has been deprecated for so long and OpenJDK itself can use v2.
I will look into whether we can instead provide only a /proc/cgroups
file, which seems to be all that OpenJDK actually needed.
In fact this may be as simple as reverting:
commit af000ce85293b8e608f696f0c6c280bc3a75887f
Author: Michal Koutný <[email protected]>
Date: Mon Sep 9 18:32:23 2024 +0200
cgroup: Do not report unavailable v1 controllers in /proc/cgroups
but I have not yet tested that.
Yes, this seems to work.
I compiled the following class:
--- BEGIN ---
import java.lang.System;
class Main {
public static void main(String[] args) {
int count = Integer.parseInt(args[0]);
char[] one_mb;
int i;
for (i = 0; i != count; i++) {
one_mb = new char[0x100000];
System.out.println(one_mb[0] | one_mb[0xfffff]);
}
}
};
--- END ---
and ran it with an argument of "1000".
I used a podman container of Debian limited to 50 MiB. With the current
kernel from trixie, it OOMs. With that commit reverted, it runs to
completion.
I will open an MR tomorrow.
Ben.
--
Ben Hutchings
73.46% of all statistics are made up.
-----BEGIN PGP SIGNATURE-----
iQIzBAABCgAdFiEErCspvTSmr92z9o8157/I7JWGEQkFAmhtn7AACgkQ57/I7JWG EQmidQ/9EcNZc0aaWCi3xCMEPHn9TIOm8tNRfS/HNBDtBU/yDJFwVeo+aL+BiV61 AXWw4l8YNgx6Gca9W6XVD9qZUBdE2IzUBqVMFgggbpqSxXeEaSPxwxwJYUDeuo9y VNsBSBdVSwgkOswjQtQOebOr7LhZB1YtmZEfxGqQD3PHnmIG4yOZ/V5e657P5QL7 MwwETHPQFGm87pLE7sVi5Ddj3f1DyvwNzxdS4GzDX8szNP38o7srntK0sCufTXLi nHxNRB581XP95SaTND9Ttty3cMlTdQh/beIk