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

Add arm64 support for "make check" #27

Merged
merged 2 commits into from
Jun 21, 2023
Merged

Conversation

yanjiew1
Copy link
Collaborator

This pull request add arm64 support for "make check".

I added a new kernel config file for arm64.
The command used to configure the kernel is changed from "make oldconfig" to "make olddefconfig" as "make oldconfig" may ask users to confirm configs that are not included in ".config" file.

The arm64 kernel configuration file is generated by using the command, "make savedefconfig", which can be used to generate a minimal configuration file with the filename, "defconfig". This file can be renamed to ".config", and then we can use "make olddefconfig" to convert it to a full configuration file.

"make oldconfig" may ask the user to confirm settings not in the
".config" file. However, the stdout is redirected to /dev/null.

By changing "oldconfig" to "olddefconfig", the kernel configuration
system will just use the default values for settings not found in
".config".
mk/external.mk Outdated Show resolved Hide resolved
Copy link
Contributor

@jserv jserv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the naming scheme configs/linux-aarch64.config and configs/linux-x86_64.config

@jserv
Copy link
Contributor

jserv commented Jun 20, 2023

By using olddefconfig, we can shorten the configures for x86-64. Let's do that in further pull request(s).

mk/external.mk Outdated Show resolved Hide resolved
mk/external.mk Outdated Show resolved Hide resolved
@jserv jserv merged commit b8887b8 into sysprog21:master Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants