Page MenuHomeFreeBSD

sockets: Add hhook in sonewconn for inheriting OSD specific data
ClosedPublic

Authored by stevek on Apr 4 2024, 12:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 3 2024, 6:10 PM
Unknown Object (File)
Oct 2 2024, 6:16 PM
Unknown Object (File)
Oct 2 2024, 9:24 AM
Unknown Object (File)
Oct 2 2024, 12:58 AM
Unknown Object (File)
Oct 1 2024, 11:58 PM
Unknown Object (File)
Oct 1 2024, 11:21 PM
Unknown Object (File)
Sep 30 2024, 2:04 PM
Unknown Object (File)
Sep 15 2024, 2:35 PM
Subscribers

Details

Summary

Added HHOOK_SOCKET_NEWCONN and bumped HHOOK_SOCKET_LAST

Obtained from: Juniper Networks, Inc.

Diff Detail

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

Event Timeline

I was about to ask "where is #ifdef TCP_HHOOKS?" But then I realized that socket_hhh isn't hidden under any ifdef.

Unrelated to this review, but may I ask if it is possible to hide socket_hhh and other stuff that came in 4ec7371233268 under TCP_HHOOKS? Or maybe SOCKET_HHOOKS, if these two are used as two independent features.

This revision is now accepted and ready to land.Apr 4 2024, 3:47 AM

I was about to ask "where is #ifdef TCP_HHOOKS?" But then I realized that socket_hhh isn't hidden under any ifdef.

Unrelated to this review, but may I ask if it is possible to hide socket_hhh and other stuff that came in 4ec7371233268 under TCP_HHOOKS? Or maybe SOCKET_HHOOKS, if these two are used as two independent features.

I like the idea to put it under an #ifdef.

I was about to ask "where is #ifdef TCP_HHOOKS?" But then I realized that socket_hhh isn't hidden under any ifdef.

Unrelated to this review, but may I ask if it is possible to hide socket_hhh and other stuff that came in 4ec7371233268 under TCP_HHOOKS? Or maybe SOCKET_HHOOKS, if these two are used as two independent features.

I think it should be possibly. I'll look at doing the changes to add option handling to the code.