Page MenuHomeFreeBSD
Differential D42186

pci: Set the domain in the pci_generic driver
ClosedPublic
Actions

Authored by andrew on Oct 13 2023, 10:24 AM.
Tags
None
Referenced Files
F154206147: D42186.diff
Mon, Apr 27, 2:46 AM
F154179290: D42186.id128725.diff
Sun, Apr 26, 8:58 PM
Unknown Object (File)
Sat, Apr 25, 9:19 PM
Unknown Object (File)
Sat, Apr 25, 5:04 PM
Unknown Object (File)
Sat, Apr 25, 2:45 AM
Unknown Object (File)
Fri, Apr 24, 8:52 PM
Unknown Object (File)
Thu, Apr 23, 1:15 AM
Unknown Object (File)
Wed, Apr 22, 7:24 PM
View All Files
Subscribers
markj

Details

Reviewers
jhb
imp
gallatin
markj
Commits
rG729576a3d540: pci: Set the domain in the pci_generic driver
rG7098f3c7569c: pci: Set the domain in the pci_generic driver
Summary

Tell bus_dma(9) which NUMA domain the PCI driver is closest to so it
can allocate memory from there when possible.

Sponsored by: Arm Ltd

Diff Detail

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

Event Timeline

andrew requested review of this revision.Oct 13 2023, 10:24 AM
andrew created this revision.
Harbormaster completed remote builds in B53957: Diff 128709.Oct 13 2023, 10:24 AM
andrew mentioned this in D42187: arm64: Teach bus_dma on arm64 about NUMA.Oct 13 2023, 3:01 PM
markj added a subscriber: markj.Oct 13 2023, 3:05 PM
markj added inline comments.
sys/dev/pci/pci_host_generic.c
108

If you're intentionally squashing errors, please add a comment and/or don't bother setting error.

andrew updated this revision to Diff 128725.Oct 13 2023, 3:19 PM

Stop setting error

Harbormaster completed remote builds in B53968: Diff 128725.Oct 13 2023, 3:19 PM
markj accepted this revision.Oct 13 2023, 3:20 PM
markj added inline comments.
sys/dev/pci/pci_host_generic.c
111
This revision is now accepted and ready to land.Oct 13 2023, 3:20 PM
andrew updated this revision to Diff 128740.Oct 13 2023, 4:24 PM

Add (void)

This revision now requires review to proceed.Oct 13 2023, 4:24 PM
Harbormaster completed remote builds in B53971: Diff 128740.Oct 13 2023, 4:24 PM
jhb accepted this revision.Oct 13 2023, 10:10 PM
jhb added inline comments.
sys/dev/pci/pci_host_generic.c
106

Nit: blank line before comment.

This revision is now accepted and ready to land.Oct 13 2023, 10:10 PM
Closed by commit rG7098f3c7569c: pci: Set the domain in the pci_generic driver (authored by andrew). · Explain WhyOct 16 2023, 10:48 AM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG7098f3c7569c: pci: Set the domain in the pci_generic driver.
andrew added a commit: rG729576a3d540: pci: Set the domain in the pci_generic driver.Feb 19 2024, 4:46 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
pci/
8 lines

Diff 128725

View Options

sys/dev/pci/pci_host_generic.c

Loading...