net/mlx5: Add support for flow table destination number
authorYishai Hadas <yishaih@mellanox.com>
Tue, 19 Jun 2018 12:23:36 +0000 (15:23 +0300)
committerLeon Romanovsky <leonro@mellanox.com>
Tue, 24 Jul 2018 05:51:20 +0000 (08:51 +0300)
Add support to set a destination from a flow table number.
This functionality will be used in downstream patches from this
series by the DEVX stuff.

Signed-off-by: Yishai Hadas <yishaih@mellanox.com>
Acked-by: Saeed Mahameed <saeedm@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
drivers/net/ethernet/mellanox/mlx5/core/diag/fs_tracepoint.c
drivers/net/ethernet/mellanox/mlx5/core/fs_cmd.c
drivers/net/ethernet/mellanox/mlx5/core/fs_core.c
include/linux/mlx5/fs.h
include/linux/mlx5/mlx5_ifc.h

index b3820a34e773a37370c322c530667c71c9992b25..0f11fff32a9b9215f9ea31169fd2c437ef7a9cde 100644 (file)
@@ -240,6 +240,9 @@ const char *parse_fs_dst(struct trace_seq *p,
        case MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE:
                trace_seq_printf(p, "ft=%p\n", dst->ft);
                break;
+       case MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE_NUM:
+               trace_seq_printf(p, "ft_num=%u\n", dst->ft_num);
+               break;
        case MLX5_FLOW_DESTINATION_TYPE_TIR:
                trace_seq_printf(p, "tir=%u\n", dst->tir_num);
                break;
index 6a62b84e57f4fcb90fb54c7e8fb1ccfb9c686503..8e01f818021b9605743a4f3ed438e7300f378c81 100644 (file)
@@ -368,18 +368,20 @@ static int mlx5_cmd_set_fte(struct mlx5_core_dev *dev,
                int list_size = 0;
 
                list_for_each_entry(dst, &fte->node.children, node.list) {
-                       unsigned int id;
+                       unsigned int id, type = dst->dest_attr.type;
 
-                       if (dst->dest_attr.type == MLX5_FLOW_DESTINATION_TYPE_COUNTER)
+                       if (type == MLX5_FLOW_DESTINATION_TYPE_COUNTER)
                                continue;
 
-                       MLX5_SET(dest_format_struct, in_dests, destination_type,
-                                dst->dest_attr.type);
-                       if (dst->dest_attr.type ==
-                           MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE) {
+                       switch (type) {
+                       case MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE_NUM:
+                               id = dst->dest_attr.ft_num;
+                               type = MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE;
+                               break;
+                       case MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE:
                                id = dst->dest_attr.ft->id;
-                       } else if (dst->dest_attr.type ==
-                                  MLX5_FLOW_DESTINATION_TYPE_VPORT) {
+                               break;
+                       case MLX5_FLOW_DESTINATION_TYPE_VPORT:
                                id = dst->dest_attr.vport.num;
                                MLX5_SET(dest_format_struct, in_dests,
                                         destination_eswitch_owner_vhca_id_valid,
@@ -387,9 +389,13 @@ static int mlx5_cmd_set_fte(struct mlx5_core_dev *dev,
                                MLX5_SET(dest_format_struct, in_dests,
                                         destination_eswitch_owner_vhca_id,
                                         dst->dest_attr.vport.vhca_id);
-                       } else {
+                               break;
+                       default:
                                id = dst->dest_attr.tir_num;
                        }
+
+                       MLX5_SET(dest_format_struct, in_dests, destination_type,
+                                type);
                        MLX5_SET(dest_format_struct, in_dests, destination_id, id);
                        in_dests += MLX5_ST_SZ_BYTES(dest_format_struct);
                        list_size++;
index a8d7d00f92f6615855f6653bf560f43073928c8d..0d8378243903e0cc9899631273957769ba851445 100644 (file)
@@ -1356,7 +1356,9 @@ static bool mlx5_flow_dests_cmp(struct mlx5_flow_destination *d1,
                    (d1->type == MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE &&
                     d1->ft == d2->ft) ||
                    (d1->type == MLX5_FLOW_DESTINATION_TYPE_TIR &&
-                    d1->tir_num == d2->tir_num))
+                    d1->tir_num == d2->tir_num) ||
+                   (d1->type == MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE_NUM &&
+                    d1->ft_num == d2->ft_num))
                        return true;
        }
 
index c40f2fc68655d368a49a1d7b0904e5eda45bbc78..af0592400499b2c08f2aef1fd679607e6fe80c0a 100644 (file)
@@ -89,6 +89,7 @@ struct mlx5_flow_destination {
        enum mlx5_flow_destination_type type;
        union {
                u32                     tir_num;
+               u32                     ft_num;
                struct mlx5_flow_table  *ft;
                struct mlx5_fc          *counter;
                struct {
index ae12120ef02103c23b2278c5ddd7d9de38ed246d..c14b815595056ecc783adc8cb06bcc2975f8aa5c 100644 (file)
@@ -1180,6 +1180,7 @@ enum mlx5_flow_destination_type {
 
        MLX5_FLOW_DESTINATION_TYPE_PORT         = 0x99,
        MLX5_FLOW_DESTINATION_TYPE_COUNTER      = 0x100,
+       MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE_NUM = 0x101,
 };
 
 struct mlx5_ifc_dest_format_struct_bits {