Re: [PATCH L-C v3 0/7] linux-custom recipe rework

[Cedric Hombourger <Cedric_Hombourger@mentor.com>]

On 11/8/19 9:01 AM, Cedric Hombourger wrote:
>
> On 11/8/19 8:54 AM, Jan Kiszka wrote:
>> On 08.11.19 08:42, Cedric Hombourger wrote:
>>> On 11/8/19 8:37 AM, Jan Kiszka wrote:
>>>> On 08.11.19 08:27, Cedric Hombourger wrote:
>>>>> On 11/8/19 8:12 AM, Jan Kiszka wrote:
>>>>>> On 08.11.19 08:07, Cedric Hombourger wrote:
>>>>>>> The Linux kernel is packaged a bit differently between Debian 
>>>>>>> variants
>>>>>>> and in particular for the perf package(s) in Debian vs Ubuntu. 
>>>>>>> Since
>>>>>>> there is no way the kernel build scripts would ever get distro
>>>>>>> specific
>>>>>>> support for building more than the kernel and headers, we need 
>>>>>>> our own
>>>>>>> build/packaging scripts.
>>>>>>>
>>>>>>> Version 3 of this changeset was tested against ci_build.sh and some
>>>>>>> images were
>>>>>>> booted on hardware available here. No regressions were detected (to
>>>>>>> date). This
>>>>>>> was also tested on the creator-ci40-marduk machine (subject of
>>>>>>> another patch
>>>>>>> series).
>>>>>> dts files weren't tested so far. They end up in the wrong directory
>>>>>> on the target and, thus, will not be found by the boot loader. I can
>>>>>> likely tell you in a moment which part exactly is wrong.
>>>>> ok thanks. the boards I tested either do not use dtb files or require
>>>>> a custom dtb path setting anyway
>>>> I strongly recommend to switch everything to ditro boot. That's
>>>> usually just a simple patch to the default u-boot env. More and more
>>>> vendors support that, and it's the unified way of booting Isar.
>>> I can't agree more (need to work with the vendor of the board I have to
>>> update their insanely old u-boot, it is insane to have it supported 
>>> with
>>> mainline kernels but keep a 4-years old u-boot).
>> Ah, another "we are done" BSP. Yeah, never use BSPs...
>>
>>
>> BTW, another fallout from the full nightly build:
>>
>> NOTE: recipe isar-image-base-1.0-r0: task do_copy_boot_files: Started
>> ERROR: mc:qemumipsel-bullseye:isar-image-base-1.0-r0 
>> do_copy_boot_files: Execution of 
>> '/builds/ebsy/debian/isar/build/tmp/work/debian-bullseye-mipsel/isar-image-base-qemumipsel-ext4-img/1.0-r0/temp/run.do_copy_boot_files.105304' 
>> failed with exit code 1:
>> cp: cannot stat 
>> '/builds/ebsy/debian/isar/build/tmp/work/debian-bullseye-mipsel/isar-image-base-qemumipsel-ext4-img/1.0-r0/rootfs/vmlinuz': 
>> No such file or directory
>> WARNING: exit code 1 from a shell command.
>>
>> ERROR: Logfile of failure stored in: 
>> /builds/ebsy/debian/isar/build/tmp/work/debian-bullseye-mipsel/isar-image-base-qemumipsel-ext4-img/1.0-r0/temp/log.do_copy_boot_files.105304
>> Log data follows:
>> | DEBUG: Executing shell function do_copy_boot_files
>> | cp: cannot stat 
>> '/builds/ebsy/debian/isar/build/tmp/work/debian-bullseye-mipsel/isar-image-base-qemumipsel-ext4-img/1.0-r0/rootfs/vmlinuz': 
>> No such file or directory
>> | WARNING: exit code 1 from a shell command.
>> | ERROR: Execution of 
>> '/builds/ebsy/debian/isar/build/tmp/work/debian-bullseye-mipsel/isar-image-base-qemumipsel-ext4-img/1.0-r0/temp/run.do_copy_boot_files.105304' 
>> failed with exit code 1:
>> | cp: cannot stat 
>> '/builds/ebsy/debian/isar/build/tmp/work/debian-bullseye-mipsel/isar-image-base-qemumipsel-ext4-img/1.0-r0/rootfs/vmlinuz': 
>> No such file or directory
>> | WARNING: exit code 1 from a shell command.
>> |
>> NOTE: recipe isar-image-base-1.0-r0: task do_copy_boot_files: Failed
>> ERROR: Task 
>> (mc:qemumipsel-bullseye:/builds/ebsy/debian/isar/meta-isar/recipes-core/images/isar-image-base.bb:do_copy_boot_files) 
>> failed with exit code '1'
>>
>> Can send you the link the full log on our internal CI on
>> code.siemens.com.
> shouldn't be needed. looks like KERNEL_FILE setting isn't honored for 
> this target. should be easy to determine why that isn't the case

from a quick code inspection:

meta-isar/conf/machine/qemumipsel.conf:KERNEL_FILE ?= "vmlinux"

sounds like it should be = not ?=

but we could also provide a better default so we don't have to do this 
for every single MIPS machine

our global def config has:

KERNEL_FILE ?= "vmlinuz"

we could add

KERNEL_FILE_mipsel ?= "vmlinux"

will add this change to the relevant patch series

>>
>> Jan
>>