diff options
author | Lubomir Rintel <lkundrak@v3.sk> | 2022-08-25 15:52:27 +0200 |
---|---|---|
committer | Lubomir Rintel <lkundrak@v3.sk> | 2022-08-25 15:52:27 +0200 |
commit | 521848915b1c19efe63281e511a957192e2f9a3d (patch) | |
tree | 1f889a2d5687f38e39d3c432106e4c74cce725e7 | |
parent | b58fc7088b2233caea373f6485a5d85dfad3e16e (diff) | |
download | NetworkManager-lr/carrier-wait-assumed.tar.gz |
device: wait for carrier on unavailable device even when it gets a connection assumedlr/carrier-wait-assumed
The test in question leaves the device with a master set, which caused a
connection to get assumed and therefore the previous fix didn't kick in.
Fixes-test: @restart_L2_only_lacp
Fixes: 5b7f8f3f7051 ('device: wait for carrier even if it wasn't us who brought the device IFF_UP')
-rw-r--r-- | src/core/devices/nm-device.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/core/devices/nm-device.c b/src/core/devices/nm-device.c index c51da9ac82..5b551d224b 100644 --- a/src/core/devices/nm-device.c +++ b/src/core/devices/nm-device.c @@ -15778,6 +15778,13 @@ _set_state_full(NMDevice *self, NMDeviceState state, NMDeviceStateReason reason, if (priv->sys_iface_state == NM_DEVICE_SYS_IFACE_STATE_MANAGED) ip6_managed_setup(self); device_init_static_sriov_num_vfs(self); + + /* We didn't bring the device up and we have little idea + * when was it brought up. Play it safe and assume it could + * have been brought up very recently and it might one of + * those who take time to detect carrier. + */ + carrier_detect_wait(self); } if (priv->sys_iface_state == NM_DEVICE_SYS_IFACE_STATE_MANAGED) { @@ -15785,13 +15792,6 @@ _set_state_full(NMDevice *self, NMDeviceState state, NMDeviceStateReason reason, if (!nm_device_bring_up(self, TRUE, &no_firmware) && no_firmware) _LOGW(LOGD_PLATFORM, "firmware may be missing."); nm_device_set_firmware_missing(self, no_firmware ? TRUE : FALSE); - } else { - /* We didn't bring the device up and we have little idea - * when was it brought up. Play it safe and assume it could - * have been brought up very recently and it might one of - * those who take time to detect carrier. - */ - carrier_detect_wait(self); } /* Ensure the device gets deactivated in response to stuff like |