interface: do not release device on link down
authorErik Karlsson <erik.karlsson@genexis.eu>
Tue, 19 Sep 2023 11:37:59 +0000 (13:37 +0200)
committerFelix Fietkau <nbd@nbd.name>
Mon, 25 Sep 2023 12:00:56 +0000 (14:00 +0200)
commiteb69d3f1c45d5b955cd10c6dcb28fee3d7a60d8e
tree474536f9ee2b366863c9bdb24e2c20f54dae2da6
parent7a58b995fdbecd9beed57e4d66d42cb3cf66aee2
interface: do not release device on link down

On IFPEV_DOWN, do not release the main device if the interface is
intended to be brought back up again by interface_handle_config_change
and there is no configuration change. This avoids releasing the device
and then claiming it again when link is lost.

Releasing and then claiming the device can, if the driver is
configured to power the PHY down when IFF_UP is cleared, cause the PHY
to be power cycled. This can in combination with glitchy hardware
and/or drivers where it takes a few seconds for the link to stabilize
after the PHY has been powered up cause a loop where loss of link
causes netifd to power cycle the PHY and when the PHY comes back up
the link glitches and the cycle repeats.

Signed-off-by: Erik Karlsson <erik.karlsson@genexis.eu>
interface.c