Page MenuHomeFreeBSD

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

Authored by khng on Jun 11 2024, 10:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 2:59 AM
Unknown Object (File)
Sun, Dec 8, 2:59 AM
Unknown Object (File)
Sun, Dec 8, 2:59 AM
Unknown Object (File)
Fri, Dec 6, 8:04 AM
Unknown Object (File)
Mon, Dec 2, 8:42 AM
Unknown Object (File)
Nov 23 2024, 2:05 PM
Unknown Object (File)
Nov 17 2024, 5:05 PM
Unknown Object (File)
Nov 12 2024, 8:19 PM
Subscribers

Details

Summary

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

Sponsored by: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

khng requested review of this revision.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

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.Oct 19 2024, 12:52 PM

Remove duplicated fsanitize flags during module build

This revision now requires review to proceed.Nov 10 2024, 3:46 AM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 12 2024, 7:05 PM
This revision was automatically updated to reflect the committed changes.