Re: [patch net-next 03/10] ipv4: fib_rules: Dump FIB rules when registering FIB notifier
From: David Ahern <hidden>
Date: 2017-03-13 14:52:48
On 3/13/17 1:38 AM, Jiri Pirko wrote:
quoted hunk ↗ jump to hunk
From: Ido Schimmel <redacted> In commit c3852ef7f2f8 ("ipv4: fib: Replay events when registering FIB notifier") we dumped the FIB tables and replayed the events to the passed notification block. However, we merely sent a RULE_ADD notification in case custom rules were in use. As explained in previous patches, this approach won't work anymore. Instead, we should notify the caller about all the FIB rules and let it act accordingly. Upon registration to the FIB notification chain, replay a RULE_ADD notification for each programmed FIB rule, custom or not. The integrity of the dump is ensured by the mechanism introduced in the above mentioned commit. Prevent regressions by making sure current listeners correctly sanitize the notified rules. Signed-off-by: Ido Schimmel <redacted> Signed-off-by: Jiri Pirko <redacted> --- drivers/net/ethernet/mellanox/mlxsw/Kconfig | 1 + drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c | 11 ++++++++++- drivers/net/ethernet/rocker/Kconfig | 2 +- drivers/net/ethernet/rocker/rocker_main.c | 15 +++++++++++++-- net/ipv4/fib_rules.c | 19 ++++++++++++++++--- 5 files changed, 41 insertions(+), 7 deletions(-)diff --git a/drivers/net/ethernet/mellanox/mlxsw/Kconfig b/drivers/net/ethernet/mellanox/mlxsw/Kconfig index ef23eae..a549a89 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/Kconfig +++ b/drivers/net/ethernet/mellanox/mlxsw/Kconfig@@ -73,6 +73,7 @@ config MLXSW_SWITCHX2 config MLXSW_SPECTRUM tristate "Mellanox Technologies Spectrum support" depends on MLXSW_CORE && MLXSW_PCI && NET_SWITCHDEV && VLAN_8021Q + depends on IP_MULTIPLE_TABLES depends on PSAMPLE || PSAMPLE=n select PARMAN default m
Why require multiple tables just b/c the h/w has the ability to support multiple tables?