Page MenuHomeFreeBSD

bhyve: Appease warning about a potentially unaligned pointer.
ClosedPublic

Authored by jhb on Nov 23 2022, 11:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 9, 6:11 PM
Unknown Object (File)
Thu, May 9, 6:11 PM
Unknown Object (File)
Thu, May 9, 6:06 PM
Unknown Object (File)
Mon, May 6, 8:35 PM
Unknown Object (File)
Sun, May 5, 2:21 AM
Unknown Object (File)
Fri, Apr 19, 4:08 PM
Unknown Object (File)
Apr 12 2024, 7:24 PM
Unknown Object (File)
Apr 12 2024, 2:15 PM
Subscribers

Details

Summary

When initializing the device model for a PCI pass through device that
uses MSI-X, bhyve reads the MSI-X capability from the real device to
save a copy in the emulated PCI config space. It also saves a copy in
a local struct msixcap on the stack. Since struct msixcap is packed,
GCC complains that casting a pointer to the struct to a uint32_t
pointer may result in an unaligned pointer.

This path is not performance critical, so to appease the compiler,
simply change the pointer to a char * and use memcpy to copy the 4
bytes read in each iteration of the loop.

Diff Detail

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

Event Timeline

jhb requested review of this revision.Nov 23 2022, 11:16 PM

I realize the person from illumos might have had a different fix for this, but I think this might be the cleanest of the options. I tried adding an __Alignas(4) on the local variable and that didn't appease GCC's warning here.

This revision is now accepted and ready to land.Nov 24 2022, 6:38 AM