Merge pull request #13 from hyperlight-dev/fix-sparsify-message

fix: sparsify message reports resulting disk size instead of amount removed

Author: danbugs

examples/dotnet-nativeaot/HelloNativeAot.csproj

@@ -7,6 +7,6 @@
     <InvariantGlobalization>true</InvariantGlobalization>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="runtime.linux-musl-x64.Microsoft.DotNet.ILCompiler" Version="9.0.15" />
+    <PackageReference Include="runtime.linux-musl-x64.Microsoft.DotNet.ILCompiler" Version="9.0.16" />
   </ItemGroup>
 </Project>

host/src/lib.rs

@@ -1578,11 +1578,8 @@ fn sparsify_snapshot(path: &Path) -> Result<()> {
     drop(mmap);
 
     if punched > 0 {
-        eprintln!(
-            "  sparsified: punched {} zero pages ({} MiB saved on disk)",
-            punched,
-            punched * 4 / 1024
-        );
+        let disk_mib = (len - punched * PAGE as u64) / 1024 / 1024;
+        eprintln!("  sparsified: {disk_mib} MiB on disk (punched {punched} zero pages)",);
     }
 
     Ok(())
@@ -1669,11 +1666,8 @@ fn sparsify_snapshot(path: &Path) -> Result<()> {
     drop(mmap);
 
     if punched > 0 {
-        eprintln!(
-            "  sparsified: punched {} zero pages ({} MiB saved on disk)",
-            punched,
-            punched * 4 / 1024
-        );
+        let disk_mib = (len - punched * PAGE as u64) / 1024 / 1024;
+        eprintln!("  sparsified: {disk_mib} MiB on disk (punched {punched} zero pages)",);
     }
 
     Ok(())