Re: [PATCH 0/2] Noinstr fixes for K[CA]SAN with GCOV
Marco Elver <[email protected]> Thu, 18 Dec 2025 10:56:48 +0100
| Newsgroups | org.kernel.vger.linux-toolchains,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <CANpmjNOQJVRf5Ffk0-WMcFkTfAuh5J-ZoPHC+4BdXgLLf22Rjg@mail.gmail.com> |
On Thu, 18 Dec 2025 at 10:51, Peter Zijlstra <[email protected]> wrote: > On Sat, Dec 13, 2025 at 08:59:44AM +0900, Ard Biesheuvel wrote: > > > > After that I sat down and finally got around to implement the builtin > > > that should solve this once and for all, regardless of where it's > > > called: https://github.com/llvm/llvm-project/pull/172030 > > > What this will allow us to do is to remove the > > > "K[AC]SAN_SANITIZE_noinstr.o := n" lines from the Makefile, and purely > > > rely on the noinstr attribute, even in the presence of explicit > > > instrumentation calls. > > > > > > > Excellent! Thanks for the quick fix. Happy to test and/or look into > > the kernel side of this once this lands. > > Well, would not GCC need to grow the same thing and then we must wait > until these versions are the minimum supported versions for sanitizer > builds. > > I mean, the extension is nice, but I'm afraid we can't really use it > until much later :/ Unfortunately, yes. But let's try to get the builtin into Clang and GCC now (for the latter, need to Cc GCC folks to help). Then we wait for 5 years. :-) There's a possibility to try and backport it to stable Clang and GCC versions, but it's a long stretch (extremely unlikely).