Added total size padding override. Documented unused parameter#102
Added total size padding override. Documented unused parameter#102ZhekaS wants to merge 2 commits into
Conversation
Yes.
I have two reservations about this option.
|
OK, I think I see the issue that I totally missed when working on this PR, that the TAB might actually contain many TBFs with different size requirements. I guess a more targeted flag is indeed required. EDIT: EDIT2: |
|
@bradjc I think the most "granular" control could be achieved by optionally passing the required padding size via a metadata in the ELF file itself, similarly to |
Instead of a --total-size-padding command-line option that applies uniformly to every ELF, read a per-binary `tbf_total_size_padding` symbol from the ELF's symbol table. This is looked up in the same pass as `tbf_protected_region_size` and, when present, overrides the architecture-default trailing padding.
Fixes #101
Fixes #100
- Added an option--total-size-padding <padding>to override the default set by the specific architecturetbf_total_size_paddinglookup that sets the override of the architecture specific padding--minimum-ram-sizethat it has no effect