Page MenuHomeFreeBSD

sys/conf: Introduce NOSAN_CFLAGS and NOSAN_C
AcceptedPublic

Authored by andrew on Jun 5 2024, 12:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 10, 4:17 AM
Unknown Object (File)
Wed, Jul 10, 4:16 AM
Unknown Object (File)
Wed, Jul 10, 4:16 AM
Unknown Object (File)
Sat, Jul 6, 8:39 PM
Unknown Object (File)
Mon, Jul 1, 3:20 AM
Unknown Object (File)
Sun, Jun 30, 12:01 PM
Unknown Object (File)
Jun 12 2024, 12:50 AM
Unknown Object (File)
Jun 6 2024, 5:28 PM
Subscribers

Details

Summary

To simplify disabling the kernel sanitizers in some files add
NOSAN_CFLAGS and NOSAN_C variables. These are CFLAGS and NORMAL_C with
the sanitizer flags removed.

While here add MSAN_CFLAGS to simplify keeping KMSAN in kern_kcov.c

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58218
Build 55106: arc lint + arc unit

Event Timeline

Sorry for not having done this when I added the -fno-sanitize* pattern.

This revision is now accepted and ready to land.Jun 5 2024, 1:53 PM

Fix the module build. NOSAN_CFLAGS was wrong when used in kmod.mk

This revision now requires review to proceed.Jun 10 2024, 4:49 PM
sys/conf/kern.pre.mk
220

Does it not work to define these in kern.mk instead of here and in kmod.mk?

Move NOSAN_CFLAGS to kern.mk

This revision is now accepted and ready to land.Wed, Jun 19, 4:38 PM
markj added inline comments.
sys/conf/kern.mk
297

Maybe mention NOSAN_C in a comment here.