Paths

Table of Contentst

Differential D40037

bhyve: import OpRegion definitions
ClosedPublic
Actions

Authored by corvink on May 10 2023, 12:45 PM.

Tags

None

Referenced Files

	F96299116: D40037.id121793.diff
	Tue, Sep 24, 10:33 AM

	Unknown Object (File)
	Mon, Sep 23, 9:16 AM

	Unknown Object (File)
	Tue, Sep 17, 4:55 AM

	Unknown Object (File)
	Mon, Sep 9, 2:50 AM

	Unknown Object (File)
	Sun, Sep 8, 7:55 AM

	Unknown Object (File)
	Sun, Sep 8, 12:38 AM

	Unknown Object (File)
	Sat, Sep 7, 12:13 PM

	Unknown Object (File)
	Sun, Sep 1, 11:16 AM

View All 73 Files

Subscribers

Details

Reviewers

jhb
markj

Group Reviewers

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.

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

Path

Size

usr.sbin/

bhyve/

pci_gvt-d-opregion.h

186 lines

Diff 121842

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

Loading...