Page MenuHomeFreeBSD

netlink: convert to IfAPI.

Authored by melifaro on Jun 16 2023, 2:02 PM.
Referenced Files
Unknown Object (File)
Thu, Jul 11, 7:08 PM
Unknown Object (File)
Thu, Jun 27, 3:08 PM
Unknown Object (File)
Fri, Jun 21, 5:56 AM
Unknown Object (File)
Thu, Jun 20, 5:10 AM
Unknown Object (File)
Sat, Jun 15, 11:25 PM
Unknown Object (File)
Jun 3 2024, 3:48 AM
Unknown Object (File)
May 29 2024, 3:59 PM
Unknown Object (File)
May 25 2024, 2:01 PM



Convert to IfAPI everything except IF_AFDATA_WLOCK usage in neigh.c.

I don't think it's a good idea to to expose locking as function-based API. I'd prefer to leave neigh.c with access to a private header for some time.
I two options here: 1) move actual LLE insertion code inside if_lltable.c

  1. refactoring IF_AFDATA locking in general to have per-af lock instead of a global one.

Currently I'm leaning for the first option, which is fairly simple.

Diff Detail

rG FreeBSD src repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

I agree with option 1, moving the LLE insertion code to if_llatbl.c


I'm fine with including the private header for now, let's make sure "for now" doesn't become "forever", though :)

This revision is now accepted and ready to land.Jun 16 2023, 3:36 PM

I agree with option 1, moving the LLE insertion code to if_llatbl.c

Yep. I drafted something, but I need to think about the desired KPI a bit more, as I want to accommodate child LLE inclusion (IPv4 over IPv6 LLEs ATM). Probably I'll come up with something in the next couple of days.


Sure. I'd really love to reduce the if_private.h inclusion to the bare minimum everywhere.

This revision was automatically updated to reflect the committed changes.