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

	F136729583: D19257.id54237.diff
	Wed, Nov 19, 2:47 AM

	F136662917: D19257.id54237.diff
	Tue, Nov 18, 7:23 PM

	F136613140: D19257.id54151.diff
	Tue, Nov 18, 2:19 PM

	Unknown Object (File)
	Tue, Nov 18, 9:29 AM

	Unknown Object (File)
	Thu, Nov 6, 10:37 PM

	Unknown Object (File)
	Mon, Nov 3, 5:00 AM

	Unknown Object (File)
	Thu, Oct 30, 5:20 PM

	Unknown Object (File)
	Sun, Oct 26, 3:46 AM

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 Skipped

Unit

Tests Skipped

Build Status

Buildable 22626

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
224	The alloc_current pointer should be set before registering the eventhandler.
245–246	the alloc_current pointer should be cleared after deregistering the eventhandler.
sys/kern/init_main.c
109	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	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	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

sys/

compat/

linuxkpi/

common/

include/

linux/

4 lines

src/

linux_current.c

4 lines

kern/

11 lines

sys/

7 lines

Diff 54151

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

Loading...

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

Loading...

sys/kern/init_main.c

Loading...

sys/sys/systm.h

Loading...