Page MenuHomeFreeBSD

allow late binding of linux_alloc_current
ClosedPublic

Authored by mmacy on Feb 19 2019, 10:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 27 2023, 1:15 PM
Unknown Object (File)
Dec 27 2023, 1:07 PM
Unknown Object (File)
Dec 27 2023, 1:07 PM
Unknown Object (File)
Dec 27 2023, 1:07 PM
Unknown Object (File)
Dec 24 2023, 11:47 PM
Unknown Object (File)
Dec 20 2023, 3:16 AM
Unknown Object (File)
Dec 18 2023, 3:24 AM
Unknown Object (File)
Dec 3 2023, 2:29 AM
Subscribers

Details

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

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?

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.

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
This revision was automatically updated to reflect the committed changes.