Page MenuHomeFreeBSD
Differential D34774

linuxkpi: Move pci_alloc_irq_vectors to .c file
ClosedPublic
Actions

Authored by imp on Apr 4 2022, 5:04 PM.
Tags
None
Referenced Files
F167268773: D34774.id104572.diff
Thu, Aug 20, 1:08 PM
Unknown Object (File)
Wed, Aug 19, 5:00 PM
Unknown Object (File)
Wed, Aug 19, 4:16 PM
Unknown Object (File)
Wed, Aug 19, 10:54 AM
Unknown Object (File)
Wed, Aug 19, 9:18 AM
Unknown Object (File)
Tue, Aug 18, 8:19 PM
Unknown Object (File)
Tue, Aug 18, 8:06 PM
Unknown Object (File)
Mon, Aug 17, 8:37 PM
View All Files
Subscribers
emaste
• hselasky
linuxkpi

Details

Reviewers
• hselasky
Commits
rGebd35d1ceb36: linuxkpi: Move pci_alloc_irq_vectors to .c file
rGbc01b383945c: linuxkpi: Move pci_alloc_irq_vectors to .c file
rG36b5c440028b: linuxkpi: Move pci_alloc_irq_vectors to .c file
Summary

pci_alloc_irq_vectors encodes the size of struct msix_entry
into its code. Move from .h to .c to move this knowledge from
client modules to linuxkpi module.

Sponsored by: Netflix

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44992
Build 41880: arc lint + arc unit

Event Timeline

imp created this revision.Apr 4 2022, 5:04 PM
Herald added subscribers: linuxkpi, emaste. · View Herald TranscriptApr 4 2022, 5:04 PM
imp requested review of this revision.Apr 4 2022, 5:04 PM
Harbormaster completed remote builds in B44992: Diff 104572.Apr 4 2022, 5:04 PM
imp added a parent revision: D34773: linuxkpi: Move pci_request_region and _lkpi_pci_iomap into .c.Apr 4 2022, 5:04 PM
imp added a child revision: D34775: linuxkpi: Move lkpi_pcim_iomap_devres_find to .c file.
• hselasky accepted this revision.Apr 4 2022, 5:21 PM
• hselasky added a subscriber: • hselasky.

Maybe prefix those functions with lkpi_xxx

This revision is now accepted and ready to land.Apr 4 2022, 5:21 PM
imp added a comment.Apr 4 2022, 8:08 PM
In D34774#788093, @hselasky wrote:

Maybe prefix those functions with lkpi_xxx

The current code is a bit inconsistent about this sort of thing. What's the rule we should enforce?

Closed by commit rG36b5c440028b: linuxkpi: Move pci_alloc_irq_vectors to .c file (authored by imp). · Explain WhyApr 5 2022, 5:10 AM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG36b5c440028b: linuxkpi: Move pci_alloc_irq_vectors to .c file.
imp added a commit: rGbc01b383945c: linuxkpi: Move pci_alloc_irq_vectors to .c file.Apr 5 2022, 2:43 PM
imp added a commit: rGebd35d1ceb36: linuxkpi: Move pci_alloc_irq_vectors to .c file.Apr 6 2022, 5:46 PM

Revision Contents
Changeset List

PathSize
sys/
compat/
linuxkpi/
common/
include/
linux/
38 lines
src/
36 lines

Diff 104572

View Options

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

Loading...
View Options

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

Loading...