On Mon, Nov 6, 2023 at 4:21 AM A. Sverdlin
<[email protected]> wrote:
>
> From: Alexander Sverdlin <[email protected]>
>
> There are use cases for type=kmeta in kernel-yocto.bbclass-derived SRC_URI
> which do not point to git://git.yoctoproject.org/yocto-kernel-cache but to
> file:// local storage containing just some .scc files. It's not obvious
> that they shall pull in the whole automatic kernel configuration of
> meta-virtualization. Make this behaviour configurable by allowing
> KERNEL_META_TYPE to be defined in local.conf or distro config.
This is an ok temporary solution, but I'd only merge something like
this with a bbwarning also being emitted at the same time.
As I keep looking at it, there's very little in the .inc file that isn't
controlled by a distro feature, so there's only small tweaks needed
to make sure that only fragments and kernel features that are
quite well targeted are enabled.
Having the kernel-cache as a dependency is not much to ask,
combined with the ability to use them to configure kernels
properly and audit the results.
The hours (more like weeks and months) spent debugging runtime
issues due to incorrectly configured kernels has been huge over
the lifetime of meta-virtualization. Asking for a few fragments to
be downloaded (which can be mirrored) is minor in the scope of
things.
Which is why if a kernel type that can merge fragments, but yet
is disabling the feature specification, the layer should emit a
warning. I'd then have another variable which inhibits the warning
message. Since that is something I can ask about in bug reports
and until the warning message is re-enabled, the reporter is on
their own.
Cheers,
Bruce
> ---
> recipes-kernel/linux/linux-%.bbappend | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/recipes-kernel/linux/linux-%.bbappend
> b/recipes-kernel/linux/linux-%.bbappend
> index 5f31578..c2fc91c 100644
> --- a/recipes-kernel/linux/linux-%.bbappend
> +++ b/recipes-kernel/linux/linux-%.bbappend
> @@ -5,6 +5,6 @@ LINUX_MAJOR = "${@(d.getVar('LINUX_VERSION') or
> "x.y").split('.')[0]}"
> LINUX_MINOR = "${@(d.getVar('LINUX_VERSION') or "x.y").split('.')[1]}"
>
>
> -KERNEL_META_TYPE = "${@'yocto' if d.getVar('SRC_URI').find('type=kmeta') > 0
> else 'none'}"
> +KERNEL_META_TYPE ?= "${@'yocto' if d.getVar('SRC_URI').find('type=kmeta') >
> 0 else 'none'}"
>
> include ${@bb.utils.contains('DISTRO_FEATURES', 'virtualization',
> 'linux-${KERNEL_META_TYPE}_${LINUX_MAJOR}.${LINUX_MINOR}_virtualization.inc',
> '', d)}
> --
> 2.41.0
>
--
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8425):
https://lists.yoctoproject.org/g/meta-virtualization/message/8425
Mute This Topic: https://lists.yoctoproject.org/mt/102416724/21656
Group Owner: [email protected]
Unsubscribe:
https://lists.yoctoproject.org/g/meta-virtualization/leave/6693005/21656/1014668956/xyzzy
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
