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, Apr 18, 11:39 PM
Unknown Object (File)
Wed, Apr 10, 7:24 PM
Unknown Object (File)
Mar 19 2024, 9:34 AM
Unknown Object (File)
Feb 10 2024, 2:48 PM
Unknown Object (File)
Feb 4 2024, 9:00 AM
Unknown Object (File)
Jan 9 2024, 8:16 PM
Unknown Object (File)
Dec 11 2023, 3:07 AM
Unknown Object (File)
Sep 20 2023, 2:56 AM
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 Not Applicable
Unit
Tests Not Applicable

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.