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
F98961495: D14251.id39057.diff
Sat, Oct 5, 12:39 PM
Unknown Object (File)
Wed, Oct 2, 1:14 AM
Unknown Object (File)
Wed, Oct 2, 12:58 AM
Unknown Object (File)
Tue, Oct 1, 10:48 PM
Unknown Object (File)
Mon, Sep 30, 12:51 PM
Unknown Object (File)
Thu, Sep 12, 4:08 AM
Unknown Object (File)
Sun, Sep 8, 9:34 PM
Unknown Object (File)
Sun, Sep 8, 6:31 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.