Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[browser][AOT] wasm-ld : error : initial memory too small #96840

Closed
pavelsavara opened this issue Jan 11, 2024 · 3 comments
Closed

[browser][AOT] wasm-ld : error : initial memory too small #96840

pavelsavara opened this issue Jan 11, 2024 · 3 comments
Labels
arch-wasm WebAssembly architecture area-Build-mono blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab os-browser Browser variant of arch-wasm
Milestone

Comments

@pavelsavara
Copy link
Member

pavelsavara commented Jan 11, 2024

We are doing some estimate how much memory we need, based on binary size. I guess this estimate is no good in this case.

Error Blob

{
  "ErrorMessage": "wasm-ld : error : initial memory too small",
  "BuildRetry": false,
  "ErrorPattern": "",
  "ExcludeConsoleLog": false
}

Reproduction Steps

Log

  Linking for initial memory $(WasmInitialHeapSize)=18874368 bytes. Set this msbuild property to change the value.
  Linking with emcc with -O2 -Wl,-O0 -Wl,--lto-O0. This may take a while ...
   "C:/helix/work/correlation/build/emsdk/node/14.18.2_64bit/bin/node.exe" C:\helix\work\correlation\build\emsdk\upstream\emscripten\src\compiler.js C:\Users\ContainerAdministrator\AppData\Local\Temp\tmppkdhk_2y.json --symbols-only
   "C:/helix/work/correlation/build/emsdk/upstream/bin\wasm-ld.exe" @C:\Users\ContainerAdministrator\AppData\Local\Temp\emscripten_w6_esgr0.rsp.utf-8
wasm-ld : error : initial memory too small, 19468624 bytes needed [C:\helix\work\workitem\e\publish\ProxyProjectForAOTOnHelix.proj]

Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=523499

cc @radical

Known issue validation

Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=523499
Error message validated: wasm-ld : error : initial memory too small
Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 1/11/2024 2:03:08 PM UTC

Report

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 0
@pavelsavara pavelsavara added arch-wasm WebAssembly architecture blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' area-Build-mono Known Build Error Use this to report build issues in the .NET Helix tab os-browser Browser variant of arch-wasm labels Jan 11, 2024
@pavelsavara pavelsavara added this to the 9.0.0 milestone Jan 11, 2024
@ghost
Copy link

ghost commented Jan 11, 2024

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Error Blob

{
  "ErrorMessage": "wasm-ld : error : initial memory too small",
  "BuildRetry": false,
  "ErrorPattern": "",
  "ExcludeConsoleLog": false
}

Reproduction Steps

Log

  Linking for initial memory $(WasmInitialHeapSize)=18874368 bytes. Set this msbuild property to change the value.
  Linking with emcc with -O2 -Wl,-O0 -Wl,--lto-O0. This may take a while ...
   "C:/helix/work/correlation/build/emsdk/node/14.18.2_64bit/bin/node.exe" C:\helix\work\correlation\build\emsdk\upstream\emscripten\src\compiler.js C:\Users\ContainerAdministrator\AppData\Local\Temp\tmppkdhk_2y.json --symbols-only
   "C:/helix/work/correlation/build/emsdk/upstream/bin\wasm-ld.exe" @C:\Users\ContainerAdministrator\AppData\Local\Temp\emscripten_w6_esgr0.rsp.utf-8
wasm-ld : error : initial memory too small, 19468624 bytes needed [C:\helix\work\workitem\e\publish\ProxyProjectForAOTOnHelix.proj]

Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=523499

We are doing some estimate how much memory we need, based on binary size. I guess this estimate is no good in this case.

cc @radical

Author: pavelsavara
Assignees: -
Labels:

arch-wasm, blocking-clean-ci, area-Build-mono, Known Build Error, os-browser

Milestone: 9.0.0

@radical
Copy link
Member

radical commented Jan 11, 2024

the computed size from the binlog: WasmInitialHeapSize=18939904. And the error has 19468624 bytes needed. difference of ~500kb.

@radical
Copy link
Member

radical commented Jan 11, 2024

cc @radekdoulik

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-Build-mono blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab os-browser Browser variant of arch-wasm
Projects
None yet
Development

No branches or pull requests

2 participants