Page MenuHomeFreeBSD
Differential D40037

bhyve: import OpRegion definitions
ClosedPublic
Actions

Authored by corvink on May 10 2023, 12:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 1, 7:21 AM
Unknown Object (File)
Fri, Feb 28, 10:50 AM
Unknown Object (File)
Thu, Feb 27, 11:36 AM
Unknown Object (File)
Wed, Feb 26, 10:15 PM
Unknown Object (File)
Wed, Feb 26, 6:36 PM
Unknown Object (File)
Wed, Feb 26, 6:36 PM
Unknown Object (File)
Tue, Feb 18, 4:29 AM
Unknown Object (File)
Sun, Feb 16, 6:52 PM
View All 99 Files
Subscribers
bcran
imp
rgrimes

Details

Reviewers
jhb
markj
Group Reviewers
bhyve
Commits
rG6ae7ed9e3854: bhyve: import OpRegion definitions
rGb67f0d360b87: bhyve: import OpRegion definitions
Summary

Those definitions are required for the GVT-d emulation to parse the
OpRegion.

Diff Detail

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

Event Timeline

corvink created this revision.May 10 2023, 12:45 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptMay 10 2023, 12:45 PM
corvink requested review of this revision.May 10 2023, 12:45 PM
Harbormaster completed remote builds in B51411: Diff 121793.May 10 2023, 12:45 PM
corvink added a child revision: D40038: bhyve: add empty GVT-d emulation.May 10 2023, 12:51 PM
corvink added a parent revision: D40036: bhyve: add helper for passthru specific mmio ranges.
markj added inline comments.May 10 2023, 2:50 PM
usr.sbin/bhyve/pci_gvt-d.h
5 ↗(On Diff #121793)

This file comes directly from edk2, but is missing the corresponding copyright header.

35 ↗(On Diff #121793)

Please use __packed, to be consistent with the rest of bhyve.

corvink updated this revision to Diff 121842.May 11 2023, 10:35 AM
  • use copyright header from edk2
  • make use of __packed instead of pragma pack
Harbormaster completed remote builds in B51437: Diff 121842.May 11 2023, 10:35 AM
corvink edited child revisions, added: D40040: bhyve: read OpRegion address and size for GVT-d; removed: D40038: bhyve: add empty GVT-d emulation.May 11 2023, 12:32 PM
corvink removed a parent revision: D40036: bhyve: add helper for passthru specific mmio ranges.
markj accepted this revision as: markj.May 11 2023, 2:37 PM
This revision is now accepted and ready to land.May 11 2023, 2:37 PM
Closed by commit rGb67f0d360b87: bhyve: import OpRegion definitions (authored by corvink). · Explain WhyMay 12 2023, 7:49 AM
This revision was automatically updated to reflect the committed changes.
corvink added a commit: rGb67f0d360b87: bhyve: import OpRegion definitions.
corvink added a commit: rG6ae7ed9e3854: bhyve: import OpRegion definitions.Aug 18 2023, 7:43 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
186 lines

Diff 121842

View Options

usr.sbin/bhyve/pci_gvt-d-opregion.h

Loading...