-
Notifications
You must be signed in to change notification settings - Fork 14
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
ThinLTO breaks kernel with ZFS built-in #1731
Comments
Hmm, there is another problem. I worked around this issue (by disabling ZSWAP), but then the resultant kernel does not boot on real hardware, it boots in qemu, however. |
Chiming in as I've also been able to reproduce this exact issue earlier, even with an LLVM toolchain built from Git sources that are only a business week old (some of those TCs from that time had other problems like miscompiling This seems to no longer be reproducible. I've built a kernel with I'm not really sure which commit fixed it, but at least the commit I used is no longer affected by this bug, at least from my (admittedly somewhat minimal) testing. Versions of other things:
|
@0n-s thanks, i will try to repro again later. |
Hi. It is built-in as well, with I will try using Clang+LLVM 18 main to see if its still reproducible there. The main problem for me is that it does not produce any kernel output on boot, but compiling with CFI and LTO off reveals very similar kernel errors to the ones listed above, specially on non-clean unmounts. FYI, there are parts of ZFS and Bcachefs that overlap, to the extent of having this many issues with the keyword "bcachefs" (wild guess: might this have something to do with Kernel commit: |
I reported this in #1440, but after further investigation this looks like a different issue.
The symptom is the same as #1440:
However, this only happens, if I enable the ZFS module as kernel builtin. (Of course, disabling CONFIG_ZSWAP helps, since it worksaround that jump label).
Interesting how ZFS can trigger an codegen change in code that is seemingly completely unrelated, through LTO.
This is probably related to openzfs/zfs#13549
The text was updated successfully, but these errors were encountered: