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
F106166741: D44928.diff
Thu, Dec 26, 11:56 AM
Unknown Object (File)
Tue, Dec 24, 4:40 PM
Unknown Object (File)
Fri, Dec 20, 5:32 AM
Unknown Object (File)
Nov 6 2024, 10:18 PM
Unknown Object (File)
Oct 19 2024, 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
Subscribers

Details

Summary

There are no in-tree consumers of these hooks.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 57339
Build 54227: arc lint + arc unit

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
437

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
437

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