Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use minimal configurations for Kconfig configuration files (Linux, coreboot) #416

Closed
paulmenzel opened this issue May 19, 2018 · 3 comments

Comments

@paulmenzel
Copy link
Contributor

In projects using Kconfig, make help shows the description below for savedefconfig.

savedefconfig - Save current config as ./defconfig (minimal config)

Using the minimal configuration avoids the hassle to update configuration files on version updates even when nothing from our selections actually changed. It’s also easier to grasp, what is actually selected as it’s much shorter.

@osresearch
Copy link
Collaborator

osresearch commented Jul 18, 2018

For qemu-linuxboot the generated defconfig file is quite a bit shorter (9K versus 16KB).

How do you apply it? Running make defconfig uses x86_64_defconfig, although I'm not sure where it is coming from. Trying to override on the command line doesn't seem to work, either:

[user@heads-f26 linux-qemu]$ rm .config
[user@heads-f26 linux-qemu]$ make defconfig KBUILD_DEFCONFIG=`pwd`/qemu-defconfig
  GEN     ./Makefile
*** Default configuration is based on target '/home/user/heads/build/linux-4.9.80/linux-qemu/qemu-defconfig'
  GEN     ./Makefile
make[5]: Nothing to be done for '/home/user/heads/build/linux-4.9.80/linux-qemu/qemu-defconfig'.
[user@heads-f26 linux-qemu]$ grep LOCAL .config
grep: .config: No such file or directory
[user@heads-f26 linux-qemu]$ grep LOCAL qemu-defconfig 
CONFIG_LOCALVERSION="-headsdef"

@paulmenzel
Copy link
Contributor Author

paulmenzel commented Jul 18, 2018 via email

osresearch added a commit that referenced this issue Aug 9, 2018
This reduces the amount of noise in the Linux kernel config files
by only storing the differences from the stock configuration.
It adds a new makefile target 'linux.saveconfig' to convert the
build tree's .config file into config/linux-linuxboot.config.
@osresearch
Copy link
Collaborator

This is now implemented in the 4.14 update PR: f4e25dd

It cleans up the config/linux-*.config files immensely. Thanks for suggesting it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants