Page MenuHomeFreeBSD

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

Authored by glebius on Wed, Apr 24, 6:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 3:50 PM
Unknown Object (File)
Sun, Apr 28, 10:46 AM
Unknown Object (File)
Sat, Apr 27, 6:34 PM
Unknown Object (File)
Fri, Apr 26, 4:42 PM
Unknown Object (File)
Fri, Apr 26, 2:53 PM
Unknown Object (File)
Fri, Apr 26, 9:17 AM
Unknown Object (File)
Fri, Apr 26, 9:17 AM
Unknown Object (File)
Fri, Apr 26, 9:12 AM
Subscribers

Details

Reviewers
stevek
tuexen
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 57379
Build 54267: 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.Thu, Apr 25, 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.Wed, May 1, 8:33 PM