Re: [PATCH 3/4] linux-custom: Use uncompressed kernel image on arm64

[Jan Kiszka <jan.kiszka@siemens.com>]

On 2018-08-27 10:31, [ext] Jan Kiszka wrote:
> On 2018-08-27 10:29, Henning Schild wrote:
>> Am Mon, 27 Aug 2018 09:19:54 +0200
>> schrieb "[ext] Jan Kiszka" <jan.kiszka@siemens.com>:
>>
>>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>>
>>> U-Boot does not directly support decompression on bootloader side like
>>> Image.gz would require. We would have to enable the unzip command and
>>> write special scripts that perform the decompression prior to calling
>>> booti. As the Debian kernel also comes uncompressed, choose that path
>>> instead.
>>>
>>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>>> ---
>>>   meta/recipes-kernel/linux/files/build-kernel.sh | 7 +++++++
>>>   1 file changed, 7 insertions(+)
>>>
>>> diff --git a/meta/recipes-kernel/linux/files/build-kernel.sh
>>> b/meta/recipes-kernel/linux/files/build-kernel.sh index
>>> 8b58e93..ef30aea 100644 ---
>>> a/meta/recipes-kernel/linux/files/build-kernel.sh +++
>>> b/meta/recipes-kernel/linux/files/build-kernel.sh @@ -95,6 +95,13 @@
>>> if [ \"\$1\" != upgrade ] && command -v linux-update-symlinks
>>>> /dev/null; then\\ linux-update-symlinks remove
>>>> ${PV}  /boot/vmlinuz-${PV}\\ fi"
>>> +# Make sure arm64 kernels are decompressed
>>> +if [ "$target_arch" = "arm64" ]; then
>>> +    vmlinuz=${REPACK_LINUX_IMAGE_DIR}/boot/vmlinuz-${PV}
>>> +    mv $vmlinuz $vmlinuz.gz
>>> +    gunzip $vmlinuz
>>> +fi
>>
>> That looks weird. The gunzip should not work after the mv. And would is
>> the filename after the intended patch? If it still contains a "z", that
>> would be misleading.
> 
> gunzip appends .gz. We maybe that also works explicitly, didn't test.

Of course, both variants work. Will send v2 which addresses $vmlinuz.gz 
explicitly.

Jan

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux