Page MenuHomeFreeBSD
Differential D31673

LinuxKPI: Remove FreeBSD struct resource from all LKPI headers
ClosedPublic
Actions

Authored by wulf on Aug 24 2021, 11:52 PM.
Tags
None
Referenced Files
F167040081: D31673.id94152.diff
Tue, Aug 18, 5:33 PM
F167038526: D31673.diff
Tue, Aug 18, 5:10 PM
Unknown Object (File)
Tue, Aug 11, 3:16 PM
Unknown Object (File)
Tue, Aug 11, 8:16 AM
Unknown Object (File)
Mon, Aug 10, 12:02 PM
Unknown Object (File)
Mon, Aug 10, 6:14 AM
Unknown Object (File)
Fri, Aug 7, 3:46 PM
Unknown Object (File)
Mon, Jul 27, 4:13 PM
View All Files
Subscribers
emaste
imp
linuxkpi

Details

Reviewers
• hselasky
manu
bz
Commits
rGfd6adf784e86: LinuxKPI: Remove FreeBSD struct resource from all LKPI headers
rG66ea390652d2: LinuxKPI: Remove FreeBSD struct resource from all LKPI headers
Summary

Remove FreeBSD struct resource from all LKPI headers
except linux/pci.h to avoid conflicts with Linux version.
This allows to #define resource in drm-kmod globally and strip a lot of #ifdef-s

MFC after: 2 weeks

Diff Detail

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

Event Timeline

wulf created this revision.Aug 24 2021, 11:52 PM
Herald added subscribers: linuxkpi, emaste, imp. · View Herald TranscriptAug 24 2021, 11:52 PM
wulf requested review of this revision.Aug 24 2021, 11:52 PM
Harbormaster completed remote builds in B41194: Diff 94152.Aug 24 2021, 11:52 PM
• hselasky added a comment.Aug 25 2021, 7:46 AM

Where did the tasklet support go?

sys/compat/linuxkpi/common/include/linux/interrupt.h
107

"device" is no longer __unused here.

wulf updated this revision to Diff 94160.Aug 25 2021, 9:07 AM

Remove extraneous __unused tag

Harbormaster completed remote builds in B41197: Diff 94160.Aug 25 2021, 9:07 AM
wulf added a comment.Aug 25 2021, 9:15 AM
In D31673#714450, @hselasky wrote:

Where did the tasklet support go?

They are left untouched. Nothing happened.

Probably you are confused by using of sys/compat/linuxkpi/common/include/linux/interrupt.h as a base of two diffs.

manu accepted this revision.Aug 25 2021, 9:22 AM
This revision is now accepted and ready to land.Aug 25 2021, 9:22 AM
• hselasky accepted this revision.Aug 25 2021, 9:34 AM

Make sure you build the kernel WITH_OFED=YES

wulf mentioned this in D32168: LinuxKPI: Hide some symbols in linux_interrupt.c.Sep 27 2021, 11:51 PM
bz added a child revision: D32168: LinuxKPI: Hide some symbols in linux_interrupt.c.Sep 28 2021, 7:09 AM
Closed by commit rG66ea390652d2: LinuxKPI: Remove FreeBSD struct resource from all LKPI headers (authored by wulf). · Explain WhySep 29 2021, 8:28 PM
This revision was automatically updated to reflect the committed changes.
wulf added a commit: rG66ea390652d2: LinuxKPI: Remove FreeBSD struct resource from all LKPI headers.
wulf added a commit: rGfd6adf784e86: LinuxKPI: Remove FreeBSD struct resource from all LKPI headers.Oct 13 2021, 9:20 AM

Revision Contents
Changeset List

PathSize
sys/
compat/
linuxkpi/
common/
include/
linux/
168 lines
src/
41 lines
src/
linux_interrupt.c/
include/
linux/
interrupt.h
167 lines
conf/
2 lines
modules/
linuxkpi/
1 line

Diff 94160

View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

sys/conf/files

Loading...
View Options

sys/modules/linuxkpi/Makefile

Loading...