]> git.baikalelectronics.ru Git - kernel.git/commit
mlxsw: spectrum_router: Remove abort mechanism
authorAmit Cohen <amcohen@nvidia.com>
Tue, 8 Jun 2021 12:44:07 +0000 (15:44 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 8 Jun 2021 21:39:06 +0000 (14:39 -0700)
commitfa387e99699eccd07422b4c298b868ac9c09f5fe
tree26cce1f58ce9794662d466125297f320fe596ca5
parent4281520d11171031df230daf255ce6f725f82b5a
mlxsw: spectrum_router: Remove abort mechanism

The abort mechanism was introduced in commit 9bf1751d4d8e ("fib: hook
IPv4 fib for hardware offload") with the purpose of falling back to
software-based routing in case of a route programming error in hardware.
The process is irreversible and requires users to reload the offloading
driver or reboot the machine.

While this approach might make sense in theory, it makes very little
sense in practice. In the case of high speed ASICs such as the Spectrum
ASIC, the abort mechanism effectively kills the machine upon a non-fatal
error such as a route programming error.

Such an extreme policy does not belong in the kernel, especially when
user space can simply try to reprogram the route following the
RTM_NEWROUTE failure notification.

Therefore, remove the abort mechanism.

Signed-off-by: Amit Cohen <amcohen@nvidia.com>
Reviewed-by: Petr Machata <petrm@nvidia.com>
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c
drivers/net/ethernet/mellanox/mlxsw/spectrum_router.h