Merge branch 'bugfix/fix_esp32s3_diram_calculation' into 'master'

Tools: Fix diram size calculation Closes IDFGH-8504 See merge request espressif/esp-idf!21243
espressif · Nov 29, 2022 · 2eabf60 · 2eabf60
2 parents 0786539 + 0810dd2
commit 2eabf60
Show file tree

Hide file tree

Showing 4 changed files with 34,134 additions and 14 deletions.
diff --git a/tools/idf_size.py b/tools/idf_size.py
@@ -584,13 +584,17 @@ def in_iram(x: MemRegions.Region) -> bool:
         r = StructureForSummary()
 
         diram_filter = filter(in_diram, segments)
-        r.diram_total = int(get_size(diram_filter) / 2)
+        r.diram_total = get_size(diram_filter)
 
         dram_filter = filter(in_dram, segments)
         r.dram_total = get_size(dram_filter)
         iram_filter = filter(in_iram, segments)
         r.iram_total = get_size(iram_filter)
 
+        # This fixes counting the diram twice if the cache fills the iram entirely
+        if r.iram_total == 0:
+            r.diram_total //= 2
+
         def filter_in_section(sections: Iterable[MemRegions.Region], section_to_check: str) -> List[MemRegions.Region]:
             return list(filter(lambda x: LinkingSections.in_section(x.section, section_to_check), sections))  # type: ignore