Differential D19257

allow late binding of linux_alloc_current
ClosedPublic
Actions

Authored by mmacy on Feb 19 2019, 10:47 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 11, 6:20 PM

	Unknown Object (File)
	Mar 10 2025, 8:21 PM

	Unknown Object (File)
	Feb 16 2025, 10:51 AM

	Unknown Object (File)
	Jan 20 2025, 11:28 PM

	Unknown Object (File)
	Jan 17 2025, 9:01 PM

	Unknown Object (File)
	Jan 17 2025, 11:47 AM

	Unknown Object (File)
	Dec 29 2024, 7:25 AM

	Unknown Object (File)
	Oct 4 2024, 5:10 AM

View All 88 Files

Subscribers

Details

Reviewers

Commits

rS344478: lkpi: allow late binding of linux_alloc_current

Summary

lindebugfs may be used independently of the lkpi. This allows it to call linux_alloc_current without having a static dependency.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mmacy created this revision.Feb 19 2019, 10:47 PM

Herald added subscribers: emaste, imp. · View Herald TranscriptFeb 19 2019, 10:47 PM

Harbormaster completed remote builds in B22588: Diff 54098.Feb 19 2019, 10:47 PM

mmacy added a child revision: D19258: import linux debugfs support.Feb 19 2019, 10:55 PM

• hselasky added inline comments.Feb 20 2019, 7:59 AM

sys/compat/linuxkpi/common/src/linux_current.c
223 ↗	(On Diff #54098)	The alloc_current pointer should be set before registering the eventhandler.
246 ↗	(On Diff #54098)	the alloc_current pointer should be cleared after deregistering the eventhandler.
sys/kern/init_main.c
109 ↗	(On Diff #54098)	Is the header file for this prototype included?

• hselasky added inline comments.Feb 20 2019, 8:02 AM

sys/compat/linuxkpi/common/include/linux/compat.h
49 ↗	(On Diff #54098)	For sake of optimisation I want you to always set the lkpi_alloc_current pointer. I.E. In main make a lkpi_default_alloc_current() function, which is used to initialize the pointer and does nothing.

incorporate feedback

Harbormaster completed remote builds in B22626: Diff 54151.Feb 20 2019, 9:10 PM

Looks good.

sys/kern/init_main.c
113 ↗	(On Diff #54151)	I wonder if it is more correct that the noop function returns EINVAL, or something like that.

This revision is now accepted and ready to land.Feb 22 2019, 9:43 PM

Closed by commit rS344478: lkpi: allow late binding of linux_alloc_current (authored by mmacy). · Explain WhyFeb 22 2019, 11:15 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

compat/

linuxkpi/

common/

include/

linux/

5 lines

src/

linux_current.c

3 lines

kern/

10 lines

sys/

7 lines

Diff 54237

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

Loading...

head/sys/compat/linuxkpi/common/src/linux_current.c

Loading...

head/sys/kern/init_main.c

Loading...

head/sys/sys/systm.h

Loading...