-
Notifications
You must be signed in to change notification settings - Fork 97
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
[clang] additional trees/branches #103
Comments
I'm unconvinced about stable as it'd seem better to expand stable coverage to include allmodconfig (which IIRC we dropped for builder resource) as at present a bunch of stable isn't getting tested at all. |
allmodconfig SGTM; getting some coverage of stable is important. |
There's outstanding issues with the UI for clang reports that we probably want to address before rolling this out to more trees: #127 It also looks like arm64 big endian boots are broken with clang, that probably needs looking at too. |
Since boots were disabled for clang, issue #127 isn't a blocker for these additional trees and branches builds to be added. |
So, should we add arm64 defconfig and x86_64_defconfig on stable 4.9, 4.14, 4.19 with clang-8? Adding allmodconfig to all the stable branches for arm64 and x86, and on top of that both for gcc and clang would mean a significant increase in build time. I think there's still enough headroom for it but we need to be sure it's really worth it and if so then we may soon need to add more builders or reduce the build coverage on other branches. |
Yes please, but can we move to clang-9 for all clang builds?
…On Fri, Nov 8, 2019, 4:58 AM Guillaume Tucker ***@***.***> wrote:
So, should we add arm64 defconfig and x86_64_defconfig on stable 4.9,
4.14, 4.19 with clang-8?
Adding allmodconfig to all the stable branches for arm64 and x86, and on
top of that both for gcc and clang would mean a significant increase in
build time. I think there's still enough headroom for it but we need to be
sure it's really worth it and if so then we may soon need to add more
builders or reduce the build coverage on other branches.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#103?email_source=notifications&email_token=AAN5IXZV6WJP4NOASPSWLUTQSVO67A5CNFSM4H3LSSV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDRQVYY#issuecomment-551750371>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAN5IX7BSKP4NEGFF5KPOS3QSVO67ANCNFSM4H3LSSVQ>
.
|
Since clang-8 is supported upstream for at least arm64 and is shipping in released distributions like Debian it seems good to keep that, especially since features are being added to newer versions of clang and it's likely we'll get breakage for users if those aren't properly version guarded. Ideally we'd be able to support multiple versions of compilers... |
Yes we can add clang-9, which would mean more build power of course but if it's only -next master then it should be fine. There were issues with the Docker image for clang-9 though so we would need to get that back on track first. Sounds like it's worth a separate Github issue (for clang-9). |
Also removes clang-8 support. Fixes kernelci#238 Fixes kernelci#105 Fixes kernelci#103
per our discussion on the mailing list, I think it would be worthwhile to expand testing coverage of defconfigs from -next to mainline, and stable for 4.19, 4.14, 4.9, and maybe 4.4 (we do support arm64 and x86_64 defconfigs back to 4.4).
The text was updated successfully, but these errors were encountered: