mlxsw: spectrum_router: Make IPMR-related APIs family agnostic
authorYuval Mintz <yuvalm@mellanox.com>
Mon, 26 Mar 2018 12:01:42 +0000 (15:01 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Mar 2018 17:14:44 +0000 (13:14 -0400)
spectrum_router and spectrum_mr have several APIs that are used to
manipulate configurations originating from ipmr fib notifications.
Following previous patches all the protocol-specifics that are necessary
for the configuration are hidden within spectrum_mr. This allows us to
clean the API and make sure that other than choosing the mr_table based
on the fib notification family, spectrum_router wouldn't care about the
source of the notification when passing it onward to spectrum_mr.

This would later allow us to leverage the same code for fib
notifications originating from ip6mr.

Signed-off-by: Yuval Mintz <yuvalm@mellanox.com>
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/spectrum_mr.c
drivers/net/ethernet/mellanox/mlxsw/spectrum_mr.h
drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c

index ae8c5b5387db8c0cc163a7732516de1ce688bd16..ba6bcbe6f75bf184ee8290a8574f8722a13b02f7 100644 (file)
@@ -323,8 +323,8 @@ static void mlxsw_sp_mr_route_erase(struct mlxsw_sp_mr_table *mr_table,
 }
 
 static struct mlxsw_sp_mr_route *
-mlxsw_sp_mr_route4_create(struct mlxsw_sp_mr_table *mr_table,
-                         struct mfc_cache *mfc)
+mlxsw_sp_mr_route_create(struct mlxsw_sp_mr_table *mr_table,
+                        struct mr_mfc *mfc)
 {
        struct mlxsw_sp_mr_route_vif_entry *rve, *tmp;
        struct mlxsw_sp_mr_route *mr_route;
@@ -339,13 +339,13 @@ mlxsw_sp_mr_route4_create(struct mlxsw_sp_mr_table *mr_table,
 
        /* Find min_mtu and link iVIF and eVIFs */
        mr_route->min_mtu = ETH_MAX_MTU;
-       mr_cache_hold(&mfc->_c);
-       mr_route->mfc = &mfc->_c;
+       mr_cache_hold(mfc);
+       mr_route->mfc = mfc;
        mr_table->ops->key_create(mr_table, &mr_route->key, mr_route->mfc);
 
        mr_route->mr_table = mr_table;
        for (i = 0; i < MAXVIFS; i++) {
-               if (mfc->_c.mfc_un.res.ttls[i] != 255) {
+               if (mfc->mfc_un.res.ttls[i] != 255) {
                        err = mlxsw_sp_mr_route_evif_link(mr_route,
                                                          &mr_table->vifs[i]);
                        if (err)
@@ -356,44 +356,30 @@ mlxsw_sp_mr_route4_create(struct mlxsw_sp_mr_table *mr_table,
                }
        }
        mlxsw_sp_mr_route_ivif_link(mr_route,
-                                   &mr_table->vifs[mfc->_c.mfc_parent]);
+                                   &mr_table->vifs[mfc->mfc_parent]);
 
        mr_route->route_action = mlxsw_sp_mr_route_action(mr_route);
        return mr_route;
 err:
-       mr_cache_put(&mfc->_c);
+       mr_cache_put(mfc);
        list_for_each_entry_safe(rve, tmp, &mr_route->evif_list, route_node)
                mlxsw_sp_mr_route_evif_unlink(rve);
        kfree(mr_route);
        return ERR_PTR(err);
 }
 
-static void mlxsw_sp_mr_route4_destroy(struct mlxsw_sp_mr_table *mr_table,
-                                      struct mlxsw_sp_mr_route *mr_route)
+static void mlxsw_sp_mr_route_destroy(struct mlxsw_sp_mr_table *mr_table,
+                                     struct mlxsw_sp_mr_route *mr_route)
 {
        struct mlxsw_sp_mr_route_vif_entry *rve, *tmp;
 
        mlxsw_sp_mr_route_ivif_unlink(mr_route);
-       mr_cache_put((struct mr_mfc *)mr_route->mfc);
+       mr_cache_put(mr_route->mfc);
        list_for_each_entry_safe(rve, tmp, &mr_route->evif_list, route_node)
                mlxsw_sp_mr_route_evif_unlink(rve);
        kfree(mr_route);
 }
 
-static void mlxsw_sp_mr_route_destroy(struct mlxsw_sp_mr_table *mr_table,
-                                     struct mlxsw_sp_mr_route *mr_route)
-{
-       switch (mr_table->proto) {
-       case MLXSW_SP_L3_PROTO_IPV4:
-               mlxsw_sp_mr_route4_destroy(mr_table, mr_route);
-               break;
-       case MLXSW_SP_L3_PROTO_IPV6:
-               /* fall through */
-       default:
-               WARN_ON_ONCE(1);
-       }
-}
-
 static void mlxsw_sp_mr_mfc_offload_set(struct mlxsw_sp_mr_route *mr_route,
                                        bool offload)
 {
@@ -422,18 +408,18 @@ static void __mlxsw_sp_mr_route_del(struct mlxsw_sp_mr_table *mr_table,
        mlxsw_sp_mr_route_destroy(mr_table, mr_route);
 }
 
-int mlxsw_sp_mr_route4_add(struct mlxsw_sp_mr_table *mr_table,
-                          struct mfc_cache *mfc, bool replace)
+int mlxsw_sp_mr_route_add(struct mlxsw_sp_mr_table *mr_table,
+                         struct mr_mfc *mfc, bool replace)
 {
        struct mlxsw_sp_mr_route *mr_orig_route = NULL;
        struct mlxsw_sp_mr_route *mr_route;
        int err;
 
-       if (!mr_table->ops->is_route_valid(mr_table, &mfc->_c))
+       if (!mr_table->ops->is_route_valid(mr_table, mfc))
                return -EINVAL;
 
        /* Create a new route */
-       mr_route = mlxsw_sp_mr_route4_create(mr_table, mfc);
+       mr_route = mlxsw_sp_mr_route_create(mr_table, mfc);
        if (IS_ERR(mr_route))
                return PTR_ERR(mr_route);
 
@@ -478,7 +464,7 @@ int mlxsw_sp_mr_route4_add(struct mlxsw_sp_mr_table *mr_table,
                                       &mr_orig_route->ht_node,
                                       mlxsw_sp_mr_route_ht_params);
                list_del(&mr_orig_route->node);
-               mlxsw_sp_mr_route4_destroy(mr_table, mr_orig_route);
+               mlxsw_sp_mr_route_destroy(mr_table, mr_orig_route);
        }
 
        mlxsw_sp_mr_mfc_offload_update(mr_route);
@@ -491,17 +477,17 @@ err_rhashtable_insert:
        list_del(&mr_route->node);
 err_no_orig_route:
 err_duplicate_route:
-       mlxsw_sp_mr_route4_destroy(mr_table, mr_route);
+       mlxsw_sp_mr_route_destroy(mr_table, mr_route);
        return err;
 }
 
-void mlxsw_sp_mr_route4_del(struct mlxsw_sp_mr_table *mr_table,
-                           struct mfc_cache *mfc)
+void mlxsw_sp_mr_route_del(struct mlxsw_sp_mr_table *mr_table,
+                          struct mr_mfc *mfc)
 {
        struct mlxsw_sp_mr_route *mr_route;
        struct mlxsw_sp_mr_route_key key;
 
-       mr_table->ops->key_create(mr_table, &key, &mfc->_c);
+       mr_table->ops->key_create(mr_table, &key, mfc);
        mr_route = rhashtable_lookup_fast(&mr_table->route_ht, &key,
                                          mlxsw_sp_mr_route_ht_params);
        if (mr_route)
index 5d26a122af49df4c81eedd2519588e55e01599ad..6f94fc9eea1b906333f72cb495b5c791bdc77ef2 100644 (file)
@@ -109,10 +109,10 @@ struct mlxsw_sp_mr_table;
 int mlxsw_sp_mr_init(struct mlxsw_sp *mlxsw_sp,
                     const struct mlxsw_sp_mr_ops *mr_ops);
 void mlxsw_sp_mr_fini(struct mlxsw_sp *mlxsw_sp);
-int mlxsw_sp_mr_route4_add(struct mlxsw_sp_mr_table *mr_table,
-                          struct mfc_cache *mfc, bool replace);
-void mlxsw_sp_mr_route4_del(struct mlxsw_sp_mr_table *mr_table,
-                           struct mfc_cache *mfc);
+int mlxsw_sp_mr_route_add(struct mlxsw_sp_mr_table *mr_table,
+                         struct mr_mfc *mfc, bool replace);
+void mlxsw_sp_mr_route_del(struct mlxsw_sp_mr_table *mr_table,
+                          struct mr_mfc *mfc);
 int mlxsw_sp_mr_vif_add(struct mlxsw_sp_mr_table *mr_table,
                        struct net_device *dev, vifi_t vif_index,
                        unsigned long vif_flags,
index caa17e0a39faca31b640b9f1dbfb51d71c2504ae..14b887b96cbb8a8324d9d9be4a826200df9a7b16 100644 (file)
@@ -5393,10 +5393,23 @@ static int __mlxsw_sp_router_set_abort_trap(struct mlxsw_sp *mlxsw_sp,
        return 0;
 }
 
+static struct mlxsw_sp_mr_table *
+mlxsw_sp_router_fibmr_family_to_table(struct mlxsw_sp_vr *vr, int family)
+{
+       switch (family) {
+       case RTNL_FAMILY_IPMR:
+               return vr->mr_table[MLXSW_SP_L3_PROTO_IPV4];
+       default:
+               WARN_ON(1);
+               return vr->mr_table[MLXSW_SP_L3_PROTO_IPV4];
+       }
+}
+
 static int mlxsw_sp_router_fibmr_add(struct mlxsw_sp *mlxsw_sp,
                                     struct mfc_entry_notifier_info *men_info,
                                     bool replace)
 {
+       struct mlxsw_sp_mr_table *mrt;
        struct mlxsw_sp_vr *vr;
 
        if (mlxsw_sp->router->aborted)
@@ -5406,14 +5419,14 @@ static int mlxsw_sp_router_fibmr_add(struct mlxsw_sp *mlxsw_sp,
        if (IS_ERR(vr))
                return PTR_ERR(vr);
 
-       return mlxsw_sp_mr_route4_add(vr->mr_table[MLXSW_SP_L3_PROTO_IPV4],
-                                     (struct mfc_cache *) men_info->mfc,
-                                     replace);
+       mrt = mlxsw_sp_router_fibmr_family_to_table(vr, men_info->info.family);
+       return mlxsw_sp_mr_route_add(mrt, men_info->mfc, replace);
 }
 
 static void mlxsw_sp_router_fibmr_del(struct mlxsw_sp *mlxsw_sp,
                                      struct mfc_entry_notifier_info *men_info)
 {
+       struct mlxsw_sp_mr_table *mrt;
        struct mlxsw_sp_vr *vr;
 
        if (mlxsw_sp->router->aborted)
@@ -5423,8 +5436,8 @@ static void mlxsw_sp_router_fibmr_del(struct mlxsw_sp *mlxsw_sp,
        if (WARN_ON(!vr))
                return;
 
-       mlxsw_sp_mr_route4_del(vr->mr_table[MLXSW_SP_L3_PROTO_IPV4],
-                              (struct mfc_cache *) men_info->mfc);
+       mrt = mlxsw_sp_router_fibmr_family_to_table(vr, men_info->info.family);
+       mlxsw_sp_mr_route_del(mrt, men_info->mfc);
        mlxsw_sp_vr_put(mlxsw_sp, vr);
 }
 
@@ -5432,6 +5445,7 @@ static int
 mlxsw_sp_router_fibmr_vif_add(struct mlxsw_sp *mlxsw_sp,
                              struct vif_entry_notifier_info *ven_info)
 {
+       struct mlxsw_sp_mr_table *mrt;
        struct mlxsw_sp_rif *rif;
        struct mlxsw_sp_vr *vr;
 
@@ -5442,9 +5456,9 @@ mlxsw_sp_router_fibmr_vif_add(struct mlxsw_sp *mlxsw_sp,
        if (IS_ERR(vr))
                return PTR_ERR(vr);
 
+       mrt = mlxsw_sp_router_fibmr_family_to_table(vr, ven_info->info.family);
        rif = mlxsw_sp_rif_find_by_dev(mlxsw_sp, ven_info->dev);
-       return mlxsw_sp_mr_vif_add(vr->mr_table[MLXSW_SP_L3_PROTO_IPV4],
-                                  ven_info->dev,
+       return mlxsw_sp_mr_vif_add(mrt, ven_info->dev,
                                   ven_info->vif_index,
                                   ven_info->vif_flags, rif);
 }
@@ -5453,6 +5467,7 @@ static void
 mlxsw_sp_router_fibmr_vif_del(struct mlxsw_sp *mlxsw_sp,
                              struct vif_entry_notifier_info *ven_info)
 {
+       struct mlxsw_sp_mr_table *mrt;
        struct mlxsw_sp_vr *vr;
 
        if (mlxsw_sp->router->aborted)
@@ -5462,8 +5477,8 @@ mlxsw_sp_router_fibmr_vif_del(struct mlxsw_sp *mlxsw_sp,
        if (WARN_ON(!vr))
                return;
 
-       mlxsw_sp_mr_vif_del(vr->mr_table[MLXSW_SP_L3_PROTO_IPV4],
-                           ven_info->vif_index);
+       mrt = mlxsw_sp_router_fibmr_family_to_table(vr, ven_info->info.family);
+       mlxsw_sp_mr_vif_del(mrt, ven_info->vif_index);
        mlxsw_sp_vr_put(mlxsw_sp, vr);
 }