Page MenuHomeFreeBSD
Differential D33739

bhyve: Map the right BAR in init_msix_table()
ClosedPublic
Actions

Authored by markj on Jan 4 2022, 7:11 PM.
Tags
None
Referenced Files
F153474147: D33739.id100974.diff
Tue, Apr 21, 9:05 AM
F153473051: D33739.id100933.diff
Tue, Apr 21, 8:57 AM
F153447819: D33739.id100974.diff
Tue, Apr 21, 5:46 AM
F153443436: D33739.id.diff
Tue, Apr 21, 5:12 AM
F153399879: D33739.diff
Mon, Apr 20, 10:49 PM
Unknown Object (File)
Mon, Apr 20, 4:13 PM
Unknown Object (File)
Sat, Apr 18, 10:55 PM
Unknown Object (File)
Wed, Apr 15, 11:50 AM
View All Files
Subscribers
bcran
imp
rgrimes

Details

Reviewers
jhb
Group Reviewers
bhyve
Commits
rG4558c11f1b4d: bhyve: Correct unmapping of the MSI-X table BAR
rG76b45e688ae8: bhyve: Map the right BAR in init_msix_table()
Summary

The PBA and MSI-X table can reside in different BARs.

Reported by: Andy Fiddaman <andy@omniosce.org>
Fixes: 7fa233534736 ("bhyve: Map the MSI-X table unconditionally for passthrough")
MFC after: 3 days

Diff Detail

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

Event Timeline

markj created this revision.Jan 4 2022, 7:11 PM
Herald added a reviewer: bhyve. · View Herald TranscriptJan 4 2022, 7:11 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript
markj requested review of this revision.Jan 4 2022, 7:11 PM
Harbormaster completed remote builds in B43700: Diff 100933.Jan 4 2022, 7:11 PM
markj mentioned this in D32359: bhyve: Map the MSI-X table unconditionally when passthrough is enabled.Jan 4 2022, 7:13 PM
markj updated this revision to Diff 100944.Jan 4 2022, 9:32 PM

Fix a second bug, also reported by Andy:

If the last page containing the table is not also the last page
of the BAR, then we would call mprotect(PROT_NONE) on the wrong
portion of the mapping.

Harbormaster completed remote builds in B43709: Diff 100944.Jan 4 2022, 9:32 PM
jhb accepted this revision.Jan 4 2022, 10:35 PM
This revision is now accepted and ready to land.Jan 4 2022, 10:35 PM
Closed by commit rG76b45e688ae8: bhyve: Map the right BAR in init_msix_table() (authored by markj). · Explain WhyJan 5 2022, 3:12 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG76b45e688ae8: bhyve: Map the right BAR in init_msix_table().
markj added a commit: rG4558c11f1b4d: bhyve: Correct unmapping of the MSI-X table BAR.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
2 lines

Diff 100974

View Options

usr.sbin/bhyve/pci_passthru.c

Loading...