Paths

Table of Contentst

Differential D52068

LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions()
ClosedPublic
Actions

Authored by bz on Aug 21 2025, 8:59 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 6, 4:42 AM

	Unknown Object (File)
	Fri, Sep 4, 6:39 PM

	Unknown Object (File)
	Wed, Sep 2, 3:18 PM

	Unknown Object (File)
	Wed, Sep 2, 3:50 AM

	Unknown Object (File)
	Sun, Aug 30, 9:34 PM

	Unknown Object (File)
	Mon, Aug 24, 6:23 PM

	Unknown Object (File)
	Sun, Aug 23, 6:53 PM

	Unknown Object (File)
	Sun, Aug 23, 4:21 AM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rGc017775775ee: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions()
rGff31767e530a: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions()
rG7e21158d44cd: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions()

Summary

Factor out the pci_request_region() implementation into an internal
function and make pci_request_region() a simple wrapperaround it.
Likewise implement pcim_request_all_regions() as a loop calling
pci_request_region() for each entry.

In two cases which we returned an error before (bar index is valid but
bar is not (no len), and neither IO nor MEM) we now reutrn success
(nothing to do for us). Otherwise callers, especially
pcim_request_all_regions() would error out for the wrong reasons.
This seems to also match the expected behaviour of pci_request_region().

Sponsored by: The FreeBSD Foundation
MFC after: 3 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bz created this revision.Aug 21 2025, 8:59 AM

Herald added subscribers: emaste, imp. · View Herald TranscriptAug 21 2025, 8:59 AM

bz requested review of this revision.Aug 21 2025, 8:59 AM

Harbormaster completed remote builds in B66406: Diff 160722.Aug 21 2025, 8:59 AM

dumbbell requested changes to this revision.Sep 3 2025, 8:31 PM

dumbbell added a subscriber: dumbbell.

dumbbell added inline comments.

sys/compat/linuxkpi/common/src/linux_pci.c
1199	Typo in "automatiaclly".
1202	Typo in "rack"?
1224	Should you take this opportunity to add a `linuxkpi_` prefix to the function name?

This revision now requires changes to proceed.Sep 3 2025, 8:31 PM

bz marked 3 inline comments as done.Sep 3 2025, 10:11 PM

Fix typos
Add linuxkpI_ prefix to function

Harbormaster completed remote builds in B66802: Diff 161468.Sep 3 2025, 10:12 PM

I spotted another typo (see the attached comment). I approve it at the same time to not waste time in back and forth comments.

sys/compat/linuxkpi/common/src/linux_pci.c
1199	Oops, another typo I notice just now…

This revision is now accepted and ready to land.Sep 4 2025, 4:15 PM

bz marked an inline comment as done.Sep 4 2025, 8:05 PM

bz added inline comments.

sys/compat/linuxkpi/common/src/linux_pci.c
1199	Yepp and that was there before. Fixes as well. Thank you!

Juts pushing it, as it is easier for the final build to have them all togetehr

This revision now requires review to proceed.Sep 4 2025, 8:08 PM

Harbormaster completed remote builds in B66824: Diff 161554.Sep 4 2025, 8:08 PM

This revision was not accepted when it landed; it landed in state Needs Review.Sep 5 2025, 11:29 PM

Closed by commit rG7e21158d44cd: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions() (authored by bz). · Explain Why

This revision was automatically updated to reflect the committed changes.

bz added a commit: rG7e21158d44cd: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions().

bz added a commit: rGff31767e530a: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions().Sep 15 2025, 7:13 PM

bz added a commit: rGc017775775ee: LinuxKPI: pci: implement [linuxkpi_]pcim_request_all_regions().Jan 16 2026, 7:43 PM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

11 lines

src/

55 lines

Diff 161630

sys/compat/linuxkpi/common/include/linux/pci.h

Loading...

sys/compat/linuxkpi/common/src/linux_pci.c

Loading...