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
F83927451: D40999.id136849.diff
Thu, May 16, 11:36 PM
F83927447: D40999.id.diff
Thu, May 16, 11:36 PM
Unknown Object (File)
Wed, May 15, 12:55 AM
Unknown Object (File)
Wed, May 15, 12:32 AM
Unknown Object (File)
Thu, May 2, 11:35 AM
Unknown Object (File)
Thu, May 2, 11:31 AM
Unknown Object (File)
Sat, Apr 27, 12:37 PM
Unknown Object (File)
Sat, Apr 27, 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 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.