Page MenuHomeFreeBSD

bhyve: Move some more amd64-specific drivers to their own subdir
ClosedPublic

Authored by markj on Jun 14 2023, 9:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 15, 12:44 AM
Unknown Object (File)
Sat, May 11, 8:16 PM
Unknown Object (File)
Wed, May 8, 3:36 PM
Unknown Object (File)
May 3 2024, 2:59 AM
Unknown Object (File)
Apr 21 2024, 5:38 PM
Unknown Object (File)
Mar 9 2024, 7:32 PM
Unknown Object (File)
Mar 5 2024, 4:17 AM
Unknown Object (File)
Feb 29 2024, 4:47 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Jun 14 2023, 9:52 PM
jhb added inline comments.
usr.sbin/bhyve/snapshot.c
139

Hummm, this is fine for now. I suspect some of Gustav's other changes in flight for this stuff might remove the need for this as a hardcoded table anyway.

This revision is now accepted and ready to land.Jun 14 2023, 11:28 PM
markj added inline comments.
usr.sbin/bhyve/snapshot.c
139

Yes, this is ugly, but so is the table. Device model drivers should contain whatever metadata is needed for snapshots to work.

markj marked an inline comment as done.

Rebase, handle review feedback.

This revision now requires review to proceed.Jun 15 2023, 10:13 PM
This revision is now accepted and ready to land.Jun 16 2023, 5:31 AM

Will you combine all of the move-amd64-files into one commit?

Will you combine all of the move-amd64-files into one commit?

I could, but I wasn't specifically planning to. Do you have a preference?

I could, but I wasn't specifically planning to. Do you have a preference?

At first I saw only a couple of changes moving files and it seemed a bit odd, then I realized there's a dozen or so. Given that it's already split up like this I'd leave it alone rather than spending a lot more effort for little value.