]> 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)
commit0f285df0330c93d94a7b4dfbc74fc78ac7efa011
tree26cce1f58ce9794662d466125297f320fe596ca5
parentd7aa3cb1360af781d62a451c95ad8cf43b5c0605
mlxsw: spectrum_router: Remove abort mechanism

The abort mechanism was introduced in commit 59932b6aaf99 ("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