Skip to content

Commit

Permalink
nanny: reset instead to rearm worker on disable
Browse files Browse the repository at this point in the history
Reset interface worker on ifreload/ifdown to cleanup fsm state,
master / lower hierarchy references and requirements, so there
are no leftovers for the ifup part of ifreload that may change
e.g. lower link reference (vlan's underlying device changes)
with the new policy/config it is applying.
  • Loading branch information
mtomaschewski committed May 15, 2024
1 parent d3027bb commit c6eb69d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion nanny/interface.c
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ ni_managed_netdev_disable(ni_managed_device_t *mdev)

ni_nanny_schedule_recheck(&mgr->down, w);
ni_nanny_unschedule(&mgr->recheck, w);
ni_ifworker_rearm(w);
ni_ifworker_reset(w);

mdev->monitor = FALSE;
return TRUE;
Expand Down

0 comments on commit c6eb69d

Please sign in to comment.