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

Introduce a tool to display memory usage. #188

Merged

jyao1 merged 1 commit into intel:main from xiaoyuxlu:memory_usage

Dec 26, 2023

Contributor

xiaoyuxlu commented Dec 25, 2023 •

edited

Loading

Fix #34 #155 #151

minor fix
add max heap usage info
add a script to display memory usage.

xiaoyuxlu force-pushed the memory_usage branch from 5fc3932 to 2fb434b Compare

December 25, 2023 06:06

Contributor Author

xiaoyuxlu commented Dec 25, 2023

results on 60f7c4a

|                         |      Requester        |      Responder        |
|                         | stack     |  heap     | stack     |  heap     |
| ----------------------- |-----------|-----------|-----------|-----------|
| async + release         |     412250|     321231|     337994|     254263|
| tokio + release         |     412682|    1134431|     338346|    1067463|
|async + releas + w/o hash|     525530|     369919|     446234|     303143|
| sync + release          |     480802|      68655|     151722|      72647|
|sync + release + w/o hash|     611130|      68463|     247354|      72647|
| async + debug           |     412314|     321231|     333930|     254263|
| sync + debug            |     480834|      68655|     147610|      72647|

xiaoyuxlu mentioned this pull request

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

Closed

jyao1 reviewed

View reviewed changes

sh_script/collect_memory_usage.py Outdated Show resolved Hide resolved

jyao1 reviewed

View reviewed changes

sh_script/collect_memory_usage.py Outdated Show resolved Hide resolved

jyao1 reviewed

View reviewed changes

sh_script/collect_memory_usage.py Outdated Show resolved Hide resolved

jyao1 reviewed

View reviewed changes

sh_script/collect_memory_usage.py Outdated Show resolved Hide resolved

jyao1 reviewed

View reviewed changes

sh_script/collect_memory_usage.py Outdated Show resolved Hide resolved

jyao1 reviewed

View reviewed changes

test/spdm-requester-emu/src/main.rs Outdated Show resolved Hide resolved

xiaoyuxlu force-pushed the memory_usage branch from 2fb434b to 7572755 Compare

December 26, 2023 05:28


          Introduce a tool to display memory usage.

13b9baa

1. minor fix
2. add max heap usage info
3. add a script to display memory usage.

Signed-off-by: xiaoyuxlu <[email protected]>

xiaoyuxlu force-pushed the memory_usage branch from 7572755 to 13b9baa Compare

December 26, 2023 05:59

jyao1 approved these changes

View reviewed changes

jyao1 merged commit 3d3d79e into intel:main

47 checks passed

xiaoyuxlu mentioned this pull request

need use release build for stack/heap size collection. #155

Closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet