PCI DEN0115: Reliably check for a memory resource during probe.
ClosedPublic
Actions

Authored by jhb on Jul 13 2023, 8:34 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	May 17 2024, 12:41 AM

	Unknown Object (File)
	May 8 2024, 7:19 AM

	Unknown Object (File)
	May 2 2024, 12:48 PM

	Unknown Object (File)
	May 2 2024, 12:48 PM

	Unknown Object (File)
	May 2 2024, 11:08 AM

	Unknown Object (File)
	May 2 2024, 9:43 AM

	Unknown Object (File)
	Feb 19 2024, 6:50 AM

	Unknown Object (File)
	Dec 20 2023, 8:07 AM

View All 12 Files

Subscribers

imp

Details

Reviewers

andrew
jrtc27

Commits

rG6582301f83c6: PCI DEN0115: Reliably check for a memory resource during probe.

Summary

rid was stack garbage, so the bus_alloc_resource_any() call could fail
and fall through to the SMCCC version check even if a bridge had a
memory resource.

Debugging help: jrtc27
Sponsored by: DARPA

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Jul 13 2023, 8:34 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 13 2023, 8:34 PM

jhb requested review of this revision.Jul 13 2023, 8:34 PM

Harbormaster completed remote builds in B52627: Diff 124628.Jul 13 2023, 8:34 PM

jrtc27 accepted this revision.Jul 13 2023, 8:45 PM

This revision is now accepted and ready to land.Jul 13 2023, 8:45 PM

Fixes: c94e4d91da37 ("Add a PCI driver that follows the Arm DEN0115 spec")?

Closed by commit rG6582301f83c6: PCI DEN0115: Reliably check for a memory resource during probe. (authored by jhb). · Explain WhyJul 14 2023, 5:32 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG6582301f83c6: PCI DEN0115: Reliably check for a memory resource during probe..

Revision Contents
Changeset List

Path

Size

sys/

dev/

pci/

pci_host_generic_den0115.c

3 lines

Diff 124685

View Options

sys/dev/pci/pci_host_generic_den0115.c

PCI DEN0115: Reliably check for a memory resource during probe.ClosedPublicActions