bhyve: add hook for command register of passthru devices
Needs ReviewPublic
Actions

Authored by corvink on Mar 28 2023, 10:27 AM.

Tags

None

Referenced Files

	F122263155: D39292.id.diff
	Thu, Jul 3, 8:30 PM

	F122194906: D39292.id119548.diff
	Thu, Jul 3, 5:09 AM

	Unknown Object (File)
	Tue, Jul 1, 7:32 PM

	Unknown Object (File)
	Sat, Jun 28, 1:53 PM

	Unknown Object (File)
	Fri, Jun 20, 3:26 PM

	Unknown Object (File)
	Apr 18 2025, 11:36 AM

	Unknown Object (File)
	Mar 16 2025, 4:35 PM

	Unknown Object (File)
	Feb 26 2025, 3:22 PM

Subscribers

Details

Reviewers

jhb
markj

Group Reviewers

bhyve

Summary

Move the emulation of the command register into it's own handler.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 50633
Build 47524: arc lint + arc unit

Event Timeline

corvink created this revision.Mar 28 2023, 10:27 AM

Herald added a reviewer: bhyve. · View Herald TranscriptMar 28 2023, 10:27 AM

Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript

corvink requested review of this revision.Mar 28 2023, 10:27 AM

Harbormaster completed remote builds in B50615: Diff 119548.Mar 28 2023, 10:27 AM

corvink added a child revision: D39293: bhyve: add hook for msi/msix accesses to passthru devices.Mar 28 2023, 10:27 AM

corvink added a parent revision: D33010: bhyve: add hook for PCI header of passthru devices.

markj accepted this revision as: markj.Mar 28 2023, 1:52 PM

This revision is now accepted and ready to land.Mar 28 2023, 1:52 PM

guest-patmaddox added a subscriber: guest-patmaddox.Mar 28 2023, 4:02 PM

Fix emulation of command writes if LEGACY_SUPPORT is enabled.

This revision now requires review to proceed.Mar 29 2023, 7:03 AM

Harbormaster completed remote builds in B50633: Diff 119582.Mar 29 2023, 7:03 AM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

pci_passthru.c

96 lines

Diff 119582

View Options

usr.sbin/bhyve/pci_passthru.c

bhyve: add hook for command register of passthru devicesNeeds ReviewPublicActions