Skip to content

Commit

Permalink
Merge pull request #6138 from pmem/stack_usage
Browse files Browse the repository at this point in the history
common: change lower_limit (#6138)

Co-Authored-By: Oksana Salyk <[email protected]>
  • Loading branch information
osalyk and Oksana Salyk authored Feb 4, 2025
2 parents b60b28e + c1f1f55 commit 5264967
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/scan_stack_usage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,13 +54,13 @@ jobs:
./make_extra.py
./make_cflow.sh
# The lower limit comes up from the DAOS memory requirements.
# 16kB - 4kB - 752B = 11536B
# 16kB - 4kB - 720B = 11568B
# 16kB = Stack allocated for a single Argobot's ULT
# 4kB = a maximum DAOS' stack usage up to calling a PMDK API calls
# 752B = safety margin
# 720B = safety margin
# ~ = Some OSes, e.g. Ubuntu 22.04, generate call stacks of size
# a little bit over the exact limit which is not deemed a problem at the moment.
./make_call_stacks.py --filter-api-file examples/api_filter.txt --filter-lower-limit 11536
./make_call_stacks.py --filter-api-file examples/api_filter.txt --filter-lower-limit 11568
- name: List per-function stack usage for all call stacks of interest
working-directory: ${{ env.CALL_STACKS_TOOLS_PATH }}
Expand Down
2 changes: 1 addition & 1 deletion utils/call_stacks_analysis/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
./make_extra.py && \
./make_cflow.sh && \
./make_call_stacks.py --filter-api-file examples/api_filter.txt \
--filter-lower-limit 11536 --dump-all-stacks
--filter-lower-limit 11568 --dump-all-stacks
```

If succesfull, it produces:
Expand Down

0 comments on commit 5264967

Please sign in to comment.