Skip to content

extra: check-size: use ELF section flags instead of name matching#385

Open
gibix wants to merge 1 commit intoarduino:mainfrom
gibix:extra/check-size
Open

extra: check-size: use ELF section flags instead of name matching#385
gibix wants to merge 1 commit intoarduino:mainfrom
gibix:extra/check-size

Conversation

@gibix
Copy link
Member

@gibix gibix commented Mar 5, 2026

Select sections by their ELF flags (SHF_ALLOC, SHF_WRITE) rather than matching against a hardcoded list of section names. This is more robust as it automatically handles renamed or custom sections.

  • Static mode: SHF_ALLOC + SHF_WRITE (writable RAM sections)
  • Dynamic mode: SHF_ALLOC (all loaded sections go to LLEXT heap)
  • NO_RELOC: .llext.rodata.noreloc still excluded by name

@gibix gibix mentioned this pull request Mar 5, 2026
Select sections by their ELF flags (SHF_ALLOC, SHF_WRITE) rather than
matching against a hardcoded list of section names. This is more robust
as it automatically handles renamed or custom sections.

- Static mode: SHF_ALLOC + SHF_WRITE (writable RAM sections)
- Dynamic mode: SHF_ALLOC (all loaded sections go to LLEXT heap)
- NO_RELOC: .llext.rodata.noreloc still excluded by name

Signed-off-by: Gilberto Conti <g.conti@arduino.cc>
@gibix gibix force-pushed the extra/check-size branch from 41b724e to 603976f Compare March 5, 2026 16:26
@github-actions
Copy link

github-actions bot commented Mar 8, 2026

Built 0.53.2-0.dev+603976f

CI run PASSED 🟢

ArtifactBoardCoreTestsRAMSketchesWarningsErrors
zephyr_contrib ek_ra8d1 📗

11.9%

2--
frdm_mcxn947 3 🏷️

58.0%

2--
frdm_rw612 1 🏷️

83.0%

2--
✔️* zephyr_main giga 4 🏷️ ✅*

54.5%

4412-
nano33ble 1 🏷️ ✅*

78.7%

228-
nano_matter 📗 ✔️*

⚠️ 85.7%

208(2*)
niclasense 2 🏷️ ✅*

⚠️ 87.3%

208-
opta 4 🏷️ ✔️*

46.7%

5426(2*)
portentac33 3 🏷️ ✔️*

⚠️ 95.1%

5626(8*)
portentah7 3 🏷️ ✔️*

47.3%

5826(2*)
✅* zephyr_unoq unoq 📗 ✅*

26.3%

528-
Legend

BoardTestStatus description
🔥 🔥 Test run failed to complete.
🔴 Test completed with unexpected errors.
✔️* 🚫 Test completed with errors, but all are known/expected.
✅* 🟡 Test completed with some warnings; no errors detected.
🟢 Test passed successfully, with no warnings or errors.
🌑 🌑 Test was skipped.

@CLAassistant
Copy link

CLAassistant commented Mar 18, 2026

CLA assistant check
All committers have signed the CLA.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants