RFR: 8321931: memory_swap_current_in_bytes reports 0 as "unlimited" [v2]
Gerard Ziemski
gziemski at openjdk.org
Thu Jan 18 16:21:39 UTC 2024
> We make a distinction between 0 for a limit vs non-limit cgroup values.
>
> For all values (limit or non-limit), 0 simply means simply 0.
>
> Output before from `jcmd PID VM.info`:
>
>
> container (cgroup) information:
> container_type: cgroupv2
> cpu_cpuset_cpus: not supported
> cpu_memory_nodes: not supported
> active_processor_count: 8
> cpu_quota: not supported
> cpu_period: not supported
> cpu_shares: not supported
> memory_limit_in_bytes: unlimited
> memory_and_swap_limit_in_bytes: unlimited
> memory_soft_limit_in_bytes: unlimited
> memory_usage_in_bytes: 11129120 k
> memory_max_usage_in_bytes: not supported
> memory_swap_current_in_bytes: unlimited
> memory_swap_max_limit_in_bytes: unlimited
> maximum number of tasks: 18963
> current number of tasks: 33
>
>
> Output now from `jcmd PID VM.info`:
>
>
> container (cgroup) information:
> container_type: cgroupv2
> cpu_cpuset_cpus: not supported
> cpu_memory_nodes: not supported
> active_processor_count: 8
> cpu_quota: not supported
> cpu_period: not supported
> cpu_shares: not supported
> memory_limit_in_bytes: unlimited
> memory_and_swap_limit_in_bytes: unlimited
> memory_soft_limit_in_bytes: unlimited
> memory_usage_in_bytes: 4962584 k
> memory_max_usage_in_bytes: not supported
> memory_swap_current_in_bytes: 0
> memory_swap_max_limit_in_bytes: unlimited
> maximum number of tasks: 18963
> current number of tasks: 33
>
> In this example `memory_swap_current_in_bytes` should be printed as equal to 0, not "unlimited".
Gerard Ziemski has updated the pull request incrementally with one additional commit since the last revision:
0 means 0 and has no special meaning either for limit, or non-limit type values
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/17314/files
- new: https://git.openjdk.org/jdk/pull/17314/files/92133dfb..81d43629
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=17314&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=17314&range=00-01
Stats: 7 lines in 4 files changed: 0 ins; 0 del; 7 mod
Patch: https://git.openjdk.org/jdk/pull/17314.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/17314/head:pull/17314
PR: https://git.openjdk.org/jdk/pull/17314
More information about the hotspot-runtime-dev
mailing list