[PATCH 1/1] rockchip: make_fit_atf: ignore empty PT_LOAD segment
Heinrich Schuchardt
xypron.glpk at gmx.de
Wed Sep 16 12:04:29 CEST 2020
On 16.09.20 10:40, Kever Yang wrote:
>
> On 2020/9/15 上午9:43, Heinrich Schuchardt wrote:
>> The linker sometimes creates PT_LOAD segments with length (p_filesz) zero
>> as described in https://man7.org/linux/man-pages/man5/elf.5.html. This
>> leads to build failures. We should ignore empty segments.
>>
>> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
>
>
> Reviewed-by: Kever Yang <kever.yang at rock-chips.com>
>
Hello Tom,
now that the patch has been reviewed I hope you can use it to get the
efi-2020-10-rc5 pull request merged.
Best regards
Heinrich
>
> Thanks,
>
> - Kever
>
>> ---
>> arch/arm/mach-rockchip/make_fit_atf.py | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-rockchip/make_fit_atf.py
>> b/arch/arm/mach-rockchip/make_fit_atf.py
>> index d15c32b303..f3224d2555 100755
>> --- a/arch/arm/mach-rockchip/make_fit_atf.py
>> +++ b/arch/arm/mach-rockchip/make_fit_atf.py
>> @@ -189,8 +189,9 @@ def unpack_elf(filename):
>> p_type, p_flags, p_offset = struct.unpack_from('<LLQ', elf,
>> offset)
>> if p_type == 1: # PT_LOAD
>> p_paddr, p_filesz = struct.unpack_from('<2Q', elf,
>> offset + 0x18)
>> - p_data = elf[p_offset:p_offset + p_filesz]
>> - segments.append((index, e_entry, p_paddr, p_data))
>> + if p_filesz > 0:
>> + p_data = elf[p_offset:p_offset + p_filesz]
>> + segments.append((index, e_entry, p_paddr, p_data))
>> return segments
>>
>> def main():
>> --
>> 2.28.0
>>
>>
>>
>
>
More information about the U-Boot
mailing list