diff options
| author | Petr Machata <[email protected]> | 2022-06-28 12:37:44 +0200 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2022-06-29 14:01:23 +0100 |
| commit | 04cfbc1d89d4cc73b5b328e3bacf24d43e9aa4b7 (patch) | |
| tree | f6146a6db4c69fb83c5be65fc0d5a992a26994b3 /include/linux | |
| parent | 9bacb93bcfb83ab0e939ce9185f8ba910fb6adb0 (diff) | |
selftests: forwarding: ethtool_extended_state: Convert to busywait
Currently, this script sets up the test scenario, which is supposed to end
in an inability of the system to negotiate a link. It then waits for a bit,
and verifies that the system can diagnose why the link was not established.
The wait time for the scenario where different link speeds are forced on
the two ends of a loopback cable, was set to 4 seconds, which exactly
covered it. As of a recent mlxsw firmware update, this time gets longer,
and this test starts failing.
The time that selftests currently wait for links to be established is
currently $WAIT_TIMEOUT, or 20 seconds. It seems reasonable that if this is
the time necessary to establish and bring up a link, it should also be
enough to determine that a link cannot be established and why.
Therefore in this patch, convert the sleeps to busywaits, so that if a
failure is established sooner (as is expected), the test runs quicker. And
use $WAIT_TIMEOUT as the time to wait.
Signed-off-by: Petr Machata <[email protected]>
Reviewed-by: Amit Cohen <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions