Bug 203495
Summary: | KASAN: make inline instrumentation the default mode | ||
---|---|---|---|
Product: | Memory Management | Reporter: | Andrey Konovalov (andreyknvl) |
Component: | Sanitizers | Assignee: | MM/Sanitizers virtual assignee (mm_sanitizers) |
Status: | RESOLVED CODE_FIX | ||
Severity: | normal | CC: | dvyukov, kasan-dev, paul.heidekrueger |
Priority: | P1 | ||
Hardware: | All | ||
OS: | Linux | ||
Kernel Version: | upstream | Subsystem: | |
Regression: | No | Bisected commit-id: |
Description
Andrey Konovalov
2019-05-03 15:59:31 UTC
AFAICT, the versions of GCC and clang which are supported by the kernel all support inline instrumentation by now. So, there would be no need to make a distinction. Or am I missing something? Would you still welcome a patch making inline instrumentation the default? Outline instrumentation leads to smaller code size, and IIRC some arches support only outline. But making inline the default sounds good. On PowerPC, inline instrumentation will be disabled if PPC_RADIX_MMU is set. But Kconfig will already take care of disabling inline instrumentation in that case. That's the only arch I could find where this is relevant. I'll be submitting a patch then. Thanks! Resolved by Paul with [1]. [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=83a6fdd6c27d4f6f51fa1092805676b24e0f8827 |