Re: [PATCH] ARM: ubsan: select ARCH_HAS_UBSAN_SANITIZE_ALL

From: Florian Fainelli
Date: Wed Oct 12 2022 - 15:46:52 EST


On 10/4/22 01:51, Linus Walleij wrote:
On Wed, Sep 28, 2022 at 7:47 PM Florian Fainelli <f.fainelli@xxxxxxxxx> wrote:

From: Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx>

To enable UBSAN on ARM, this patch enables ARCH_HAS_UBSAN_SANITIZE_ALL
from arm confiuration. Basic kernel bootup test is passed on arm with
CONFIG_UBSAN_SANITIZE_ALL enabled.

Signed-off-by: Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx>
[florian: rebased against v6.0-rc7]
Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx>

Pretty cool that it "just works", I was thinking to myself that I
should try this one
day. Sorry I was late to look at the patch.

Have you also looked at KCSAN?

Not yet, that would be next. Most often, at least for ARCH_BRCMSTB the same hardware and set of drivers run on ARM64 which is typically much more feature full, so we get coverage on *SAN features that way.
--
Florian