Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

optimize stack usage on vendor request. #190

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

optimize stack usage on vendor request. #190

wants to merge 1 commit into from

Conversation

longlongyang
Copy link
Contributor

fix #186

@longlongyang
optimize stack usage on vendor request.
a60170a 
fix intel#186

Signed-off-by: Yang, Longlong <longlong.yang@intel.com>
@longlongyang
Copy link
Contributor Author

cmd:
cargo run -p spdm-requester-emu --no-default-features --features "spdm-ring,hashed-transcript-data,async-executor,test_stack_size"

Before PR
max stack usage(no idekm): 312,666
max stack usage: 411,866

After PR
max stack usage(no idekm): 312,666
max stack usage: 379,898

Reduced 31,968 bytes.

@jyao1
Copy link
Contributor

jyao1 commented Dec 27, 2023

please follow the configuration in original issue report.

@jyao1
Copy link
Contributor

jyao1 commented Jan 1, 2024

Do we still need this, since #191 is merged ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Optimize IDEKM stack usage
2 participants
@longlongyang @jyao1