Page MenuHomeFreeBSD

ena: Make first_interrupt a uint8_t
ClosedPublic

Authored by markj on Jun 30 2022, 6:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 28, 6:27 PM
Unknown Object (File)
Tue, Nov 18, 3:39 AM
Unknown Object (File)
Fri, Nov 14, 1:42 PM
Unknown Object (File)
Fri, Nov 14, 1:42 PM
Unknown Object (File)
Fri, Nov 14, 1:42 PM
Unknown Object (File)
Fri, Nov 14, 10:56 AM
Unknown Object (File)
Thu, Nov 13, 11:59 PM
Unknown Object (File)
Tue, Nov 4, 2:30 PM
Subscribers

Details

Summary

Avoid passing vars of type bool to atomic(9) routines. The size of bool
is not fixed by the C standard, even though it's one byte by convention.

This fixes the KASAN and KMSAN builds, which fail because the compiler
refuses to silently cast a _Bool * to a uint8_t * when calling the
atomic(9) interceptors.

Fixes: 0ac122c388d9 ("ena: Use atomic_load/store functions for first_interrupt variable")

Test Plan

build GENERIC-KASAN and GENERIC-KMSAN amd64 kernels

Diff Detail

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