On Mon, Nov 6, 2023 at 8:41 AM Bruce Ashfield via lists.yoctoproject.org <[email protected]> wrote: > > 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.
My solution is now in master-next: https://git.yoctoproject.org/meta-virtualization/commit/?h=master-next&id=45c2297a916516e5fc7b82dab46c4719590813b7 Bruce > > 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 > > > -- - 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 (#8428): https://lists.yoctoproject.org/g/meta-virtualization/message/8428 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]] -=-=-=-=-=-=-=-=-=-=-=-
