Closed Bug 1617821 Opened 5 years ago Closed 4 years ago

Spec compliance: Increase JS embedding memory limits

Categories

(Core :: JavaScript: WebAssembly, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: lth, Assigned: lth)

References

Details

By agreement, the memory max limit is raised to 4GB and the memory initial limit is raised to 2GB. We can accomodate at most 2GB anyway (see bug 1392234), but we should allow the higher limits. This means some failures will move from validation to module instantiation or memory grow.

There's a potential for some arithmetic overflow errors here, so we need to audit the code carefully after changing the limits, and we need tests that are appropriate for both 32-bit and 64-bit systems, which may behave differently.

See Also: → 1615434
See Also: → 1673619

This is a relative triviality that I should just fix, and/or it will be fixed as part of fixing bug 1673619.

Assignee: nobody → lhansen
Status: NEW → ASSIGNED
Priority: P3 → P2

Increasing the field limit (as opposed to allowing more than 2GB memory) was implemented some time ago. For allowing more than 2GB memory, see bug 1673619.

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.