Page MenuHomeFreeBSD

net: add if_allocdescr() to permit updating interface description from the kernel
ClosedPublic

Authored by melifaro on Nov 30 2022, 1:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 3:41 AM
Unknown Object (File)
Oct 5 2024, 5:03 AM
Unknown Object (File)
Oct 5 2024, 4:27 AM
Unknown Object (File)
Oct 4 2024, 10:56 PM
Unknown Object (File)
Oct 3 2024, 12:09 PM
Unknown Object (File)
Oct 2 2024, 8:12 AM
Unknown Object (File)
Oct 2 2024, 4:46 AM
Unknown Object (File)
Oct 2 2024, 1:11 AM

Diff Detail

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

Event Timeline

melifaro added a reviewer: network.
melifaro retitled this revision from net: add if_allocdescr() to permit updating description from the kernel to net: add if_allocdescr() to permit updating interface description from the kernel.

Does this abstract enough to be worth it? Where is it intended to be used?

sys/net/if.c
4276

I think our usual pattern for this sort of thing is to pass int flags, and do malloc(size, type, M_ZERO | flags).

4278

Also, I'm sure you meant (waitok ? M_WAITOK : 0) here.

In D37566#853879, @kp wrote:

Does this abstract enough to be worth it? Where is it intended to be used?

I have a larger diff that allows netlink to update interface properties, such as description. Given we already have if_freedescr(), I moved towards the if_allocdescr() instead of exposing M_IFDESCR.

This revision is now accepted and ready to land.Nov 30 2022, 3:42 PM
zlei added a subscriber: zlei.

Looks good to me.