[OE-core] [RFC PATCH 0/6] Kernel configuration check for recipes

[Bruce Ashfield]

On Mon, May 9, 2016 at 12:43 AM, Paul Eggleton <
paul.eggleton at linux.intel.com> wrote:

> Add the capability for other recipes to verify that kernel
> configuration options they depend upon at runtime are present, and warn
> if they are missing. This seems functional enough to me but is still in
> RFC - please let me know if it looks OK to you or I've missed anything
> (no doubt there are recipes we could add the checks to, I refer mainly
> to the mechanism itself).
>
>
> Please review the following changes for suitability for inclusion. If you
> have
> any objections or suggestions for improvement, please respond to the
> patches. If
> you agree with the changes, please provide your Acked-by.
>
>
See my replies to the patches. I'm sure it won't matter in the end, but
this is a duplication
of the mechanisms that we already have in the kernel config check phase. I
have a whole
set of changes that I completed early this year that extends that mechanism
to any kernel
(not just kernel-yocto) classes. In that mechanism there's the concept of
"required" and
"optional" Kconfig options. If they aren't present in the final .config, we
warn and optionally
error (this is versus the currently mechanism of only warning if boot
critical options are
dropped).

I'm betting that the follow on to this series will be "Hey recipes should
set the options for the kernel if
they really need them" .. which IMHO is an incredibly bad idea.

Cheers,

Bruce


> The following changes since commit
> ece101be5158beee709cdfbb85ecdbdc8d9fb864:
>
>   test-empty-image: Fix LIC_FILES_CHKSUM typo (2016-05-06 10:47:59 +0100)
>
> are available in the git repository at:
>
>   git://git.openembedded.org/openembedded-core-contrib paule/kernel-check
>
> http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/kernel-check
>
> Paul Eggleton (6):
>   classes/kernel-check: add a class to check kernel config options
>   eudev: check for required kernel config options
>   systemd: check for required kernel config options
>   classes/kernel: fix typo
>   classes/kernel: check OLDEST_KERNEL at configure time
>   classes/image: check kernel config supports IMAGE_FSTYPES items
>
>  meta/classes/image.bbclass               | 23 ++++++++
>  meta/classes/image_types.bbclass         | 11 ++++
>  meta/classes/kernel-check.bbclass        | 97
> ++++++++++++++++++++++++++++++++
>  meta/classes/kernel.bbclass              | 16 +++++-
>  meta/recipes-core/systemd/systemd_229.bb | 19 ++++++-
>  meta/recipes-core/udev/eudev_3.1.5.bb    | 22 +++++++-
>  6 files changed, 185 insertions(+), 3 deletions(-)
>  create mode 100644 meta/classes/kernel-check.bbclass
>
> --
> 2.5.5
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20160509/d77f86d0/attachment-0002.html>