sys: move SAN and COVERAGE options handling to kern.mk
AcceptedPublic
Actions

Authored by khng on Jun 11 2024, 10:23 PM.

Details

Reviewers

allanjude
markj

Summary

This allows the flags to be picked up more easily when building external
modules.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 60064
Build 56948: arc lint + arc unit

Event Timeline

khng created this revision.Jun 11 2024, 10:23 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 11 2024, 10:23 PM

khng requested review of this revision.Jun 11 2024, 10:23 PM

Harbormaster completed remote builds in B58147: Diff 139738.Jun 11 2024, 10:23 PM

khng added a child revision: D45564: sys: Emit KERN_OPTS_EXTRA options to opt_global.h.Jun 11 2024, 10:23 PM

I think there's some overlap here with D45498. At a glance this seems fine to me.

Updated to adapt to NOSAN's introduction

Harbormaster completed remote builds in B60064: Diff 145106.Fri, Oct 18, 2:00 PM

This seems ok, but the kernel build is complicated and it's hard to be confident in such changes just by reading the diff. Please make sure you test building of all of the *SAN kernels (amd64 and arm64) and look at the build output to verify that -fsanitizer is being specified both for the kernel and for kernel modules.

This revision is now accepted and ready to land.Sat, Oct 19, 12:52 PM