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.

Sign up for GitHub

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

Jump to bottom

Update memory configurations and simplify specification #494

Merged
merged 3 commits into from
Jun 17, 2024
Merged

Update memory configurations and simplify specification #494

merged 3 commits into from
Jun 17, 2024

Commits on Jun 14, 2024

  1. Simplify specification and use of the various memory configurations 

    Several changes to make memory allocation cleaner:

1) enable build-time configuration of the memory configuration for the
eservice enclave (the pservice enclave does not have dynamic memory
requirements so they remain fixed).

2) a single variable now controls the allocation for enclave and
interpreter memory; PDO_MEMORY_CONFIG may be set to SMALL, MEDIUM, or
LARGE. this replaces the old WASM_MEM_CONFIG which was used for
interpreter memory configuration. The SMALL configuration sets up the
enclave and the interpreter for 4MB, MEDIUM is for 8MB and LARGE is
for 16MB.

3) remove the stack and heap configuration from the contract build
scripts; this appears to be unnecessary though further testing is
warranted.

Signed-off-by: Mic Bowman <[email protected]>
    @cmickeyb
    cmickeyb committed Jun 14, 2024
    Configuration menu
    Copy the full SHA
    944f675 View commit details
    Browse the repository at this point in the history

  2. Update documentation for the change to PDO_MEMORY_CONFIG environment/… 

    …build variable

Signed-off-by: Mic Bowman <[email protected]>
    @cmickeyb
    cmickeyb committed Jun 14, 2024
    Configuration menu
    Copy the full SHA
    b96fd99 View commit details
    Browse the repository at this point in the history

  3. replace WASM_MEM_CONFIG with PDO_MEMORY_CONFIG in docker tools 

    Signed-off-by: Mic Bowman <[email protected]>
    @cmickeyb
    cmickeyb committed Jun 14, 2024
    Configuration menu
    Copy the full SHA
    b70702a View commit details
    Browse the repository at this point in the history