Page MenuHomeFreeBSD

lo: Use new KPI to create the first loop interface
ClosedPublic

Authored by zlei on Jun 25 2024, 10:00 AM.
Tags
None
Referenced Files
F102087931: D45728.diff
Thu, Nov 7, 11:28 AM
Unknown Object (File)
Oct 1 2024, 8:17 PM
Unknown Object (File)
Sep 27 2024, 7:57 PM
Unknown Object (File)
Sep 24 2024, 5:37 PM
Unknown Object (File)
Sep 19 2024, 3:49 AM
Unknown Object (File)
Sep 18 2024, 5:09 PM
Unknown Object (File)
Sep 18 2024, 6:55 AM
Unknown Object (File)
Sep 18 2024, 2:39 AM
Subscribers

Details

Summary

While here remove a pointless assignment to static local variable lo_cloner.

No functional change intended.

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

zlei requested review of this revision.Jun 25 2024, 10:00 AM
zlei added inline comments.
sys/net/if_loop.c
126

if_alloc() never fail since 4787572d0580 .

kp added a subscriber: kp.

I was initially worried that we'd change the behaviour of V_loif here, in that it'd get updated every time we create a new loopback interface, but I was wrong. We only set V_loif (although now via ifc_create_ifp() indirectly, rather that from lo_clone_create()) for the first, automatically created, interface.

This revision is now accepted and ready to land.Jun 25 2024, 12:21 PM
In D45728#1042970, @kp wrote:

I was initially worried that we'd change the behaviour of V_loif here, in that it'd get updated every time we create a new loopback interface, but I was wrong. We only set V_loif (although now via ifc_create_ifp() indirectly, rather that from lo_clone_create()) for the first, automatically created, interface.

lo: Use new KPI to create the first loop interface

Emm, I think the word assign is more accurate than create.