Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

spdmlib use a lot of stack. It may cause stack overflow in small stack environment. #312

Closed
xiaoyuxlu opened this issue Nov 24, 2022 · 5 comments
Closed

spdmlib use a lot of stack. It may cause stack overflow in small stack environment. #312

xiaoyuxlu opened this issue Nov 24, 2022 · 5 comments
Assignees
@longlongyang
Labels
bug Something isn't working
Milestone
2023'Q2

Comments

@xiaoyuxlu
Copy link
Collaborator

No description provided.

@xiaoyuxlu xiaoyuxlu added the enhancement New feature or request label Nov 24, 2022
@jyao1
Copy link
Owner

jyao1 commented Nov 24, 2022

please describe what is "a lot of".

@xiaoyuxlu
Copy link
Collaborator Author

default config:
requester stack use: 808040 bytes
responder stack use: 818632 bytes

change config: max_session_count = 10
requester stack use: 1580888 bytes
responder stack use: 1591480 bytes

change config: max_session_count = 1
requester stack use: 421640 bytes
responder stack use 447560 bytes

every session will cost ~120k bytes

@jyao1
Copy link
Owner

jyao1 commented Nov 24, 2022

I think #311 can help reduce the stack usage.

@jyao1 jyao1 added this to the 2023'Q1 milestone Feb 27, 2023
@longlongyang longlongyang mentioned this issue Mar 25, 2023
Closed
@jyao1 jyao1 modified the milestones: 2023'Q1, 2023'Q2 Mar 29, 2023
@jyao1 jyao1 added bug Something isn't working and removed enhancement New feature or request labels May 12, 2023
@jyao1 jyao1 mentioned this issue May 12, 2023
Closed
@jyao1
Copy link
Owner

jyao1 commented May 17, 2023

#579 is related.

@jyao1
Copy link
Owner

jyao1 commented Aug 5, 2023

See intel/rust-spdm#34

@jyao1 jyao1 closed this as completed Aug 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@longlongyang longlongyang

Labels
bug Something isn't working
Projects
None yet
Milestone
2023'Q2
Development

Successfully merging a pull request may close this issue.

reduce stack size usage. longlongyang/jyao1-rust-spdm
3 participants
@jyao1 @longlongyang @xiaoyuxlu