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
F153286918: D42186.id128740.diff
Mon, Apr 20, 6:30 AM
F153195868: D42186.diff
Sun, Apr 19, 5:59 PM
Unknown Object (File)
Wed, Apr 15, 12:25 PM
Unknown Object (File)
Tue, Apr 14, 10:06 AM
Unknown Object (File)
Tue, Apr 14, 3:51 AM
Unknown Object (File)
Fri, Apr 10, 2:15 AM
Unknown Object (File)
Mon, Apr 6, 3:39 AM
Unknown Object (File)
Sun, Apr 5, 12:18 AM
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 Not Applicable
Unit
Tests Not Applicable

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/
9 lines

Diff 128818

View Options

sys/dev/pci/pci_host_generic.c

Loading...