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
F153103096: D34774.id.diff
Sun, Apr 19, 4:13 AM
Unknown Object (File)
Sat, Apr 18, 12:30 PM
Unknown Object (File)
Sat, Apr 18, 12:10 PM
Unknown Object (File)
Fri, Apr 17, 7:57 AM
Unknown Object (File)
Wed, Apr 15, 8:48 PM
Unknown Object (File)
Wed, Apr 15, 1:32 PM
Unknown Object (File)
Sat, Mar 28, 9:41 AM
Unknown Object (File)
Sat, Mar 28, 5:17 AM
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...