Re: RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v7]

2024-03-10 Thread Jan Kratochvil
On Wed, 21 Feb 2024 14:02:12 GMT, Severin Gehwolf wrote: > I'm surprised that the hierarchical memory limit look-up wouldn't see it for > v1. Either way, using the traversal approach you'd see that `foo/bar` has `2 > GB`, when traversing up the hierarchy you'd see `1 GB` for `foo` and use that

Re: RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v7]

2024-03-10 Thread Jan Kratochvil
On Wed, 21 Feb 2024 13:20:20 GMT, Severin Gehwolf wrote: > One goal of this patch would be to unify how this works for cgroup v1 and > cgroup v2. That is not much possible anyway as currently cgroup2 has to traverse the directories while after the kernel patch gets accepted it will be faster

Re: RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v7]

2024-03-10 Thread Jan Kratochvil
On Tue, 20 Feb 2024 15:15:09 GMT, Jan Kratochvil wrote: >> The testcase requires root permissions. >> >> Designed by Severin Gehwolf, implemented by Jan Kratochvil. > > Jan Kratochvil has updated the pull request with a new target base due to a > merge or a rebase. The pull request now

Re: RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v7]

2024-03-10 Thread Severin Gehwolf
On Wed, 21 Feb 2024 11:32:46 GMT, Jan Kratochvil wrote: > After internal discussion I have realized the patch has overgrown its > intended scope. And one should also consider how easy it would be for a > backport down to JDK-8. I am going to split it into: > > 1. cgroup1 bugfix to always

Re: RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v7]

2024-03-10 Thread Severin Gehwolf
On Wed, 21 Feb 2024 13:37:22 GMT, Jan Kratochvil wrote: > > One goal of this patch would be to unify how this works for cgroup v1 and > > cgroup v2. > > That is not much possible anyway as currently cgroup2 has to traverse the > directories while after the kernel patch gets accepted it will

Re: RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v7]

2024-02-20 Thread Jan Kratochvil
> The testcase requires root permissions. Jan Kratochvil has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 23 commits: - Disable cgroup.subtree_control testcase on cgroup1 - Extend the testcase - Implement cgroup.subtree_control .