-
Notifications
You must be signed in to change notification settings - Fork 18
Conversation
afc50a7
to
bd3a85c
Compare
4.19 needs at least ClangBuiltLinux/linux@81b4568 applied to it. I'm looking into the 4.14 fail now. |
A passing x86_64 kernel has:
A failing 4.14 kernel has:
4.4 and 4.9 have the same failures as arm. |
I wonder if glibc in our buildroot is failing to parse our kernel version string correctly? Or it looks like our buildroot might need to have the correct headers: https://forums.xilinx.com/t5/Embedded-Linux/Linux-build-for-MicroZED-FATAL-kernel-too-old/td-p/812467 |
I'll check the configuration, thanks for the link! |
Yeah I think that's probably what it is because the default header value is 4.17. I've changed it to 4.4. |
Thanks for the tip @nickdesaulniers!
|
14dbedd
to
79bcc98
Compare
On Tue, Nov 13, 2018 at 11:55 AM Nathan Chancellor ***@***.***> wrote:
***@***.***
<nathanchance@7277201>
I would keep the echo-line and do an "exit 0" instead in ...
Subject: [PATCH 1/2] DO-NOT-UPSTREAM: x86: Avoid warnings/errors due to lack
of asm goto
...
@echo Compiler lacks asm-goto support.
+ @EXIT 0
+KBUILD_CFLAGS += -D__BPF_TRACING__
...
- sed@ -
… https://travis-ci.com/nathanchance/continuous-integration/jobs/158116418
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#44 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAsJyCH0WmA0_0T8kEpPBI0AnM1hLL4Fks5uuqU7gaJpZM4YYzTp>
.
|
I'm curious, why? It doesn't avoid potential merge conflicts and we don't need to see that message every build since the whole reason the patch exists is because Clang doesn't support asm-goto. |
On Tue, Nov 13, 2018 at 6:40 PM Nathan Chancellor ***@***.***> wrote:
I would keep the echo-line and do an "exit 0"
I'm curious, why? It doesn't avoid potential merge conflicts and we don't
need to see that message every build since the whole reason the patch
exists is because Clang doesn't support asm-goto.
I hope to see asm-goto support for clang-8, then I do not care :-).
- sed@ -
… |
Older headers are needed to boot older kernels. If newer headers are used with an older kernel, the kernel panics when /sbin/init starts. ClangBuiltLinux#44 (comment) Signed-off-by: Nathan Chancellor <[email protected]>
Heh, looks like I'll have to send ClangBuiltLinux/linux@81b4568 back to stable 4.19 (possibly further) |
9636a5d
to
4d86916
Compare
commit 81b45683487a ("compiler.h: give up __compiletime_assert_fallback()") is needed because travis kills the job otherwise due to log spew. See build 271. > The job exceeded the maximum log length, and has been terminated. The proper fix is to send the backport, TODO.
4d86916
to
2badf87
Compare
ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Needs to be rebased on master. Also, we could enable 4.14 right now, it doesn't need any special patches applied to it: https://travis-ci.com/nathanchance/continuous-integration/jobs/158594813
only cron jobs added. bypassing presubmit. |
done |
branched off of #40 . Let's see what's green for x86.