Skip to content
This repository has been archived by the owner on Apr 13, 2024. It is now read-only.

continuous-integration: Add allyesconfig #143

Closed
wants to merge 1 commit into from

Conversation

tapaswenipathak
Copy link
Member

@tapaswenipathak tapaswenipathak commented Mar 31, 2019

I mailed Travis about increasing the limit.

tl;dr this should close #27.

@nathanchance
Copy link
Member

allyesconfig is a build target, it should be called by mako_reactor allyesconfig here:

mako_reactor ${config}

arm64 (and maybe x86_64) on linux-next will be the only targets that build successfully with it though.

@nathanchance nathanchance added the WIP Work in progress label Apr 8, 2019
@tpimh
Copy link
Contributor

tpimh commented Oct 7, 2019

Any progress on this one? There has been changes to the build system, this should be rebased.

@nathanchance
Copy link
Member

I still do not believe this will be a good addition to our matrix. Our latest build took almost 9 hours to complete because more and more files get built with each kernel release. Adding allyesconfig to this will make it take significantly longer because we can only have three builds going at a time. For that matter, we only get 180 minutes maximum per job and an x86 allyesconfig build on a four core machine (Packet's t1.small.x86) with LLVM 10 on Ubuntu 18.04 takes around 5 hours and requires at least 8GB of disk space for the kernel source and resulting files (we only get two cores and 18GB of disk space). I don't think Travis will be able to handle this.

@tpimh
Copy link
Contributor

tpimh commented Oct 8, 2019

Maybe changes should be made only to driver.sh and not affect our travis matrix. I would suggest introducing CONFIG variable which can be specified like REPO. If unspecified, the same old set of defconfigs is used, if set to "allyesconfig", builds allyesconfig. So it would be extremely easy to test locally by just running ./driver.sh CONFIG=allyesconfig and waiting for a while.

@nathanchance
Copy link
Member

Yes that would be okay. We could also come up with a list of common configs that tend to stress issues in LLVM/Clang and build with those (some immediate ones are CONFIG_RANDOMIZE_BASE and CONFIG_KASAN).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

more configs
3 participants