Page MenuHomeFreeBSD

sockets: hide socket hhook(9)s under SOCKET_HHOOK
ClosedPublic

Authored by glebius on Apr 24 2024, 6:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 10:18 PM
Unknown Object (File)
Sat, Oct 19, 1:34 AM
Unknown Object (File)
Oct 3 2024, 8:11 PM
Unknown Object (File)
Sep 12 2024, 5:29 AM
Unknown Object (File)
Sep 9 2024, 12:08 AM
Unknown Object (File)
Sep 8 2024, 12:10 AM
Unknown Object (File)
Sep 7 2024, 10:34 PM
Unknown Object (File)
Sep 7 2024, 8:15 AM
Subscribers

Details

Summary

There are no in-tree consumers of these hooks.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Or, this can go under SOCKET_HHOOK new option. Cause these two hhooks aren't really related. What do you think?

Or, this can go under SOCKET_HHOOK new option. Cause these two hhooks aren't really related. What do you think?

I think SOCKET_HHOOK would be better, yes.

Or, this can go under SOCKET_HHOOK new option. Cause these two hhooks aren't really related. What do you think?

I think SOCKET_HHOOK would be better, yes.

So any reason you use TCP_HHOOK instead of SOCKET_HHOOK? I would prefer SOCKET_HHOOK.

glebius retitled this revision from sockets: hide socket hhook(9)s under TCP_HHOOK to sockets: hide socket hhook(9)s under SOCKET_HHOOK.Apr 25 2024, 3:29 AM

SOCKET_HHOOK instead of TCP_HHOOK

sys/kern/uipc_socket.c
438

When this comment was introduced, it applied to the if statement above and was put before the if statement. See 4ec737123326. So are you moving it down intentionally?

sys/kern/uipc_socket.c
438

Indeed. I assumed the comment says that so_gencnt and numopensockets can be made atomics. I will put the comment back.

This revision is now accepted and ready to land.May 1 2024, 8:33 PM