Page MenuHomeFreeBSD

Use syscall_helper_register() to register syscalls and do it through the module interface rather than sysinit.
ClosedPublic

Authored by brooks on Feb 7 2018, 7:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 5:48 AM
Unknown Object (File)
Mon, Dec 2, 4:41 PM
Unknown Object (File)
Mon, Dec 2, 4:41 PM
Unknown Object (File)
Mon, Dec 2, 9:45 AM
Unknown Object (File)
Fri, Nov 29, 12:39 PM
Unknown Object (File)
Fri, Nov 29, 10:34 AM
Unknown Object (File)
Nov 15 2024, 6:25 AM
Unknown Object (File)
Oct 28 2024, 8:10 PM
Subscribers

Details

Summary

This pattern is more common and easier to understand.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Looks ok to me, although, since I know nothing about the interface,
I've taken myself off the review.

sys/nlm/nlm_prot_impl.c
2404 ↗(On Diff #39019)

I think you should actually do the nlm_uninit in MOD_UNLOAD case (modules get MOD_UNLOAD invoked if MOD_LOAD fails). It's fine to have that still fallthrough though.

  • Let MOD_UNLOAD handle the failure case.
This revision is now accepted and ready to land.Feb 9 2018, 5:55 PM
This revision was automatically updated to reflect the committed changes.