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)
Wed, Apr 15, 10:17 PM
Unknown Object (File)
Wed, Apr 15, 4:45 PM
Unknown Object (File)
Wed, Apr 15, 11:22 AM
Unknown Object (File)
Wed, Apr 15, 8:24 AM
Unknown Object (File)
Wed, Apr 15, 5:57 AM
Unknown Object (File)
Sun, Apr 12, 10:28 PM
Unknown Object (File)
Wed, Apr 8, 2:58 AM
Unknown Object (File)
Mon, Apr 6, 9:49 AM
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