Page MenuHomeFreeBSD

libvmmapi: Move some ioctl wrappers to vmmapi_machdep.c
ClosedPublic

Authored by markj on Jul 12 2023, 1:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 16, 11:36 PM
Unknown Object (File)
Thu, May 16, 11:36 PM
Unknown Object (File)
May 15 2024, 12:55 AM
Unknown Object (File)
May 15 2024, 12:32 AM
Unknown Object (File)
May 2 2024, 11:35 AM
Unknown Object (File)
May 2 2024, 11:31 AM
Unknown Object (File)
Apr 27 2024, 12:37 PM
Unknown Object (File)
Apr 27 2024, 12:37 PM
Subscribers

Details

Summary

ioctls relating to segments and various x86-specific interrupt
controllers are easy candidates to move to vmmapi_machdep.c.

In vmmapi.h I'm just ifdefing MD prototypes for now. We could instead
split vmmapi.h into multiple headers, e.g., vmmapi.h and
vmmapi_machdep.h, but it's not obvious to me yet that that's the right
approach.

No functional change intended.

Diff Detail

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

Event Timeline

markj requested review of this revision.Jul 12 2023, 1:52 PM

I think #ifdef's is fine for now. It might be nice to reorder the declaration to place all the MD prototypes in one #ifdef block.

This revision is now accepted and ready to land.Jul 12 2023, 6:51 PM
In D40999#933353, @jhb wrote:

I think #ifdef's is fine for now. It might be nice to reorder the declaration to place all the MD prototypes in one #ifdef block.

I see you #ifdef more in followups, so maybe a cleanup commit at the end of the series to shuffle down into a single #ifdef block?

In D40999#933366, @jhb wrote:
In D40999#933353, @jhb wrote:

I think #ifdef's is fine for now. It might be nice to reorder the declaration to place all the MD prototypes in one #ifdef block.

I see you #ifdef more in followups, so maybe a cleanup commit at the end of the series to shuffle down into a single #ifdef block?

Yeah, good idea. I'll do that once things are close to being finalized.