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

2024-02-20 Thread Jan Kratochvil
On Wed, 31 Jan 2024 15:33:01 GMT, Jan Kratochvil wrote: >> It looks like this patch needs a bit more discussion. Generally, it would be >> good to have the functionality, but I'm unsure about the proposed >> implementation. >> >> Walking the directory hierarchy (**and** interface files) on

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

2024-02-06 Thread Severin Gehwolf
On Wed, 31 Jan 2024 15:33:01 GMT, Jan Kratochvil wrote: >> It looks like this patch needs a bit more discussion. Generally, it would be >> good to have the functionality, but I'm unsure about the proposed >> implementation. >> >> Walking the directory hierarchy (**and** interface files) on

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

2024-01-31 Thread Jan Kratochvil
On Fri, 12 Jan 2024 17:12:37 GMT, Severin Gehwolf wrote: > Walking the directory hierarchy (**and** interface files) on every call of > `physical_memory()` (or `active_processor_count()` for CPU) seems concerning. I have therefore implemented a hierarchical `memory.stat` extension for cgroup2

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

2024-01-30 Thread Jan Kratochvil
On Fri, 12 Jan 2024 16:26:58 GMT, Severin Gehwolf wrote: >> Jan Kratochvil has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix gtest testcases compilation errors > > test/hotspot/jtreg/containers/cgroup/NestedCgroup.java line 97: > >>

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

2024-01-12 Thread Severin Gehwolf
On Thu, 28 Dec 2023 15:19:23 GMT, Jan Kratochvil wrote: >> The testcase requires root permissions. > > Jan Kratochvil has updated the pull request incrementally with one additional > commit since the last revision: > > Fix gtest testcases compilation errors It looks like this patch needs a

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

2024-01-11 Thread Severin Gehwolf
On Thu, 28 Dec 2023 15:19:23 GMT, Jan Kratochvil wrote: >> The testcase requires root permissions. > > Jan Kratochvil has updated the pull request incrementally with one additional > commit since the last revision: > > Fix gtest testcases compilation errors I'm looking at it. Review should

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

2024-01-11 Thread Jan Kratochvil
On Thu, 28 Dec 2023 15:19:23 GMT, Jan Kratochvil wrote: >> The testcase requires root permissions. > > Jan Kratochvil has updated the pull request incrementally with one additional > commit since the last revision: > > Fix gtest testcases compilation errors ping, are there some concerns

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

2023-12-28 Thread Jan Kratochvil
> The testcase requires root permissions. Jan Kratochvil has updated the pull request incrementally with one additional commit since the last revision: Fix gtest testcases compilation errors - Changes: - all: https://git.openjdk.org/jdk/pull/17198/files - new: