Merge pull request #4389 from wasmerio/remove-mem-footprint

remove memory footprint computation
wasmerio · Jan 4, 2024 · 59577e7 · 59577e7
2 parents b914717 + 8bc38e6
commit 59577e7
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/lib/wasix/src/bin_factory/binary_package.rs b/lib/wasix/src/bin_factory/binary_package.rs
@@ -70,7 +70,6 @@ pub struct BinaryPackage {
     pub commands: Vec<BinaryPackageCommand>,
     pub uses: Vec<String>,
     pub version: Version,
-    pub module_memory_footprint: u64,
     pub file_system_memory_footprint: u64,
 }
 

diff --git a/lib/wasix/src/runtime/package_loader/load_package_tree.rs b/lib/wasix/src/runtime/package_loader/load_package_tree.rs
@@ -45,10 +45,6 @@ pub async fn load_package_tree(
         commands(&resolution.package.commands, &containers, resolution)?;
 
     let file_system_memory_footprint = count_file_system(&fs, Path::new("/"));
-    let atoms_in_use: HashSet<_> = commands.iter().map(|cmd| cmd.atom()).collect();
-    let module_memory_footprint = atoms_in_use
-        .iter()
-        .fold(0, |footprint, atom| footprint + atom.len() as u64);
 
     let loaded = BinaryPackage {
         package_name: root.package_name.clone(),
@@ -64,7 +60,6 @@ pub async fn load_package_tree(
         webc_fs: Arc::new(fs),
         commands,
         uses: Vec::new(),
-        module_memory_footprint,
         file_system_memory_footprint,
     };