Page MenuHomeFreeBSD
Differential D56846

bhyve: enlarge 64-bit PCI MMIO hole to 1 TiB
Needs ReviewPublic
Actions

Authored by tsoome on Wed, May 6, 10:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 20, 2:01 AM
Unknown Object (File)
Sun, May 17, 10:15 AM
Unknown Object (File)
Sat, May 16, 9:49 PM
Unknown Object (File)
Fri, May 15, 9:12 AM
Unknown Object (File)
Thu, May 14, 5:34 PM
Unknown Object (File)
Thu, May 14, 5:34 PM
Unknown Object (File)
Tue, May 12, 2:19 AM
Unknown Object (File)
Mon, May 11, 7:10 PM
View All 18 Files
Subscribers
bcran
bnovkov
imp
rgrimes

Details

Reviewers
None
Group Reviewers
bhyve
Summary

Default 32 GiB hole is insufficient for modern GPUs whose BAR1 (HBM
aperture) is 128 GiB each. Four H100 SXM5 GPUs need ~512 GiB of
prefetchable 64-bit PCI MMIO. Bump to 1 TiB for headroom, covering
future hardware like H200 (192 GiB/GPU).

Assisted-By: Claude Opus 4.7 (1M context)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 72843
Build 69726: arc lint + arc unit

Event Timeline

tsoome created this revision.Wed, May 6, 10:23 AM
Herald added a reviewer: bhyve. · View Herald TranscriptWed, May 6, 10:23 AM
Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript
tsoome requested review of this revision.Wed, May 6, 10:23 AM
Harbormaster completed remote builds in B72843: Diff 177293.Wed, May 6, 10:23 AM
bnovkov added a subscriber: bnovkov.Wed, May 6, 2:16 PM

Was this change motivated by this comment in PR290920 and have you tested this patch somehow yet?
If not, I think that commenter would be more than willing to test it.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
8 lines

Diff 177293

View Options

usr.sbin/bhyve/pci_emul.c

Loading...