Merge pull request #6078 from pmem/stack_usage_limit

utils: change lower_limit
pmem · Apr 17, 2024 · 3a208d6 · 3a208d6
2 parents 2ef62c4 + a8b1cc6
commit 3a208d6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/scan_stack_usage.yml b/.github/workflows/scan_stack_usage.yml
@@ -52,13 +52,13 @@ jobs:
           ./make_extra.py
           ./make_cflow.sh
           # The lower limit comes up from the DAOS memory requirements.
-          # 16kB - 4kB - 768B = 11520B
+          # 16kB - 4kB - 752B = 11536B
           # 16kB = Stack allocated for a single Argobot's ULT
           #  4kB = a maximum DAOS' stack usage up to calling a PMDK API calls
-          #  768B = safety margin
+          #  752B = 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 11520
+          ./make_call_stacks.py --filter-api-file examples/api_filter.txt --filter-lower-limit 11536
 
       - name: List per-function stack usage for all call stacks of interest
         working-directory: ${{ env.CALL_STACKS_TOOLS_PATH }}

diff --git a/utils/call_stacks_analysis/README.md b/utils/call_stacks_analysis/README.md
@@ -15,7 +15,7 @@
 ./make_extra.py && \
 ./make_cflow.sh && \
 ./make_call_stacks.py --filter-api-file examples/api_filter.txt \
-    --filter-lower-limit 11520 --dump-all-stacks
+    --filter-lower-limit 11536 --dump-all-stacks
 ```
 
 If succesfull, it produces: