From 2bf967d5af8a12a8ac1bfb2ed872c45a332dce17 Mon Sep 17 00:00:00 2001 From: Oksana Salyk Date: Mon, 15 Apr 2024 15:27:32 +0200 Subject: [PATCH] utils: change lower_limit --- .github/workflows/scan_stack_usage.yml | 6 +++--- utils/call_stacks_analysis/README.md | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/scan_stack_usage.yml b/.github/workflows/scan_stack_usage.yml index e133b31ce44..f6e7dce6c1f 100644 --- 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 - 784B = 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 + # 784B = 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 index 4933254effd..40a21cfc15e 100644 --- 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: