'Process::memory' returns bytes now.
authorGreg Burri <greg.burri@gmail.com>
Thu, 22 Sep 2022 09:42:07 +0000 (11:42 +0200)
committerGreg Burri <greg.burri@gmail.com>
Thu, 22 Sep 2022 09:42:07 +0000 (11:42 +0200)
backend/src/minecraft_controller.rs

index 451b016..690f3d9 100644 (file)
@@ -4,7 +4,7 @@ use chrono::{ DateTime, offset::Local };
 \r
 #[derive(Clone, Debug)]\r
 pub struct MinecraftExe {\r
-    memory: u64, // [kB].\r
+    memory: u64, // [B].\r
     load_average_5min: f64, // [%].\r
     uptime: u64, // [s].\r
     world_size: u64, // [B].\r
@@ -15,7 +15,7 @@ pub struct MinecraftExe {
 \r
 impl MinecraftExe {\r
     pub fn format_memory(&self) -> String {\r
-        format_byte_size(self.memory * 1024, 2)\r
+        format_byte_size(self.memory, 2)\r
     }\r
 \r
     pub fn format_load_average(&self) -> String {\r
@@ -145,7 +145,8 @@ pub fn get_minecraft_executable_information(world_path: &str, backup_path: &str,
 \r
         Some(\r
             MinecraftExe {\r
-                memory: process.memory() / 3, // Because the Java garbage collector ZGC reports three times more the real memory usage: https://stackoverflow.com/a/62934057/212675\r
+                // 'memory' was divided by 3 but it not necessary any more (the Java garbage collector ZGC reports three times more the real memory usage: https://stackoverflow.com/a/62934057/212675).\r
+                memory: process.memory(),\r
                 load_average_5min: system.load_average().five / system.cpus().len() as f64 * 100.,\r
                 uptime: std::time::SystemTime::now().duration_since(std::time::UNIX_EPOCH).unwrap().as_secs() - process.start_time(),\r
                 world_size,\r