From e9f0ba96ab18d985618de66b448602dd4ecd4a1c Mon Sep 17 00:00:00 2001
From: s50600822 <s50600822@gmail.com>
Date: Thu, 29 Feb 2024 13:45:58 +1100
Subject: [PATCH] demonstrate jdk cgroup bug - incl mem

---
 vs/java/cgroup/CgroupConfigPrinter.java |  11 +++++++++++
 vs/java/cgroup/application.jar          | Bin 1142 -> 1293 bytes
 vs/java/cgroup/runWithDocker.sh         |   2 +-
 vs/java/cgroup/runWithDockerJ8.sh       |   2 +-
 4 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/vs/java/cgroup/CgroupConfigPrinter.java b/vs/java/cgroup/CgroupConfigPrinter.java
index a7b0caf..dea57f0 100644
--- a/vs/java/cgroup/CgroupConfigPrinter.java
+++ b/vs/java/cgroup/CgroupConfigPrinter.java
@@ -1,3 +1,6 @@
+import java.lang.management.ManagementFactory;
+import com.sun.management.OperatingSystemMXBean;
+
 public class CgroupConfigPrinter {
     public static void main(String[] args) {
         int availableProcessors = Runtime.getRuntime().availableProcessors();
@@ -14,5 +17,13 @@ public static void main(String[] args) {
         System.out.println("Available Memory (Total): " + totalMemoryMB + " MB");
         System.out.println("Available Memory (Free): " + freeMemoryMB + " MB");
         System.out.println("Maximum Memory (Allocated): " + maxMemoryMB + " MB");
+
+        OperatingSystemMXBean osBean = (com.sun.management.OperatingSystemMXBean) ManagementFactory.getOperatingSystemMXBean();
+
+        // deprecated in favor of getTotalMemorySize(), however new method is not avail on JDK8
+        // keeps this so we can test against 8 and above
+        long totalSystemMemoryBytes = osBean.getTotalPhysicalMemorySize();
+        long totalSystemMemoryMB = totalSystemMemoryBytes / (1024 * 1024);
+        System.out.println("Total System Memory: " + totalSystemMemoryMB + " MB");
     }
 }
diff --git a/vs/java/cgroup/application.jar b/vs/java/cgroup/application.jar
index c29689d2c065f63dfb81096c9ab246902f739be1..35bdd87518a23f7a2a73d78a90a24852c1a0a644 100644
GIT binary patch
delta 908
zcmV;719SZL2#pF1P)h>@3IG5I001IwkqjvUB5jd3odP0lk=84Jg;ZNp(?AsdR!S2>
zpp;9ws__P?7wWxA5vA2Cl>$<{jZdLnYn&$Oq^TqR2LFjq>P+!Ajt@ThH}rQno=v1J
zQZ<>$?w;?Q@0@&lcEA1n@EO2)JXEj`T@penx)Ej=cxpT|5*5QPCssF~@}kEOo;PjN
zTVUwY<M$NE=#ij*sOW{t5N)}y*KN<VID=B=UUM;`$Cvl`qy$U9iU<Z6h6pt)#%6`D
zxlWPSYK~j;b1n5?5JM7%Rg56EtF*#a&Epn%Idv*<B*<Voi8a@>y@Ka*!%9gQV;F6V
z@dzxZ>t?0IT?M1)k}=6Jxww;FYv$K74i!4539tPs4&WevsbEyA+%7Sk(wF}vO~v<^
ze&<OEibI0pFvFC-^l!(bDvn{ALH3*gWEj@>hLEq0xP;>>PT(Xd*x~mXq^ej|Y_SI&
zD)x7BRz(7*8G1d(Gb(v*IqsIPeNLcfRh$#3^4R4(0EPL2ii^VBV;Ro_Yd6F*00QS2
zCR&OHu%>T+2>N&+=)6$licll>cQs^Or8=~HTE-&7_+G{=l|+OxGQ^ZOo|{(PYJ!WE
zic>T^Uh-4kKvu#{6}Pa&Fx|>%8Maa87Pq}bzB$brMb8&ujJkNW%3Z^wvVwlgKS*<f
z^qk$bCv7wx{izXeGw4OfO4RE1f8BeLLte#-P<VoWYVf67d$LtCi#t11Fkf&PYqYUG
zs0HD_|9(!BuueUZDZ5U+nsMyMX4&rx?lOcjPKgd>WZAU&O5NJz?uJm60Lw7x&tm$c
zPOU;MP;lyQk!MZeJl27GR)paY?$C82o_%2ALZNSgR{LpWG<F}~2765th7J0LePakc
zxQly#fh)U2>mjoKcn7J0zWFzZe#Gd*ZHy02G%z*UfY!i~24;Nx)S^F}NrwFKd@`&J
zBWWe&SjZohSlA!cHk!*s6O$WA4KC0uAZvrK30V0noEEUU8-9oDUyz&YAMbyUWrk!V
z7V)!2+u~M5Ty%F_ONq2NJ*0<3uSR;zK%tXH*^4Cluz-GK>8@Ns6c$~HFG-(Qh~X{9
z@d;D-ii7x0Q4IHe;UD}0P)i30ceHd`A_D*b-30&uP)h*<3IGZK00;m8B5jkZ133aB
iZIk5#Kn@~pU08RtbXp<<007+ulQRT81~UWz0000O0HwkJ

delta 756
zcmV<Q0t@|(3ib#LP)h>@3IG5I005F>kqjvUl4FrKodS|$k=84JeN@>}6G0R`O+scE
zT6V&&gBueRqKKv>h=yns3`hxOmF3frG^wFxNlj)f_y_+)U-ThZTIC1$HGYBR?HOb-
z!G7r5eb4Q4Z=YU%{{Hp@z#_^TI*?G1<mg0-p>N-OZ)R=Ntz}>B?u$ymkXp7}D_CJj
z7@0R3RCFoOIJ&`q8TwDdUx#jBIf6l}iQr^1Y-GwUF?q?-!_kXAhCxCt+uXIqmhV+W
zqv83DDA!sS1~8~#h+`P(bERz`HUi;LlowKg!*K@7%WnCW8*B%@FrB=DQHGH-YXSmm
ztD$9Ag|A@*2^A9z6NSI&^^^R1(W63_bQv|paT!-g1+!j%7jBhd&M5y&n$NVjj`QRt
z#k8cj$}nlH{Xg(J#|_+MPy;Up8HSA3B@}BQqhOZf7H*S*ZMM&#)aA@}<r%c8I6BFB
zjx6plbO&Bw+8e_0{KH85b%`!;+>@xb=L->o(tV%ffpm8{=0WW3glv5vaEW33RM8mL
zjU7p!i3MGMmTEkbYOG(ZhKk2jhrY3@qQEfL%2=Y3h)_k5m^RD<%L$zmuwdI>#SBE1
zA{0C$g^D%b3+qMC-Lq<uO2TJI7QHH6(cZG<ip|j375<LQL4adgq+8l}*G@PRWZMh<
ziYQq!c(e`oyj+H9Jf+7aTL+kYUeuRpHbs_^?VLS-0{ct@hUe6$A}8sJ7g#$E{e>iY
z$bR|?rHM<+U(o*@BX5r|HZb1A<U|vC6W5xUiSW5XWbWpYkzLHC^daP&oSIHXR!gTM
z%g@j}B$~9^M1Ej}MlsnL_=$wI3*l7>^YifOjuybFPk_D`=(N8X8fEE{<e*`NK9CaO
zo3uYJhkk+&bbmggpH{9%g1!6$P)i30zPDB$ivj=uqXYl|P)h*<3IGZK00;m8l4FxW
m1UUkdW0QOYKn{{)U0A-iRvwE2005%|lgb1=2Al%`0000Kzg+nM

diff --git a/vs/java/cgroup/runWithDocker.sh b/vs/java/cgroup/runWithDocker.sh
index 3122a61..7ee2879 100755
--- a/vs/java/cgroup/runWithDocker.sh
+++ b/vs/java/cgroup/runWithDocker.sh
@@ -1 +1 @@
-docker run --rm --cpus=2 --memory=256m --name jvm_cgroup_demo_openjdk21 -v .:/app openjdk:21  java -Xmx256m -Xms256m  -jar /app/application.jar
+docker run --rm --cpus=2 --memory=512m --name jvm_cgroup_demo_openjdk21 -v .:/app openjdk:21  java -Xmx256m -Xms256m  -jar /app/application.jar
diff --git a/vs/java/cgroup/runWithDockerJ8.sh b/vs/java/cgroup/runWithDockerJ8.sh
index e0ea056..8ec24d6 100755
--- a/vs/java/cgroup/runWithDockerJ8.sh
+++ b/vs/java/cgroup/runWithDockerJ8.sh
@@ -1,2 +1,2 @@
 ## show that cgroup for CPU lim isn't respected
-docker run --rm --cpus=2 --memory=256m --name jvm_cgroup_demo_openjdk8 -v .:/app openjdk:8  java -Xmx256m -Xms256m  -jar /app/application.jar
+docker run --rm --cpus=2 --memory=512m --name jvm_cgroup_demo_openjdk8 -v .:/app openjdk:8  java -Xmx256m -Xms256m  -jar /app/application.jar