| .. | .. |
|---|
| 32 | 32 | |
|---|
| 33 | 33 | #include <linux/mutex.h> |
|---|
| 34 | 34 | #include <linux/mlx5/driver.h> |
|---|
| 35 | +#include <linux/mlx5/vport.h> |
|---|
| 35 | 36 | #include <linux/mlx5/eswitch.h> |
|---|
| 36 | 37 | |
|---|
| 37 | 38 | #include "mlx5_core.h" |
|---|
| .. | .. |
|---|
| 58 | 59 | ADD_PRIO(num_prios_val, 0, num_levels_val, {},\ |
|---|
| 59 | 60 | __VA_ARGS__)\ |
|---|
| 60 | 61 | |
|---|
| 61 | | -#define ADD_NS(...) {.type = FS_TYPE_NAMESPACE,\ |
|---|
| 62 | +#define ADD_NS(def_miss_act, ...) {.type = FS_TYPE_NAMESPACE, \ |
|---|
| 63 | + .def_miss_action = def_miss_act,\ |
|---|
| 62 | 64 | .children = (struct init_tree_node[]) {__VA_ARGS__},\ |
|---|
| 63 | 65 | .ar_size = INIT_TREE_NODE_ARRAY_SIZE(__VA_ARGS__) \ |
|---|
| 64 | 66 | } |
|---|
| .. | .. |
|---|
| 76 | 78 | FS_CAP(flow_table_properties_nic_receive.identified_miss_table_mode), \ |
|---|
| 77 | 79 | FS_CAP(flow_table_properties_nic_receive.flow_table_modify)) |
|---|
| 78 | 80 | |
|---|
| 81 | +#define FS_CHAINING_CAPS_EGRESS \ |
|---|
| 82 | + FS_REQUIRED_CAPS( \ |
|---|
| 83 | + FS_CAP(flow_table_properties_nic_transmit.flow_modify_en), \ |
|---|
| 84 | + FS_CAP(flow_table_properties_nic_transmit.modify_root), \ |
|---|
| 85 | + FS_CAP(flow_table_properties_nic_transmit \ |
|---|
| 86 | + .identified_miss_table_mode), \ |
|---|
| 87 | + FS_CAP(flow_table_properties_nic_transmit.flow_table_modify)) |
|---|
| 88 | + |
|---|
| 89 | +#define FS_CHAINING_CAPS_RDMA_TX \ |
|---|
| 90 | + FS_REQUIRED_CAPS( \ |
|---|
| 91 | + FS_CAP(flow_table_properties_nic_transmit_rdma.flow_modify_en), \ |
|---|
| 92 | + FS_CAP(flow_table_properties_nic_transmit_rdma.modify_root), \ |
|---|
| 93 | + FS_CAP(flow_table_properties_nic_transmit_rdma \ |
|---|
| 94 | + .identified_miss_table_mode), \ |
|---|
| 95 | + FS_CAP(flow_table_properties_nic_transmit_rdma \ |
|---|
| 96 | + .flow_table_modify)) |
|---|
| 97 | + |
|---|
| 79 | 98 | #define LEFTOVERS_NUM_LEVELS 1 |
|---|
| 80 | 99 | #define LEFTOVERS_NUM_PRIOS 1 |
|---|
| 81 | 100 | |
|---|
| .. | .. |
|---|
| 86 | 105 | #define ETHTOOL_PRIO_NUM_LEVELS 1 |
|---|
| 87 | 106 | #define ETHTOOL_NUM_PRIOS 11 |
|---|
| 88 | 107 | #define ETHTOOL_MIN_LEVEL (KERNEL_MIN_LEVEL + ETHTOOL_NUM_PRIOS) |
|---|
| 89 | | -/* Vlan, mac, ttc, inner ttc, aRFS */ |
|---|
| 90 | | -#define KERNEL_NIC_PRIO_NUM_LEVELS 5 |
|---|
| 108 | +/* Vlan, mac, ttc, inner ttc, {aRFS/accel and esp/esp_err} */ |
|---|
| 109 | +#define KERNEL_NIC_PRIO_NUM_LEVELS 6 |
|---|
| 91 | 110 | #define KERNEL_NIC_NUM_PRIOS 1 |
|---|
| 92 | 111 | /* One more level for tc */ |
|---|
| 93 | 112 | #define KERNEL_MIN_LEVEL (KERNEL_NIC_PRIO_NUM_LEVELS + 1) |
|---|
| .. | .. |
|---|
| 99 | 118 | #define ANCHOR_NUM_PRIOS 1 |
|---|
| 100 | 119 | #define ANCHOR_MIN_LEVEL (BY_PASS_MIN_LEVEL + 1) |
|---|
| 101 | 120 | |
|---|
| 102 | | -#define OFFLOADS_MAX_FT 1 |
|---|
| 103 | | -#define OFFLOADS_NUM_PRIOS 1 |
|---|
| 104 | | -#define OFFLOADS_MIN_LEVEL (ANCHOR_MIN_LEVEL + 1) |
|---|
| 121 | +#define OFFLOADS_MAX_FT 2 |
|---|
| 122 | +#define OFFLOADS_NUM_PRIOS 2 |
|---|
| 123 | +#define OFFLOADS_MIN_LEVEL (ANCHOR_MIN_LEVEL + OFFLOADS_NUM_PRIOS) |
|---|
| 105 | 124 | |
|---|
| 106 | 125 | #define LAG_PRIO_NUM_LEVELS 1 |
|---|
| 107 | 126 | #define LAG_NUM_PRIOS 1 |
|---|
| 108 | 127 | #define LAG_MIN_LEVEL (OFFLOADS_MIN_LEVEL + 1) |
|---|
| 128 | + |
|---|
| 129 | +#define KERNEL_TX_IPSEC_NUM_PRIOS 1 |
|---|
| 130 | +#define KERNEL_TX_IPSEC_NUM_LEVELS 1 |
|---|
| 131 | +#define KERNEL_TX_MIN_LEVEL (KERNEL_TX_IPSEC_NUM_LEVELS) |
|---|
| 109 | 132 | |
|---|
| 110 | 133 | struct node_caps { |
|---|
| 111 | 134 | size_t arr_sz; |
|---|
| .. | .. |
|---|
| 121 | 144 | int num_leaf_prios; |
|---|
| 122 | 145 | int prio; |
|---|
| 123 | 146 | int num_levels; |
|---|
| 147 | + enum mlx5_flow_table_miss_action def_miss_action; |
|---|
| 124 | 148 | } root_fs = { |
|---|
| 125 | 149 | .type = FS_TYPE_NAMESPACE, |
|---|
| 126 | 150 | .ar_size = 7, |
|---|
| 151 | + .children = (struct init_tree_node[]){ |
|---|
| 152 | + ADD_PRIO(0, BY_PASS_MIN_LEVEL, 0, FS_CHAINING_CAPS, |
|---|
| 153 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 154 | + ADD_MULTIPLE_PRIO(MLX5_BY_PASS_NUM_PRIOS, |
|---|
| 155 | + BY_PASS_PRIO_NUM_LEVELS))), |
|---|
| 156 | + ADD_PRIO(0, LAG_MIN_LEVEL, 0, FS_CHAINING_CAPS, |
|---|
| 157 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 158 | + ADD_MULTIPLE_PRIO(LAG_NUM_PRIOS, |
|---|
| 159 | + LAG_PRIO_NUM_LEVELS))), |
|---|
| 160 | + ADD_PRIO(0, OFFLOADS_MIN_LEVEL, 0, FS_CHAINING_CAPS, |
|---|
| 161 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 162 | + ADD_MULTIPLE_PRIO(OFFLOADS_NUM_PRIOS, |
|---|
| 163 | + OFFLOADS_MAX_FT))), |
|---|
| 164 | + ADD_PRIO(0, ETHTOOL_MIN_LEVEL, 0, FS_CHAINING_CAPS, |
|---|
| 165 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 166 | + ADD_MULTIPLE_PRIO(ETHTOOL_NUM_PRIOS, |
|---|
| 167 | + ETHTOOL_PRIO_NUM_LEVELS))), |
|---|
| 168 | + ADD_PRIO(0, KERNEL_MIN_LEVEL, 0, {}, |
|---|
| 169 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 170 | + ADD_MULTIPLE_PRIO(KERNEL_NIC_TC_NUM_PRIOS, |
|---|
| 171 | + KERNEL_NIC_TC_NUM_LEVELS), |
|---|
| 172 | + ADD_MULTIPLE_PRIO(KERNEL_NIC_NUM_PRIOS, |
|---|
| 173 | + KERNEL_NIC_PRIO_NUM_LEVELS))), |
|---|
| 174 | + ADD_PRIO(0, BY_PASS_MIN_LEVEL, 0, FS_CHAINING_CAPS, |
|---|
| 175 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 176 | + ADD_MULTIPLE_PRIO(LEFTOVERS_NUM_PRIOS, |
|---|
| 177 | + LEFTOVERS_NUM_LEVELS))), |
|---|
| 178 | + ADD_PRIO(0, ANCHOR_MIN_LEVEL, 0, {}, |
|---|
| 179 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 180 | + ADD_MULTIPLE_PRIO(ANCHOR_NUM_PRIOS, |
|---|
| 181 | + ANCHOR_NUM_LEVELS))), |
|---|
| 182 | + } |
|---|
| 183 | +}; |
|---|
| 184 | + |
|---|
| 185 | +static struct init_tree_node egress_root_fs = { |
|---|
| 186 | + .type = FS_TYPE_NAMESPACE, |
|---|
| 187 | +#ifdef CONFIG_MLX5_IPSEC |
|---|
| 188 | + .ar_size = 2, |
|---|
| 189 | +#else |
|---|
| 190 | + .ar_size = 1, |
|---|
| 191 | +#endif |
|---|
| 127 | 192 | .children = (struct init_tree_node[]) { |
|---|
| 128 | | - ADD_PRIO(0, BY_PASS_MIN_LEVEL, 0, |
|---|
| 129 | | - FS_CHAINING_CAPS, |
|---|
| 130 | | - ADD_NS(ADD_MULTIPLE_PRIO(MLX5_BY_PASS_NUM_PRIOS, |
|---|
| 193 | + ADD_PRIO(0, MLX5_BY_PASS_NUM_PRIOS, 0, |
|---|
| 194 | + FS_CHAINING_CAPS_EGRESS, |
|---|
| 195 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 196 | + ADD_MULTIPLE_PRIO(MLX5_BY_PASS_NUM_PRIOS, |
|---|
| 131 | 197 | BY_PASS_PRIO_NUM_LEVELS))), |
|---|
| 132 | | - ADD_PRIO(0, LAG_MIN_LEVEL, 0, |
|---|
| 198 | +#ifdef CONFIG_MLX5_IPSEC |
|---|
| 199 | + ADD_PRIO(0, KERNEL_TX_MIN_LEVEL, 0, |
|---|
| 200 | + FS_CHAINING_CAPS_EGRESS, |
|---|
| 201 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 202 | + ADD_MULTIPLE_PRIO(KERNEL_TX_IPSEC_NUM_PRIOS, |
|---|
| 203 | + KERNEL_TX_IPSEC_NUM_LEVELS))), |
|---|
| 204 | +#endif |
|---|
| 205 | + } |
|---|
| 206 | +}; |
|---|
| 207 | + |
|---|
| 208 | +#define RDMA_RX_BYPASS_PRIO 0 |
|---|
| 209 | +#define RDMA_RX_KERNEL_PRIO 1 |
|---|
| 210 | +static struct init_tree_node rdma_rx_root_fs = { |
|---|
| 211 | + .type = FS_TYPE_NAMESPACE, |
|---|
| 212 | + .ar_size = 2, |
|---|
| 213 | + .children = (struct init_tree_node[]) { |
|---|
| 214 | + [RDMA_RX_BYPASS_PRIO] = |
|---|
| 215 | + ADD_PRIO(0, MLX5_BY_PASS_NUM_REGULAR_PRIOS, 0, |
|---|
| 133 | 216 | FS_CHAINING_CAPS, |
|---|
| 134 | | - ADD_NS(ADD_MULTIPLE_PRIO(LAG_NUM_PRIOS, |
|---|
| 135 | | - LAG_PRIO_NUM_LEVELS))), |
|---|
| 136 | | - ADD_PRIO(0, OFFLOADS_MIN_LEVEL, 0, {}, |
|---|
| 137 | | - ADD_NS(ADD_MULTIPLE_PRIO(OFFLOADS_NUM_PRIOS, OFFLOADS_MAX_FT))), |
|---|
| 138 | | - ADD_PRIO(0, ETHTOOL_MIN_LEVEL, 0, |
|---|
| 217 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 218 | + ADD_MULTIPLE_PRIO(MLX5_BY_PASS_NUM_REGULAR_PRIOS, |
|---|
| 219 | + BY_PASS_PRIO_NUM_LEVELS))), |
|---|
| 220 | + [RDMA_RX_KERNEL_PRIO] = |
|---|
| 221 | + ADD_PRIO(0, MLX5_BY_PASS_NUM_REGULAR_PRIOS + 1, 0, |
|---|
| 139 | 222 | FS_CHAINING_CAPS, |
|---|
| 140 | | - ADD_NS(ADD_MULTIPLE_PRIO(ETHTOOL_NUM_PRIOS, |
|---|
| 141 | | - ETHTOOL_PRIO_NUM_LEVELS))), |
|---|
| 142 | | - ADD_PRIO(0, KERNEL_MIN_LEVEL, 0, {}, |
|---|
| 143 | | - ADD_NS(ADD_MULTIPLE_PRIO(KERNEL_NIC_TC_NUM_PRIOS, KERNEL_NIC_TC_NUM_LEVELS), |
|---|
| 144 | | - ADD_MULTIPLE_PRIO(KERNEL_NIC_NUM_PRIOS, |
|---|
| 145 | | - KERNEL_NIC_PRIO_NUM_LEVELS))), |
|---|
| 146 | | - ADD_PRIO(0, BY_PASS_MIN_LEVEL, 0, |
|---|
| 147 | | - FS_CHAINING_CAPS, |
|---|
| 148 | | - ADD_NS(ADD_MULTIPLE_PRIO(LEFTOVERS_NUM_PRIOS, LEFTOVERS_NUM_LEVELS))), |
|---|
| 149 | | - ADD_PRIO(0, ANCHOR_MIN_LEVEL, 0, {}, |
|---|
| 150 | | - ADD_NS(ADD_MULTIPLE_PRIO(ANCHOR_NUM_PRIOS, ANCHOR_NUM_LEVELS))), |
|---|
| 223 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_SWITCH_DOMAIN, |
|---|
| 224 | + ADD_MULTIPLE_PRIO(1, 1))), |
|---|
| 225 | + } |
|---|
| 226 | +}; |
|---|
| 227 | + |
|---|
| 228 | +static struct init_tree_node rdma_tx_root_fs = { |
|---|
| 229 | + .type = FS_TYPE_NAMESPACE, |
|---|
| 230 | + .ar_size = 1, |
|---|
| 231 | + .children = (struct init_tree_node[]) { |
|---|
| 232 | + ADD_PRIO(0, MLX5_BY_PASS_NUM_PRIOS, 0, |
|---|
| 233 | + FS_CHAINING_CAPS_RDMA_TX, |
|---|
| 234 | + ADD_NS(MLX5_FLOW_TABLE_MISS_ACTION_DEF, |
|---|
| 235 | + ADD_MULTIPLE_PRIO(MLX5_BY_PASS_NUM_PRIOS, |
|---|
| 236 | + BY_PASS_PRIO_NUM_LEVELS))), |
|---|
| 151 | 237 | } |
|---|
| 152 | 238 | }; |
|---|
| 153 | 239 | |
|---|
| .. | .. |
|---|
| 158 | 244 | }; |
|---|
| 159 | 245 | |
|---|
| 160 | 246 | static const struct rhashtable_params rhash_fte = { |
|---|
| 161 | | - .key_len = FIELD_SIZEOF(struct fs_fte, val), |
|---|
| 247 | + .key_len = sizeof_field(struct fs_fte, val), |
|---|
| 162 | 248 | .key_offset = offsetof(struct fs_fte, val), |
|---|
| 163 | 249 | .head_offset = offsetof(struct fs_fte, hash), |
|---|
| 164 | 250 | .automatic_shrinking = true, |
|---|
| .. | .. |
|---|
| 166 | 252 | }; |
|---|
| 167 | 253 | |
|---|
| 168 | 254 | static const struct rhashtable_params rhash_fg = { |
|---|
| 169 | | - .key_len = FIELD_SIZEOF(struct mlx5_flow_group, mask), |
|---|
| 255 | + .key_len = sizeof_field(struct mlx5_flow_group, mask), |
|---|
| 170 | 256 | .key_offset = offsetof(struct mlx5_flow_group, mask), |
|---|
| 171 | 257 | .head_offset = offsetof(struct mlx5_flow_group, hash), |
|---|
| 172 | 258 | .automatic_shrinking = true, |
|---|
| .. | .. |
|---|
| 182 | 268 | static void del_sw_fte(struct fs_node *node); |
|---|
| 183 | 269 | static void del_sw_prio(struct fs_node *node); |
|---|
| 184 | 270 | static void del_sw_ns(struct fs_node *node); |
|---|
| 185 | | -/* Delete rule (destination) is special case that |
|---|
| 271 | +/* Delete rule (destination) is special case that |
|---|
| 186 | 272 | * requires to lock the FTE for all the deletion process. |
|---|
| 187 | 273 | */ |
|---|
| 188 | 274 | static void del_sw_hw_rule(struct fs_node *node); |
|---|
| .. | .. |
|---|
| 242 | 328 | } |
|---|
| 243 | 329 | } |
|---|
| 244 | 330 | |
|---|
| 245 | | -static void down_write_ref_node(struct fs_node *node) |
|---|
| 331 | +static void down_write_ref_node(struct fs_node *node, bool locked) |
|---|
| 246 | 332 | { |
|---|
| 247 | 333 | if (node) { |
|---|
| 248 | | - down_write(&node->lock); |
|---|
| 334 | + if (!locked) |
|---|
| 335 | + down_write(&node->lock); |
|---|
| 249 | 336 | refcount_inc(&node->refcount); |
|---|
| 250 | 337 | } |
|---|
| 251 | 338 | } |
|---|
| .. | .. |
|---|
| 256 | 343 | up_read(&node->lock); |
|---|
| 257 | 344 | } |
|---|
| 258 | 345 | |
|---|
| 259 | | -static void up_write_ref_node(struct fs_node *node) |
|---|
| 346 | +static void up_write_ref_node(struct fs_node *node, bool locked) |
|---|
| 260 | 347 | { |
|---|
| 261 | 348 | refcount_dec(&node->refcount); |
|---|
| 262 | | - up_write(&node->lock); |
|---|
| 349 | + if (!locked) |
|---|
| 350 | + up_write(&node->lock); |
|---|
| 263 | 351 | } |
|---|
| 264 | 352 | |
|---|
| 265 | | -static void tree_put_node(struct fs_node *node) |
|---|
| 353 | +static void tree_put_node(struct fs_node *node, bool locked) |
|---|
| 266 | 354 | { |
|---|
| 267 | 355 | struct fs_node *parent_node = node->parent; |
|---|
| 268 | 356 | |
|---|
| .. | .. |
|---|
| 270 | 358 | if (node->del_hw_func) |
|---|
| 271 | 359 | node->del_hw_func(node); |
|---|
| 272 | 360 | if (parent_node) { |
|---|
| 273 | | - /* Only root namespace doesn't have parent and we just |
|---|
| 274 | | - * need to free its node. |
|---|
| 275 | | - */ |
|---|
| 276 | | - down_write_ref_node(parent_node); |
|---|
| 361 | + down_write_ref_node(parent_node, locked); |
|---|
| 277 | 362 | list_del_init(&node->list); |
|---|
| 278 | | - if (node->del_sw_func) |
|---|
| 279 | | - node->del_sw_func(node); |
|---|
| 280 | | - up_write_ref_node(parent_node); |
|---|
| 281 | | - } else { |
|---|
| 282 | | - kfree(node); |
|---|
| 283 | 363 | } |
|---|
| 364 | + node->del_sw_func(node); |
|---|
| 365 | + if (parent_node) |
|---|
| 366 | + up_write_ref_node(parent_node, locked); |
|---|
| 284 | 367 | node = NULL; |
|---|
| 285 | 368 | } |
|---|
| 286 | 369 | if (!node && parent_node) |
|---|
| 287 | | - tree_put_node(parent_node); |
|---|
| 370 | + tree_put_node(parent_node, locked); |
|---|
| 288 | 371 | } |
|---|
| 289 | 372 | |
|---|
| 290 | | -static int tree_remove_node(struct fs_node *node) |
|---|
| 373 | +static int tree_remove_node(struct fs_node *node, bool locked) |
|---|
| 291 | 374 | { |
|---|
| 292 | 375 | if (refcount_read(&node->refcount) > 1) { |
|---|
| 293 | 376 | refcount_dec(&node->refcount); |
|---|
| 294 | 377 | return -EEXIST; |
|---|
| 295 | 378 | } |
|---|
| 296 | | - tree_put_node(node); |
|---|
| 379 | + tree_put_node(node, locked); |
|---|
| 297 | 380 | return 0; |
|---|
| 298 | 381 | } |
|---|
| 299 | 382 | |
|---|
| .. | .. |
|---|
| 308 | 391 | } |
|---|
| 309 | 392 | |
|---|
| 310 | 393 | return NULL; |
|---|
| 394 | +} |
|---|
| 395 | + |
|---|
| 396 | +static bool is_fwd_next_action(u32 action) |
|---|
| 397 | +{ |
|---|
| 398 | + return action & (MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_PRIO | |
|---|
| 399 | + MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_NS); |
|---|
| 311 | 400 | } |
|---|
| 312 | 401 | |
|---|
| 313 | 402 | static bool check_valid_spec(const struct mlx5_flow_spec *spec) |
|---|
| .. | .. |
|---|
| 377 | 466 | fs_get_obj(ft, node); |
|---|
| 378 | 467 | dev = get_dev(&ft->node); |
|---|
| 379 | 468 | root = find_root(&ft->node); |
|---|
| 469 | + trace_mlx5_fs_del_ft(ft); |
|---|
| 380 | 470 | |
|---|
| 381 | 471 | if (node->active) { |
|---|
| 382 | | - err = root->cmds->destroy_flow_table(dev, ft); |
|---|
| 472 | + err = root->cmds->destroy_flow_table(root, ft); |
|---|
| 383 | 473 | if (err) |
|---|
| 384 | 474 | mlx5_core_warn(dev, "flow steering can't destroy ft\n"); |
|---|
| 385 | 475 | } |
|---|
| .. | .. |
|---|
| 393 | 483 | fs_get_obj(ft, node); |
|---|
| 394 | 484 | |
|---|
| 395 | 485 | rhltable_destroy(&ft->fgs_hash); |
|---|
| 396 | | - fs_get_obj(prio, ft->node.parent); |
|---|
| 397 | | - prio->num_ft--; |
|---|
| 486 | + if (ft->node.parent) { |
|---|
| 487 | + fs_get_obj(prio, ft->node.parent); |
|---|
| 488 | + prio->num_ft--; |
|---|
| 489 | + } |
|---|
| 398 | 490 | kfree(ft); |
|---|
| 491 | +} |
|---|
| 492 | + |
|---|
| 493 | +static void modify_fte(struct fs_fte *fte) |
|---|
| 494 | +{ |
|---|
| 495 | + struct mlx5_flow_root_namespace *root; |
|---|
| 496 | + struct mlx5_flow_table *ft; |
|---|
| 497 | + struct mlx5_flow_group *fg; |
|---|
| 498 | + struct mlx5_core_dev *dev; |
|---|
| 499 | + int err; |
|---|
| 500 | + |
|---|
| 501 | + fs_get_obj(fg, fte->node.parent); |
|---|
| 502 | + fs_get_obj(ft, fg->node.parent); |
|---|
| 503 | + dev = get_dev(&fte->node); |
|---|
| 504 | + |
|---|
| 505 | + root = find_root(&ft->node); |
|---|
| 506 | + err = root->cmds->update_fte(root, ft, fg, fte->modify_mask, fte); |
|---|
| 507 | + if (err) |
|---|
| 508 | + mlx5_core_warn(dev, |
|---|
| 509 | + "%s can't del rule fg id=%d fte_index=%d\n", |
|---|
| 510 | + __func__, fg->id, fte->index); |
|---|
| 511 | + fte->modify_mask = 0; |
|---|
| 399 | 512 | } |
|---|
| 400 | 513 | |
|---|
| 401 | 514 | static void del_sw_hw_rule(struct fs_node *node) |
|---|
| 402 | 515 | { |
|---|
| 403 | | - struct mlx5_flow_root_namespace *root; |
|---|
| 404 | 516 | struct mlx5_flow_rule *rule; |
|---|
| 405 | | - struct mlx5_flow_table *ft; |
|---|
| 406 | | - struct mlx5_flow_group *fg; |
|---|
| 407 | 517 | struct fs_fte *fte; |
|---|
| 408 | | - int modify_mask; |
|---|
| 409 | | - struct mlx5_core_dev *dev = get_dev(node); |
|---|
| 410 | | - int err; |
|---|
| 411 | | - bool update_fte = false; |
|---|
| 412 | 518 | |
|---|
| 413 | 519 | fs_get_obj(rule, node); |
|---|
| 414 | 520 | fs_get_obj(fte, rule->node.parent); |
|---|
| 415 | | - fs_get_obj(fg, fte->node.parent); |
|---|
| 416 | | - fs_get_obj(ft, fg->node.parent); |
|---|
| 417 | 521 | trace_mlx5_fs_del_rule(rule); |
|---|
| 418 | | - if (rule->sw_action == MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_PRIO) { |
|---|
| 522 | + if (is_fwd_next_action(rule->sw_action)) { |
|---|
| 419 | 523 | mutex_lock(&rule->dest_attr.ft->lock); |
|---|
| 420 | 524 | list_del(&rule->next_ft); |
|---|
| 421 | 525 | mutex_unlock(&rule->dest_attr.ft->lock); |
|---|
| .. | .. |
|---|
| 423 | 527 | |
|---|
| 424 | 528 | if (rule->dest_attr.type == MLX5_FLOW_DESTINATION_TYPE_COUNTER && |
|---|
| 425 | 529 | --fte->dests_size) { |
|---|
| 426 | | - modify_mask = BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_ACTION) | |
|---|
| 427 | | - BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_FLOW_COUNTERS); |
|---|
| 530 | + fte->modify_mask |= |
|---|
| 531 | + BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_ACTION) | |
|---|
| 532 | + BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_FLOW_COUNTERS); |
|---|
| 428 | 533 | fte->action.action &= ~MLX5_FLOW_CONTEXT_ACTION_COUNT; |
|---|
| 429 | | - update_fte = true; |
|---|
| 534 | + goto out; |
|---|
| 535 | + } |
|---|
| 536 | + |
|---|
| 537 | + if (rule->dest_attr.type == MLX5_FLOW_DESTINATION_TYPE_PORT && |
|---|
| 538 | + --fte->dests_size) { |
|---|
| 539 | + fte->modify_mask |= BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_ACTION); |
|---|
| 540 | + fte->action.action &= ~MLX5_FLOW_CONTEXT_ACTION_ALLOW; |
|---|
| 430 | 541 | goto out; |
|---|
| 431 | 542 | } |
|---|
| 432 | 543 | |
|---|
| 433 | 544 | if ((fte->action.action & MLX5_FLOW_CONTEXT_ACTION_FWD_DEST) && |
|---|
| 434 | 545 | --fte->dests_size) { |
|---|
| 435 | | - modify_mask = BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_DESTINATION_LIST); |
|---|
| 436 | | - update_fte = true; |
|---|
| 546 | + fte->modify_mask |= |
|---|
| 547 | + BIT(MLX5_SET_FTE_MODIFY_ENABLE_MASK_DESTINATION_LIST); |
|---|
| 437 | 548 | } |
|---|
| 438 | 549 | out: |
|---|
| 439 | | - root = find_root(&ft->node); |
|---|
| 440 | | - if (update_fte && fte->dests_size) { |
|---|
| 441 | | - err = root->cmds->update_fte(dev, ft, fg->id, modify_mask, fte); |
|---|
| 442 | | - if (err) |
|---|
| 443 | | - mlx5_core_warn(dev, |
|---|
| 444 | | - "%s can't del rule fg id=%d fte_index=%d\n", |
|---|
| 445 | | - __func__, fg->id, fte->index); |
|---|
| 446 | | - } |
|---|
| 447 | 550 | kfree(rule); |
|---|
| 448 | 551 | } |
|---|
| 449 | 552 | |
|---|
| .. | .. |
|---|
| 464 | 567 | dev = get_dev(&ft->node); |
|---|
| 465 | 568 | root = find_root(&ft->node); |
|---|
| 466 | 569 | if (node->active) { |
|---|
| 467 | | - err = root->cmds->delete_fte(dev, ft, fte); |
|---|
| 570 | + err = root->cmds->delete_fte(root, ft, fte); |
|---|
| 468 | 571 | if (err) |
|---|
| 469 | 572 | mlx5_core_warn(dev, |
|---|
| 470 | 573 | "flow steering can't delete fte in index %d of flow group id %d\n", |
|---|
| .. | .. |
|---|
| 504 | 607 | trace_mlx5_fs_del_fg(fg); |
|---|
| 505 | 608 | |
|---|
| 506 | 609 | root = find_root(&ft->node); |
|---|
| 507 | | - if (fg->node.active && root->cmds->destroy_flow_group(dev, ft, fg->id)) |
|---|
| 610 | + if (fg->node.active && root->cmds->destroy_flow_group(root, ft, fg)) |
|---|
| 508 | 611 | mlx5_core_warn(dev, "flow steering can't destroy fg %d of ft %d\n", |
|---|
| 509 | 612 | fg->id, ft->id); |
|---|
| 510 | 613 | } |
|---|
| .. | .. |
|---|
| 521 | 624 | |
|---|
| 522 | 625 | rhashtable_destroy(&fg->ftes_hash); |
|---|
| 523 | 626 | ida_destroy(&fg->fte_allocator); |
|---|
| 524 | | - if (ft->autogroup.active && fg->max_ftes == ft->autogroup.group_size) |
|---|
| 627 | + if (ft->autogroup.active && |
|---|
| 628 | + fg->max_ftes == ft->autogroup.group_size && |
|---|
| 629 | + fg->start_index < ft->autogroup.max_fte) |
|---|
| 525 | 630 | ft->autogroup.num_groups--; |
|---|
| 526 | 631 | err = rhltable_remove(&ft->fgs_hash, |
|---|
| 527 | 632 | &fg->hash, |
|---|
| .. | .. |
|---|
| 556 | 661 | } |
|---|
| 557 | 662 | |
|---|
| 558 | 663 | static struct fs_fte *alloc_fte(struct mlx5_flow_table *ft, |
|---|
| 559 | | - u32 *match_value, |
|---|
| 664 | + const struct mlx5_flow_spec *spec, |
|---|
| 560 | 665 | struct mlx5_flow_act *flow_act) |
|---|
| 561 | 666 | { |
|---|
| 562 | 667 | struct mlx5_flow_steering *steering = get_steering(&ft->node); |
|---|
| .. | .. |
|---|
| 566 | 671 | if (!fte) |
|---|
| 567 | 672 | return ERR_PTR(-ENOMEM); |
|---|
| 568 | 673 | |
|---|
| 569 | | - memcpy(fte->val, match_value, sizeof(fte->val)); |
|---|
| 674 | + memcpy(fte->val, &spec->match_value, sizeof(fte->val)); |
|---|
| 570 | 675 | fte->node.type = FS_TYPE_FLOW_ENTRY; |
|---|
| 571 | 676 | fte->action = *flow_act; |
|---|
| 677 | + fte->flow_context = spec->flow_context; |
|---|
| 572 | 678 | |
|---|
| 573 | 679 | tree_init_node(&fte->node, del_hw_fte, del_sw_fte); |
|---|
| 574 | 680 | |
|---|
| .. | .. |
|---|
| 584 | 690 | |
|---|
| 585 | 691 | static struct mlx5_flow_group *alloc_flow_group(struct mlx5_flow_steering *steering, |
|---|
| 586 | 692 | u8 match_criteria_enable, |
|---|
| 587 | | - void *match_criteria, |
|---|
| 693 | + const void *match_criteria, |
|---|
| 588 | 694 | int start_index, |
|---|
| 589 | 695 | int end_index) |
|---|
| 590 | 696 | { |
|---|
| .. | .. |
|---|
| 599 | 705 | if (ret) { |
|---|
| 600 | 706 | kmem_cache_free(steering->fgs_cache, fg); |
|---|
| 601 | 707 | return ERR_PTR(ret); |
|---|
| 602 | | -} |
|---|
| 708 | + } |
|---|
| 709 | + |
|---|
| 603 | 710 | ida_init(&fg->fte_allocator); |
|---|
| 604 | 711 | fg->mask.match_criteria_enable = match_criteria_enable; |
|---|
| 605 | 712 | memcpy(&fg->mask.match_criteria, match_criteria, |
|---|
| .. | .. |
|---|
| 613 | 720 | |
|---|
| 614 | 721 | static struct mlx5_flow_group *alloc_insert_flow_group(struct mlx5_flow_table *ft, |
|---|
| 615 | 722 | u8 match_criteria_enable, |
|---|
| 616 | | - void *match_criteria, |
|---|
| 723 | + const void *match_criteria, |
|---|
| 617 | 724 | int start_index, |
|---|
| 618 | 725 | int end_index, |
|---|
| 619 | 726 | struct list_head *prev) |
|---|
| .. | .. |
|---|
| 711 | 818 | return ft; |
|---|
| 712 | 819 | } |
|---|
| 713 | 820 | |
|---|
| 714 | | -/* If reverse if false then return the first flow table in next priority of |
|---|
| 715 | | - * prio in the tree, else return the last flow table in the previous priority |
|---|
| 716 | | - * of prio in the tree. |
|---|
| 717 | | - */ |
|---|
| 718 | | -static struct mlx5_flow_table *find_closest_ft(struct fs_prio *prio, bool reverse) |
|---|
| 821 | +static struct fs_node *find_prio_chains_parent(struct fs_node *parent, |
|---|
| 822 | + struct fs_node **child) |
|---|
| 719 | 823 | { |
|---|
| 824 | + struct fs_node *node = NULL; |
|---|
| 825 | + |
|---|
| 826 | + while (parent && parent->type != FS_TYPE_PRIO_CHAINS) { |
|---|
| 827 | + node = parent; |
|---|
| 828 | + parent = parent->parent; |
|---|
| 829 | + } |
|---|
| 830 | + |
|---|
| 831 | + if (child) |
|---|
| 832 | + *child = node; |
|---|
| 833 | + |
|---|
| 834 | + return parent; |
|---|
| 835 | +} |
|---|
| 836 | + |
|---|
| 837 | +/* If reverse is false then return the first flow table next to the passed node |
|---|
| 838 | + * in the tree, else return the last flow table before the node in the tree. |
|---|
| 839 | + * If skip is true, skip the flow tables in the same prio_chains prio. |
|---|
| 840 | + */ |
|---|
| 841 | +static struct mlx5_flow_table *find_closest_ft(struct fs_node *node, bool reverse, |
|---|
| 842 | + bool skip) |
|---|
| 843 | +{ |
|---|
| 844 | + struct fs_node *prio_chains_parent = NULL; |
|---|
| 720 | 845 | struct mlx5_flow_table *ft = NULL; |
|---|
| 721 | 846 | struct fs_node *curr_node; |
|---|
| 722 | 847 | struct fs_node *parent; |
|---|
| 723 | 848 | |
|---|
| 724 | | - parent = prio->node.parent; |
|---|
| 725 | | - curr_node = &prio->node; |
|---|
| 849 | + if (skip) |
|---|
| 850 | + prio_chains_parent = find_prio_chains_parent(node, NULL); |
|---|
| 851 | + parent = node->parent; |
|---|
| 852 | + curr_node = node; |
|---|
| 726 | 853 | while (!ft && parent) { |
|---|
| 727 | | - ft = find_closest_ft_recursive(parent, &curr_node->list, reverse); |
|---|
| 854 | + if (parent != prio_chains_parent) |
|---|
| 855 | + ft = find_closest_ft_recursive(parent, &curr_node->list, |
|---|
| 856 | + reverse); |
|---|
| 728 | 857 | curr_node = parent; |
|---|
| 729 | 858 | parent = curr_node->parent; |
|---|
| 730 | 859 | } |
|---|
| .. | .. |
|---|
| 732 | 861 | } |
|---|
| 733 | 862 | |
|---|
| 734 | 863 | /* Assuming all the tree is locked by mutex chain lock */ |
|---|
| 735 | | -static struct mlx5_flow_table *find_next_chained_ft(struct fs_prio *prio) |
|---|
| 864 | +static struct mlx5_flow_table *find_next_chained_ft(struct fs_node *node) |
|---|
| 736 | 865 | { |
|---|
| 737 | | - return find_closest_ft(prio, false); |
|---|
| 866 | + return find_closest_ft(node, false, true); |
|---|
| 738 | 867 | } |
|---|
| 739 | 868 | |
|---|
| 740 | 869 | /* Assuming all the tree is locked by mutex chain lock */ |
|---|
| 741 | | -static struct mlx5_flow_table *find_prev_chained_ft(struct fs_prio *prio) |
|---|
| 870 | +static struct mlx5_flow_table *find_prev_chained_ft(struct fs_node *node) |
|---|
| 742 | 871 | { |
|---|
| 743 | | - return find_closest_ft(prio, true); |
|---|
| 872 | + return find_closest_ft(node, true, true); |
|---|
| 873 | +} |
|---|
| 874 | + |
|---|
| 875 | +static struct mlx5_flow_table *find_next_fwd_ft(struct mlx5_flow_table *ft, |
|---|
| 876 | + struct mlx5_flow_act *flow_act) |
|---|
| 877 | +{ |
|---|
| 878 | + struct fs_prio *prio; |
|---|
| 879 | + bool next_ns; |
|---|
| 880 | + |
|---|
| 881 | + next_ns = flow_act->action & MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_NS; |
|---|
| 882 | + fs_get_obj(prio, next_ns ? ft->ns->node.parent : ft->node.parent); |
|---|
| 883 | + |
|---|
| 884 | + return find_next_chained_ft(&prio->node); |
|---|
| 744 | 885 | } |
|---|
| 745 | 886 | |
|---|
| 746 | 887 | static int connect_fts_in_prio(struct mlx5_core_dev *dev, |
|---|
| .. | .. |
|---|
| 749 | 890 | { |
|---|
| 750 | 891 | struct mlx5_flow_root_namespace *root = find_root(&prio->node); |
|---|
| 751 | 892 | struct mlx5_flow_table *iter; |
|---|
| 752 | | - int i = 0; |
|---|
| 753 | 893 | int err; |
|---|
| 754 | 894 | |
|---|
| 755 | 895 | fs_for_each_ft(iter, prio) { |
|---|
| 756 | | - i++; |
|---|
| 757 | | - err = root->cmds->modify_flow_table(dev, iter, ft); |
|---|
| 896 | + err = root->cmds->modify_flow_table(root, iter, ft); |
|---|
| 758 | 897 | if (err) { |
|---|
| 759 | | - mlx5_core_warn(dev, "Failed to modify flow table %d\n", |
|---|
| 760 | | - iter->id); |
|---|
| 898 | + mlx5_core_err(dev, |
|---|
| 899 | + "Failed to modify flow table id %d, type %d, err %d\n", |
|---|
| 900 | + iter->id, iter->type, err); |
|---|
| 761 | 901 | /* The driver is out of sync with the FW */ |
|---|
| 762 | | - if (i > 1) |
|---|
| 763 | | - WARN_ON(true); |
|---|
| 764 | 902 | return err; |
|---|
| 765 | 903 | } |
|---|
| 766 | 904 | } |
|---|
| 767 | 905 | return 0; |
|---|
| 906 | +} |
|---|
| 907 | + |
|---|
| 908 | +static struct mlx5_flow_table *find_closet_ft_prio_chains(struct fs_node *node, |
|---|
| 909 | + struct fs_node *parent, |
|---|
| 910 | + struct fs_node **child, |
|---|
| 911 | + bool reverse) |
|---|
| 912 | +{ |
|---|
| 913 | + struct mlx5_flow_table *ft; |
|---|
| 914 | + |
|---|
| 915 | + ft = find_closest_ft(node, reverse, false); |
|---|
| 916 | + |
|---|
| 917 | + if (ft && parent == find_prio_chains_parent(&ft->node, child)) |
|---|
| 918 | + return ft; |
|---|
| 919 | + |
|---|
| 920 | + return NULL; |
|---|
| 768 | 921 | } |
|---|
| 769 | 922 | |
|---|
| 770 | 923 | /* Connect flow tables from previous priority of prio to ft */ |
|---|
| .. | .. |
|---|
| 772 | 925 | struct mlx5_flow_table *ft, |
|---|
| 773 | 926 | struct fs_prio *prio) |
|---|
| 774 | 927 | { |
|---|
| 928 | + struct fs_node *prio_parent, *parent = NULL, *child, *node; |
|---|
| 775 | 929 | struct mlx5_flow_table *prev_ft; |
|---|
| 930 | + int err = 0; |
|---|
| 776 | 931 | |
|---|
| 777 | | - prev_ft = find_prev_chained_ft(prio); |
|---|
| 778 | | - if (prev_ft) { |
|---|
| 932 | + prio_parent = find_prio_chains_parent(&prio->node, &child); |
|---|
| 933 | + |
|---|
| 934 | + /* return directly if not under the first sub ns of prio_chains prio */ |
|---|
| 935 | + if (prio_parent && !list_is_first(&child->list, &prio_parent->children)) |
|---|
| 936 | + return 0; |
|---|
| 937 | + |
|---|
| 938 | + prev_ft = find_prev_chained_ft(&prio->node); |
|---|
| 939 | + while (prev_ft) { |
|---|
| 779 | 940 | struct fs_prio *prev_prio; |
|---|
| 780 | 941 | |
|---|
| 781 | 942 | fs_get_obj(prev_prio, prev_ft->node.parent); |
|---|
| 782 | | - return connect_fts_in_prio(dev, prev_prio, ft); |
|---|
| 943 | + err = connect_fts_in_prio(dev, prev_prio, ft); |
|---|
| 944 | + if (err) |
|---|
| 945 | + break; |
|---|
| 946 | + |
|---|
| 947 | + if (!parent) { |
|---|
| 948 | + parent = find_prio_chains_parent(&prev_prio->node, &child); |
|---|
| 949 | + if (!parent) |
|---|
| 950 | + break; |
|---|
| 951 | + } |
|---|
| 952 | + |
|---|
| 953 | + node = child; |
|---|
| 954 | + prev_ft = find_closet_ft_prio_chains(node, parent, &child, true); |
|---|
| 783 | 955 | } |
|---|
| 784 | | - return 0; |
|---|
| 956 | + return err; |
|---|
| 785 | 957 | } |
|---|
| 786 | 958 | |
|---|
| 787 | 959 | static int update_root_ft_create(struct mlx5_flow_table *ft, struct fs_prio |
|---|
| .. | .. |
|---|
| 790 | 962 | struct mlx5_flow_root_namespace *root = find_root(&prio->node); |
|---|
| 791 | 963 | struct mlx5_ft_underlay_qp *uqp; |
|---|
| 792 | 964 | int min_level = INT_MAX; |
|---|
| 793 | | - int err; |
|---|
| 965 | + int err = 0; |
|---|
| 794 | 966 | u32 qpn; |
|---|
| 795 | 967 | |
|---|
| 796 | 968 | if (root->root_ft) |
|---|
| .. | .. |
|---|
| 802 | 974 | if (list_empty(&root->underlay_qpns)) { |
|---|
| 803 | 975 | /* Don't set any QPN (zero) in case QPN list is empty */ |
|---|
| 804 | 976 | qpn = 0; |
|---|
| 805 | | - err = root->cmds->update_root_ft(root->dev, ft, qpn, false); |
|---|
| 977 | + err = root->cmds->update_root_ft(root, ft, qpn, false); |
|---|
| 806 | 978 | } else { |
|---|
| 807 | 979 | list_for_each_entry(uqp, &root->underlay_qpns, list) { |
|---|
| 808 | 980 | qpn = uqp->qpn; |
|---|
| 809 | | - err = root->cmds->update_root_ft(root->dev, ft, |
|---|
| 981 | + err = root->cmds->update_root_ft(root, ft, |
|---|
| 810 | 982 | qpn, false); |
|---|
| 811 | 983 | if (err) |
|---|
| 812 | 984 | break; |
|---|
| .. | .. |
|---|
| 836 | 1008 | fs_get_obj(fte, rule->node.parent); |
|---|
| 837 | 1009 | if (!(fte->action.action & MLX5_FLOW_CONTEXT_ACTION_FWD_DEST)) |
|---|
| 838 | 1010 | return -EINVAL; |
|---|
| 839 | | - down_write_ref_node(&fte->node); |
|---|
| 1011 | + down_write_ref_node(&fte->node, false); |
|---|
| 840 | 1012 | fs_get_obj(fg, fte->node.parent); |
|---|
| 841 | 1013 | fs_get_obj(ft, fg->node.parent); |
|---|
| 842 | 1014 | |
|---|
| 843 | 1015 | memcpy(&rule->dest_attr, dest, sizeof(*dest)); |
|---|
| 844 | 1016 | root = find_root(&ft->node); |
|---|
| 845 | | - err = root->cmds->update_fte(get_dev(&ft->node), ft, fg->id, |
|---|
| 1017 | + err = root->cmds->update_fte(root, ft, fg, |
|---|
| 846 | 1018 | modify_mask, fte); |
|---|
| 847 | | - up_write_ref_node(&fte->node); |
|---|
| 1019 | + up_write_ref_node(&fte->node, false); |
|---|
| 848 | 1020 | |
|---|
| 849 | 1021 | return err; |
|---|
| 850 | 1022 | } |
|---|
| .. | .. |
|---|
| 893 | 1065 | list_splice_init(&old_next_ft->fwd_rules, &new_next_ft->fwd_rules); |
|---|
| 894 | 1066 | mutex_unlock(&old_next_ft->lock); |
|---|
| 895 | 1067 | list_for_each_entry(iter, &new_next_ft->fwd_rules, next_ft) { |
|---|
| 1068 | + if ((iter->sw_action & MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_NS) && |
|---|
| 1069 | + iter->ft->ns == new_next_ft->ns) |
|---|
| 1070 | + continue; |
|---|
| 1071 | + |
|---|
| 896 | 1072 | err = _mlx5_modify_rule_destination(iter, &dest); |
|---|
| 897 | 1073 | if (err) |
|---|
| 898 | 1074 | pr_err("mlx5_core: failed to modify rule to point on flow table %d\n", |
|---|
| .. | .. |
|---|
| 916 | 1092 | if (err) |
|---|
| 917 | 1093 | return err; |
|---|
| 918 | 1094 | |
|---|
| 919 | | - next_ft = first_ft ? first_ft : find_next_chained_ft(prio); |
|---|
| 1095 | + next_ft = first_ft ? first_ft : find_next_chained_ft(&prio->node); |
|---|
| 920 | 1096 | err = connect_fwd_rules(dev, ft, next_ft); |
|---|
| 921 | 1097 | if (err) |
|---|
| 922 | 1098 | return err; |
|---|
| .. | .. |
|---|
| 948 | 1124 | u16 vport) |
|---|
| 949 | 1125 | { |
|---|
| 950 | 1126 | struct mlx5_flow_root_namespace *root = find_root(&ns->node); |
|---|
| 951 | | - struct mlx5_flow_table *next_ft = NULL; |
|---|
| 1127 | + bool unmanaged = ft_attr->flags & MLX5_FLOW_TABLE_UNMANAGED; |
|---|
| 1128 | + struct mlx5_flow_table *next_ft; |
|---|
| 952 | 1129 | struct fs_prio *fs_prio = NULL; |
|---|
| 953 | 1130 | struct mlx5_flow_table *ft; |
|---|
| 954 | 1131 | int log_table_sz; |
|---|
| .. | .. |
|---|
| 965 | 1142 | err = -EINVAL; |
|---|
| 966 | 1143 | goto unlock_root; |
|---|
| 967 | 1144 | } |
|---|
| 968 | | - if (ft_attr->level >= fs_prio->num_levels) { |
|---|
| 969 | | - err = -ENOSPC; |
|---|
| 970 | | - goto unlock_root; |
|---|
| 1145 | + if (!unmanaged) { |
|---|
| 1146 | + /* The level is related to the |
|---|
| 1147 | + * priority level range. |
|---|
| 1148 | + */ |
|---|
| 1149 | + if (ft_attr->level >= fs_prio->num_levels) { |
|---|
| 1150 | + err = -ENOSPC; |
|---|
| 1151 | + goto unlock_root; |
|---|
| 1152 | + } |
|---|
| 1153 | + |
|---|
| 1154 | + ft_attr->level += fs_prio->start_level; |
|---|
| 971 | 1155 | } |
|---|
| 1156 | + |
|---|
| 972 | 1157 | /* The level is related to the |
|---|
| 973 | 1158 | * priority level range. |
|---|
| 974 | 1159 | */ |
|---|
| 975 | | - ft_attr->level += fs_prio->start_level; |
|---|
| 976 | 1160 | ft = alloc_flow_table(ft_attr->level, |
|---|
| 977 | 1161 | vport, |
|---|
| 978 | 1162 | ft_attr->max_fte ? roundup_pow_of_two(ft_attr->max_fte) : 0, |
|---|
| .. | .. |
|---|
| 985 | 1169 | |
|---|
| 986 | 1170 | tree_init_node(&ft->node, del_hw_flow_table, del_sw_flow_table); |
|---|
| 987 | 1171 | log_table_sz = ft->max_fte ? ilog2(ft->max_fte) : 0; |
|---|
| 988 | | - next_ft = find_next_chained_ft(fs_prio); |
|---|
| 989 | | - err = root->cmds->create_flow_table(root->dev, ft->vport, ft->op_mod, |
|---|
| 990 | | - ft->type, ft->level, log_table_sz, |
|---|
| 991 | | - next_ft, &ft->id, ft->flags); |
|---|
| 1172 | + next_ft = unmanaged ? ft_attr->next_ft : |
|---|
| 1173 | + find_next_chained_ft(&fs_prio->node); |
|---|
| 1174 | + ft->def_miss_action = ns->def_miss_action; |
|---|
| 1175 | + ft->ns = ns; |
|---|
| 1176 | + err = root->cmds->create_flow_table(root, ft, log_table_sz, next_ft); |
|---|
| 992 | 1177 | if (err) |
|---|
| 993 | 1178 | goto free_ft; |
|---|
| 994 | 1179 | |
|---|
| 995 | | - err = connect_flow_table(root->dev, ft, fs_prio); |
|---|
| 996 | | - if (err) |
|---|
| 997 | | - goto destroy_ft; |
|---|
| 1180 | + if (!unmanaged) { |
|---|
| 1181 | + err = connect_flow_table(root->dev, ft, fs_prio); |
|---|
| 1182 | + if (err) |
|---|
| 1183 | + goto destroy_ft; |
|---|
| 1184 | + } |
|---|
| 1185 | + |
|---|
| 998 | 1186 | ft->node.active = true; |
|---|
| 999 | | - down_write_ref_node(&fs_prio->node); |
|---|
| 1000 | | - tree_add_node(&ft->node, &fs_prio->node); |
|---|
| 1001 | | - list_add_flow_table(ft, fs_prio); |
|---|
| 1187 | + down_write_ref_node(&fs_prio->node, false); |
|---|
| 1188 | + if (!unmanaged) { |
|---|
| 1189 | + tree_add_node(&ft->node, &fs_prio->node); |
|---|
| 1190 | + list_add_flow_table(ft, fs_prio); |
|---|
| 1191 | + } else { |
|---|
| 1192 | + ft->node.root = fs_prio->node.root; |
|---|
| 1193 | + } |
|---|
| 1002 | 1194 | fs_prio->num_ft++; |
|---|
| 1003 | | - up_write_ref_node(&fs_prio->node); |
|---|
| 1195 | + up_write_ref_node(&fs_prio->node, false); |
|---|
| 1004 | 1196 | mutex_unlock(&root->chain_lock); |
|---|
| 1197 | + trace_mlx5_fs_add_ft(ft); |
|---|
| 1005 | 1198 | return ft; |
|---|
| 1006 | 1199 | destroy_ft: |
|---|
| 1007 | | - root->cmds->destroy_flow_table(root->dev, ft); |
|---|
| 1200 | + root->cmds->destroy_flow_table(root, ft); |
|---|
| 1008 | 1201 | free_ft: |
|---|
| 1009 | 1202 | rhltable_destroy(&ft->fgs_hash); |
|---|
| 1010 | 1203 | kfree(ft); |
|---|
| .. | .. |
|---|
| 1046 | 1239 | |
|---|
| 1047 | 1240 | struct mlx5_flow_table* |
|---|
| 1048 | 1241 | mlx5_create_auto_grouped_flow_table(struct mlx5_flow_namespace *ns, |
|---|
| 1049 | | - int prio, |
|---|
| 1050 | | - int num_flow_table_entries, |
|---|
| 1051 | | - int max_num_groups, |
|---|
| 1052 | | - u32 level, |
|---|
| 1053 | | - u32 flags) |
|---|
| 1242 | + struct mlx5_flow_table_attr *ft_attr) |
|---|
| 1054 | 1243 | { |
|---|
| 1055 | | - struct mlx5_flow_table_attr ft_attr = {}; |
|---|
| 1244 | + int num_reserved_entries = ft_attr->autogroup.num_reserved_entries; |
|---|
| 1245 | + int autogroups_max_fte = ft_attr->max_fte - num_reserved_entries; |
|---|
| 1246 | + int max_num_groups = ft_attr->autogroup.max_num_groups; |
|---|
| 1056 | 1247 | struct mlx5_flow_table *ft; |
|---|
| 1057 | 1248 | |
|---|
| 1058 | | - if (max_num_groups > num_flow_table_entries) |
|---|
| 1249 | + if (max_num_groups > autogroups_max_fte) |
|---|
| 1250 | + return ERR_PTR(-EINVAL); |
|---|
| 1251 | + if (num_reserved_entries > ft_attr->max_fte) |
|---|
| 1059 | 1252 | return ERR_PTR(-EINVAL); |
|---|
| 1060 | 1253 | |
|---|
| 1061 | | - ft_attr.max_fte = num_flow_table_entries; |
|---|
| 1062 | | - ft_attr.prio = prio; |
|---|
| 1063 | | - ft_attr.level = level; |
|---|
| 1064 | | - ft_attr.flags = flags; |
|---|
| 1065 | | - |
|---|
| 1066 | | - ft = mlx5_create_flow_table(ns, &ft_attr); |
|---|
| 1254 | + ft = mlx5_create_flow_table(ns, ft_attr); |
|---|
| 1067 | 1255 | if (IS_ERR(ft)) |
|---|
| 1068 | 1256 | return ft; |
|---|
| 1069 | 1257 | |
|---|
| 1070 | 1258 | ft->autogroup.active = true; |
|---|
| 1071 | 1259 | ft->autogroup.required_groups = max_num_groups; |
|---|
| 1260 | + ft->autogroup.max_fte = autogroups_max_fte; |
|---|
| 1072 | 1261 | /* We save place for flow groups in addition to max types */ |
|---|
| 1073 | | - ft->autogroup.group_size = ft->max_fte / (max_num_groups + 1); |
|---|
| 1262 | + ft->autogroup.group_size = autogroups_max_fte / (max_num_groups + 1); |
|---|
| 1074 | 1263 | |
|---|
| 1075 | 1264 | return ft; |
|---|
| 1076 | 1265 | } |
|---|
| .. | .. |
|---|
| 1089 | 1278 | start_flow_index); |
|---|
| 1090 | 1279 | int end_index = MLX5_GET(create_flow_group_in, fg_in, |
|---|
| 1091 | 1280 | end_flow_index); |
|---|
| 1092 | | - struct mlx5_core_dev *dev = get_dev(&ft->node); |
|---|
| 1093 | 1281 | struct mlx5_flow_group *fg; |
|---|
| 1094 | 1282 | int err; |
|---|
| 1095 | 1283 | |
|---|
| 1096 | | - if (ft->autogroup.active) |
|---|
| 1284 | + if (ft->autogroup.active && start_index < ft->autogroup.max_fte) |
|---|
| 1097 | 1285 | return ERR_PTR(-EPERM); |
|---|
| 1098 | 1286 | |
|---|
| 1099 | | - down_write_ref_node(&ft->node); |
|---|
| 1287 | + down_write_ref_node(&ft->node, false); |
|---|
| 1100 | 1288 | fg = alloc_insert_flow_group(ft, match_criteria_enable, match_criteria, |
|---|
| 1101 | 1289 | start_index, end_index, |
|---|
| 1102 | 1290 | ft->node.children.prev); |
|---|
| 1103 | | - up_write_ref_node(&ft->node); |
|---|
| 1291 | + up_write_ref_node(&ft->node, false); |
|---|
| 1104 | 1292 | if (IS_ERR(fg)) |
|---|
| 1105 | 1293 | return fg; |
|---|
| 1106 | 1294 | |
|---|
| 1107 | | - err = root->cmds->create_flow_group(dev, ft, fg_in, &fg->id); |
|---|
| 1295 | + err = root->cmds->create_flow_group(root, ft, fg_in, fg); |
|---|
| 1108 | 1296 | if (err) { |
|---|
| 1109 | | - tree_put_node(&fg->node); |
|---|
| 1297 | + tree_put_node(&fg->node, false); |
|---|
| 1110 | 1298 | return ERR_PTR(err); |
|---|
| 1111 | 1299 | } |
|---|
| 1112 | 1300 | trace_mlx5_fs_add_fg(fg); |
|---|
| .. | .. |
|---|
| 1244 | 1432 | fs_get_obj(ft, fg->node.parent); |
|---|
| 1245 | 1433 | root = find_root(&fg->node); |
|---|
| 1246 | 1434 | if (!(fte->status & FS_FTE_STATUS_EXISTING)) |
|---|
| 1247 | | - err = root->cmds->create_fte(get_dev(&ft->node), |
|---|
| 1248 | | - ft, fg, fte); |
|---|
| 1435 | + err = root->cmds->create_fte(root, ft, fg, fte); |
|---|
| 1249 | 1436 | else |
|---|
| 1250 | | - err = root->cmds->update_fte(get_dev(&ft->node), ft, fg->id, |
|---|
| 1251 | | - modify_mask, fte); |
|---|
| 1437 | + err = root->cmds->update_fte(root, ft, fg, modify_mask, fte); |
|---|
| 1252 | 1438 | if (err) |
|---|
| 1253 | 1439 | goto free_handle; |
|---|
| 1254 | 1440 | |
|---|
| 1255 | 1441 | fte->node.active = true; |
|---|
| 1256 | 1442 | fte->status |= FS_FTE_STATUS_EXISTING; |
|---|
| 1257 | | - atomic_inc(&fte->node.version); |
|---|
| 1443 | + atomic_inc(&fg->node.version); |
|---|
| 1258 | 1444 | |
|---|
| 1259 | 1445 | out: |
|---|
| 1260 | 1446 | return handle; |
|---|
| .. | .. |
|---|
| 1265 | 1451 | } |
|---|
| 1266 | 1452 | |
|---|
| 1267 | 1453 | static struct mlx5_flow_group *alloc_auto_flow_group(struct mlx5_flow_table *ft, |
|---|
| 1268 | | - struct mlx5_flow_spec *spec) |
|---|
| 1454 | + const struct mlx5_flow_spec *spec) |
|---|
| 1269 | 1455 | { |
|---|
| 1270 | 1456 | struct list_head *prev = &ft->node.children; |
|---|
| 1271 | | - struct mlx5_flow_group *fg; |
|---|
| 1457 | + u32 max_fte = ft->autogroup.max_fte; |
|---|
| 1272 | 1458 | unsigned int candidate_index = 0; |
|---|
| 1273 | 1459 | unsigned int group_size = 0; |
|---|
| 1460 | + struct mlx5_flow_group *fg; |
|---|
| 1274 | 1461 | |
|---|
| 1275 | 1462 | if (!ft->autogroup.active) |
|---|
| 1276 | 1463 | return ERR_PTR(-ENOENT); |
|---|
| .. | .. |
|---|
| 1278 | 1465 | if (ft->autogroup.num_groups < ft->autogroup.required_groups) |
|---|
| 1279 | 1466 | group_size = ft->autogroup.group_size; |
|---|
| 1280 | 1467 | |
|---|
| 1281 | | - /* ft->max_fte == ft->autogroup.max_types */ |
|---|
| 1468 | + /* max_fte == ft->autogroup.max_types */ |
|---|
| 1282 | 1469 | if (group_size == 0) |
|---|
| 1283 | 1470 | group_size = 1; |
|---|
| 1284 | 1471 | |
|---|
| .. | .. |
|---|
| 1291 | 1478 | prev = &fg->node.list; |
|---|
| 1292 | 1479 | } |
|---|
| 1293 | 1480 | |
|---|
| 1294 | | - if (candidate_index + group_size > ft->max_fte) |
|---|
| 1481 | + if (candidate_index + group_size > max_fte) |
|---|
| 1295 | 1482 | return ERR_PTR(-ENOSPC); |
|---|
| 1296 | 1483 | |
|---|
| 1297 | 1484 | fg = alloc_insert_flow_group(ft, |
|---|
| .. | .. |
|---|
| 1314 | 1501 | struct mlx5_flow_group *fg) |
|---|
| 1315 | 1502 | { |
|---|
| 1316 | 1503 | struct mlx5_flow_root_namespace *root = find_root(&ft->node); |
|---|
| 1317 | | - struct mlx5_core_dev *dev = get_dev(&ft->node); |
|---|
| 1318 | 1504 | int inlen = MLX5_ST_SZ_BYTES(create_flow_group_in); |
|---|
| 1319 | 1505 | void *match_criteria_addr; |
|---|
| 1320 | 1506 | u8 src_esw_owner_mask_on; |
|---|
| .. | .. |
|---|
| 1344 | 1530 | memcpy(match_criteria_addr, fg->mask.match_criteria, |
|---|
| 1345 | 1531 | sizeof(fg->mask.match_criteria)); |
|---|
| 1346 | 1532 | |
|---|
| 1347 | | - err = root->cmds->create_flow_group(dev, ft, in, &fg->id); |
|---|
| 1533 | + err = root->cmds->create_flow_group(root, ft, in, fg); |
|---|
| 1348 | 1534 | if (!err) { |
|---|
| 1349 | 1535 | fg->node.active = true; |
|---|
| 1350 | 1536 | trace_mlx5_fs_add_fg(fg); |
|---|
| .. | .. |
|---|
| 1359 | 1545 | { |
|---|
| 1360 | 1546 | if (d1->type == d2->type) { |
|---|
| 1361 | 1547 | if ((d1->type == MLX5_FLOW_DESTINATION_TYPE_VPORT && |
|---|
| 1362 | | - d1->vport.num == d2->vport.num) || |
|---|
| 1548 | + d1->vport.num == d2->vport.num && |
|---|
| 1549 | + d1->vport.flags == d2->vport.flags && |
|---|
| 1550 | + ((d1->vport.flags & MLX5_FLOW_DEST_VPORT_VHCA_ID) ? |
|---|
| 1551 | + (d1->vport.vhca_id == d2->vport.vhca_id) : true) && |
|---|
| 1552 | + ((d1->vport.flags & MLX5_FLOW_DEST_VPORT_REFORMAT_ID) ? |
|---|
| 1553 | + (d1->vport.pkt_reformat->id == |
|---|
| 1554 | + d2->vport.pkt_reformat->id) : true)) || |
|---|
| 1363 | 1555 | (d1->type == MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE && |
|---|
| 1364 | 1556 | d1->ft == d2->ft) || |
|---|
| 1365 | 1557 | (d1->type == MLX5_FLOW_DESTINATION_TYPE_TIR && |
|---|
| .. | .. |
|---|
| 1384 | 1576 | return NULL; |
|---|
| 1385 | 1577 | } |
|---|
| 1386 | 1578 | |
|---|
| 1387 | | -static bool check_conflicting_actions(u32 action1, u32 action2) |
|---|
| 1579 | +static bool check_conflicting_actions_vlan(const struct mlx5_fs_vlan *vlan0, |
|---|
| 1580 | + const struct mlx5_fs_vlan *vlan1) |
|---|
| 1388 | 1581 | { |
|---|
| 1389 | | - u32 xored_actions = action1 ^ action2; |
|---|
| 1582 | + return vlan0->ethtype != vlan1->ethtype || |
|---|
| 1583 | + vlan0->vid != vlan1->vid || |
|---|
| 1584 | + vlan0->prio != vlan1->prio; |
|---|
| 1585 | +} |
|---|
| 1586 | + |
|---|
| 1587 | +static bool check_conflicting_actions(const struct mlx5_flow_act *act1, |
|---|
| 1588 | + const struct mlx5_flow_act *act2) |
|---|
| 1589 | +{ |
|---|
| 1590 | + u32 action1 = act1->action; |
|---|
| 1591 | + u32 action2 = act2->action; |
|---|
| 1592 | + u32 xored_actions; |
|---|
| 1593 | + |
|---|
| 1594 | + xored_actions = action1 ^ action2; |
|---|
| 1390 | 1595 | |
|---|
| 1391 | 1596 | /* if one rule only wants to count, it's ok */ |
|---|
| 1392 | 1597 | if (action1 == MLX5_FLOW_CONTEXT_ACTION_COUNT || |
|---|
| .. | .. |
|---|
| 1394 | 1599 | return false; |
|---|
| 1395 | 1600 | |
|---|
| 1396 | 1601 | if (xored_actions & (MLX5_FLOW_CONTEXT_ACTION_DROP | |
|---|
| 1397 | | - MLX5_FLOW_CONTEXT_ACTION_ENCAP | |
|---|
| 1602 | + MLX5_FLOW_CONTEXT_ACTION_PACKET_REFORMAT | |
|---|
| 1398 | 1603 | MLX5_FLOW_CONTEXT_ACTION_DECAP | |
|---|
| 1399 | 1604 | MLX5_FLOW_CONTEXT_ACTION_MOD_HDR | |
|---|
| 1400 | 1605 | MLX5_FLOW_CONTEXT_ACTION_VLAN_POP | |
|---|
| .. | .. |
|---|
| 1403 | 1608 | MLX5_FLOW_CONTEXT_ACTION_VLAN_PUSH_2)) |
|---|
| 1404 | 1609 | return true; |
|---|
| 1405 | 1610 | |
|---|
| 1611 | + if (action1 & MLX5_FLOW_CONTEXT_ACTION_PACKET_REFORMAT && |
|---|
| 1612 | + act1->pkt_reformat != act2->pkt_reformat) |
|---|
| 1613 | + return true; |
|---|
| 1614 | + |
|---|
| 1615 | + if (action1 & MLX5_FLOW_CONTEXT_ACTION_MOD_HDR && |
|---|
| 1616 | + act1->modify_hdr != act2->modify_hdr) |
|---|
| 1617 | + return true; |
|---|
| 1618 | + |
|---|
| 1619 | + if (action1 & MLX5_FLOW_CONTEXT_ACTION_VLAN_PUSH && |
|---|
| 1620 | + check_conflicting_actions_vlan(&act1->vlan[0], &act2->vlan[0])) |
|---|
| 1621 | + return true; |
|---|
| 1622 | + |
|---|
| 1623 | + if (action1 & MLX5_FLOW_CONTEXT_ACTION_VLAN_PUSH_2 && |
|---|
| 1624 | + check_conflicting_actions_vlan(&act1->vlan[1], &act2->vlan[1])) |
|---|
| 1625 | + return true; |
|---|
| 1626 | + |
|---|
| 1406 | 1627 | return false; |
|---|
| 1407 | 1628 | } |
|---|
| 1408 | 1629 | |
|---|
| 1409 | | -static int check_conflicting_ftes(struct fs_fte *fte, const struct mlx5_flow_act *flow_act) |
|---|
| 1630 | +static int check_conflicting_ftes(struct fs_fte *fte, |
|---|
| 1631 | + const struct mlx5_flow_context *flow_context, |
|---|
| 1632 | + const struct mlx5_flow_act *flow_act) |
|---|
| 1410 | 1633 | { |
|---|
| 1411 | | - if (check_conflicting_actions(flow_act->action, fte->action.action)) { |
|---|
| 1634 | + if (check_conflicting_actions(flow_act, &fte->action)) { |
|---|
| 1412 | 1635 | mlx5_core_warn(get_dev(&fte->node), |
|---|
| 1413 | 1636 | "Found two FTEs with conflicting actions\n"); |
|---|
| 1414 | 1637 | return -EEXIST; |
|---|
| 1415 | 1638 | } |
|---|
| 1416 | 1639 | |
|---|
| 1417 | | - if (flow_act->has_flow_tag && |
|---|
| 1418 | | - fte->action.flow_tag != flow_act->flow_tag) { |
|---|
| 1640 | + if ((flow_context->flags & FLOW_CONTEXT_HAS_TAG) && |
|---|
| 1641 | + fte->flow_context.flow_tag != flow_context->flow_tag) { |
|---|
| 1419 | 1642 | mlx5_core_warn(get_dev(&fte->node), |
|---|
| 1420 | 1643 | "FTE flow tag %u already exists with different flow tag %u\n", |
|---|
| 1421 | | - fte->action.flow_tag, |
|---|
| 1422 | | - flow_act->flow_tag); |
|---|
| 1644 | + fte->flow_context.flow_tag, |
|---|
| 1645 | + flow_context->flow_tag); |
|---|
| 1423 | 1646 | return -EEXIST; |
|---|
| 1424 | 1647 | } |
|---|
| 1425 | 1648 | |
|---|
| .. | .. |
|---|
| 1427 | 1650 | } |
|---|
| 1428 | 1651 | |
|---|
| 1429 | 1652 | static struct mlx5_flow_handle *add_rule_fg(struct mlx5_flow_group *fg, |
|---|
| 1430 | | - u32 *match_value, |
|---|
| 1653 | + const struct mlx5_flow_spec *spec, |
|---|
| 1431 | 1654 | struct mlx5_flow_act *flow_act, |
|---|
| 1432 | 1655 | struct mlx5_flow_destination *dest, |
|---|
| 1433 | 1656 | int dest_num, |
|---|
| .. | .. |
|---|
| 1438 | 1661 | int i; |
|---|
| 1439 | 1662 | int ret; |
|---|
| 1440 | 1663 | |
|---|
| 1441 | | - ret = check_conflicting_ftes(fte, flow_act); |
|---|
| 1664 | + ret = check_conflicting_ftes(fte, &spec->flow_context, flow_act); |
|---|
| 1442 | 1665 | if (ret) |
|---|
| 1443 | 1666 | return ERR_PTR(ret); |
|---|
| 1444 | 1667 | |
|---|
| .. | .. |
|---|
| 1461 | 1684 | return handle; |
|---|
| 1462 | 1685 | } |
|---|
| 1463 | 1686 | |
|---|
| 1464 | | -struct mlx5_fc *mlx5_flow_rule_counter(struct mlx5_flow_handle *handle) |
|---|
| 1687 | +static bool counter_is_valid(u32 action) |
|---|
| 1465 | 1688 | { |
|---|
| 1466 | | - struct mlx5_flow_rule *dst; |
|---|
| 1467 | | - struct fs_fte *fte; |
|---|
| 1468 | | - |
|---|
| 1469 | | - fs_get_obj(fte, handle->rule[0]->node.parent); |
|---|
| 1470 | | - |
|---|
| 1471 | | - fs_for_each_dst(dst, fte) { |
|---|
| 1472 | | - if (dst->dest_attr.type == MLX5_FLOW_DESTINATION_TYPE_COUNTER) |
|---|
| 1473 | | - return dst->dest_attr.counter; |
|---|
| 1474 | | - } |
|---|
| 1475 | | - |
|---|
| 1476 | | - return NULL; |
|---|
| 1477 | | -} |
|---|
| 1478 | | - |
|---|
| 1479 | | -static bool counter_is_valid(struct mlx5_fc *counter, u32 action) |
|---|
| 1480 | | -{ |
|---|
| 1481 | | - if (!(action & MLX5_FLOW_CONTEXT_ACTION_COUNT)) |
|---|
| 1482 | | - return !counter; |
|---|
| 1483 | | - |
|---|
| 1484 | | - if (!counter) |
|---|
| 1485 | | - return false; |
|---|
| 1486 | | - |
|---|
| 1487 | 1689 | return (action & (MLX5_FLOW_CONTEXT_ACTION_DROP | |
|---|
| 1690 | + MLX5_FLOW_CONTEXT_ACTION_ALLOW | |
|---|
| 1488 | 1691 | MLX5_FLOW_CONTEXT_ACTION_FWD_DEST)); |
|---|
| 1489 | 1692 | } |
|---|
| 1490 | 1693 | |
|---|
| 1491 | 1694 | static bool dest_is_valid(struct mlx5_flow_destination *dest, |
|---|
| 1492 | | - u32 action, |
|---|
| 1695 | + struct mlx5_flow_act *flow_act, |
|---|
| 1493 | 1696 | struct mlx5_flow_table *ft) |
|---|
| 1494 | 1697 | { |
|---|
| 1698 | + bool ignore_level = flow_act->flags & FLOW_ACT_IGNORE_FLOW_LEVEL; |
|---|
| 1699 | + u32 action = flow_act->action; |
|---|
| 1700 | + |
|---|
| 1495 | 1701 | if (dest && (dest->type == MLX5_FLOW_DESTINATION_TYPE_COUNTER)) |
|---|
| 1496 | | - return counter_is_valid(dest->counter, action); |
|---|
| 1702 | + return counter_is_valid(action); |
|---|
| 1497 | 1703 | |
|---|
| 1498 | 1704 | if (!(action & MLX5_FLOW_CONTEXT_ACTION_FWD_DEST)) |
|---|
| 1499 | 1705 | return true; |
|---|
| 1500 | 1706 | |
|---|
| 1707 | + if (ignore_level) { |
|---|
| 1708 | + if (ft->type != FS_FT_FDB && |
|---|
| 1709 | + ft->type != FS_FT_NIC_RX) |
|---|
| 1710 | + return false; |
|---|
| 1711 | + |
|---|
| 1712 | + if (dest->type == MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE && |
|---|
| 1713 | + ft->type != dest->ft->type) |
|---|
| 1714 | + return false; |
|---|
| 1715 | + } |
|---|
| 1716 | + |
|---|
| 1501 | 1717 | if (!dest || ((dest->type == |
|---|
| 1502 | 1718 | MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE) && |
|---|
| 1503 | | - (dest->ft->level <= ft->level))) |
|---|
| 1719 | + (dest->ft->level <= ft->level && !ignore_level))) |
|---|
| 1504 | 1720 | return false; |
|---|
| 1505 | 1721 | return true; |
|---|
| 1506 | 1722 | } |
|---|
| .. | .. |
|---|
| 1510 | 1726 | struct mlx5_flow_group *g; |
|---|
| 1511 | 1727 | }; |
|---|
| 1512 | 1728 | |
|---|
| 1513 | | -struct match_list_head { |
|---|
| 1514 | | - struct list_head list; |
|---|
| 1515 | | - struct match_list first; |
|---|
| 1516 | | -}; |
|---|
| 1517 | | - |
|---|
| 1518 | | -static void free_match_list(struct match_list_head *head) |
|---|
| 1729 | +static void free_match_list(struct match_list *head, bool ft_locked) |
|---|
| 1519 | 1730 | { |
|---|
| 1520 | | - if (!list_empty(&head->list)) { |
|---|
| 1521 | | - struct match_list *iter, *match_tmp; |
|---|
| 1731 | + struct match_list *iter, *match_tmp; |
|---|
| 1522 | 1732 | |
|---|
| 1523 | | - list_del(&head->first.list); |
|---|
| 1524 | | - tree_put_node(&head->first.g->node); |
|---|
| 1525 | | - list_for_each_entry_safe(iter, match_tmp, &head->list, |
|---|
| 1526 | | - list) { |
|---|
| 1527 | | - tree_put_node(&iter->g->node); |
|---|
| 1528 | | - list_del(&iter->list); |
|---|
| 1529 | | - kfree(iter); |
|---|
| 1530 | | - } |
|---|
| 1733 | + list_for_each_entry_safe(iter, match_tmp, &head->list, |
|---|
| 1734 | + list) { |
|---|
| 1735 | + tree_put_node(&iter->g->node, ft_locked); |
|---|
| 1736 | + list_del(&iter->list); |
|---|
| 1737 | + kfree(iter); |
|---|
| 1531 | 1738 | } |
|---|
| 1532 | 1739 | } |
|---|
| 1533 | 1740 | |
|---|
| 1534 | | -static int build_match_list(struct match_list_head *match_head, |
|---|
| 1741 | +static int build_match_list(struct match_list *match_head, |
|---|
| 1535 | 1742 | struct mlx5_flow_table *ft, |
|---|
| 1536 | | - struct mlx5_flow_spec *spec) |
|---|
| 1743 | + const struct mlx5_flow_spec *spec, |
|---|
| 1744 | + bool ft_locked) |
|---|
| 1537 | 1745 | { |
|---|
| 1538 | 1746 | struct rhlist_head *tmp, *list; |
|---|
| 1539 | 1747 | struct mlx5_flow_group *g; |
|---|
| .. | .. |
|---|
| 1547 | 1755 | rhl_for_each_entry_rcu(g, tmp, list, hash) { |
|---|
| 1548 | 1756 | struct match_list *curr_match; |
|---|
| 1549 | 1757 | |
|---|
| 1550 | | - if (likely(list_empty(&match_head->list))) { |
|---|
| 1551 | | - if (!tree_get_node(&g->node)) |
|---|
| 1552 | | - continue; |
|---|
| 1553 | | - match_head->first.g = g; |
|---|
| 1554 | | - list_add_tail(&match_head->first.list, |
|---|
| 1555 | | - &match_head->list); |
|---|
| 1758 | + if (unlikely(!tree_get_node(&g->node))) |
|---|
| 1556 | 1759 | continue; |
|---|
| 1557 | | - } |
|---|
| 1558 | 1760 | |
|---|
| 1559 | 1761 | curr_match = kmalloc(sizeof(*curr_match), GFP_ATOMIC); |
|---|
| 1560 | 1762 | if (!curr_match) { |
|---|
| 1561 | 1763 | rcu_read_unlock(); |
|---|
| 1562 | | - free_match_list(match_head); |
|---|
| 1764 | + free_match_list(match_head, ft_locked); |
|---|
| 1563 | 1765 | return -ENOMEM; |
|---|
| 1564 | | - } |
|---|
| 1565 | | - if (!tree_get_node(&g->node)) { |
|---|
| 1566 | | - kfree(curr_match); |
|---|
| 1567 | | - continue; |
|---|
| 1568 | 1766 | } |
|---|
| 1569 | 1767 | curr_match->g = g; |
|---|
| 1570 | 1768 | list_add_tail(&curr_match->list, &match_head->list); |
|---|
| .. | .. |
|---|
| 1585 | 1783 | |
|---|
| 1586 | 1784 | static struct fs_fte * |
|---|
| 1587 | 1785 | lookup_fte_locked(struct mlx5_flow_group *g, |
|---|
| 1588 | | - u32 *match_value, |
|---|
| 1786 | + const u32 *match_value, |
|---|
| 1589 | 1787 | bool take_write) |
|---|
| 1590 | 1788 | { |
|---|
| 1591 | 1789 | struct fs_fte *fte_tmp; |
|---|
| .. | .. |
|---|
| 1601 | 1799 | goto out; |
|---|
| 1602 | 1800 | } |
|---|
| 1603 | 1801 | if (!fte_tmp->node.active) { |
|---|
| 1604 | | - tree_put_node(&fte_tmp->node); |
|---|
| 1802 | + tree_put_node(&fte_tmp->node, false); |
|---|
| 1605 | 1803 | fte_tmp = NULL; |
|---|
| 1606 | 1804 | goto out; |
|---|
| 1607 | 1805 | } |
|---|
| .. | .. |
|---|
| 1609 | 1807 | nested_down_write_ref_node(&fte_tmp->node, FS_LOCK_CHILD); |
|---|
| 1610 | 1808 | out: |
|---|
| 1611 | 1809 | if (take_write) |
|---|
| 1612 | | - up_write_ref_node(&g->node); |
|---|
| 1810 | + up_write_ref_node(&g->node, false); |
|---|
| 1613 | 1811 | else |
|---|
| 1614 | 1812 | up_read_ref_node(&g->node); |
|---|
| 1615 | 1813 | return fte_tmp; |
|---|
| .. | .. |
|---|
| 1618 | 1816 | static struct mlx5_flow_handle * |
|---|
| 1619 | 1817 | try_add_to_existing_fg(struct mlx5_flow_table *ft, |
|---|
| 1620 | 1818 | struct list_head *match_head, |
|---|
| 1621 | | - struct mlx5_flow_spec *spec, |
|---|
| 1819 | + const struct mlx5_flow_spec *spec, |
|---|
| 1622 | 1820 | struct mlx5_flow_act *flow_act, |
|---|
| 1623 | 1821 | struct mlx5_flow_destination *dest, |
|---|
| 1624 | 1822 | int dest_num, |
|---|
| .. | .. |
|---|
| 1630 | 1828 | struct match_list *iter; |
|---|
| 1631 | 1829 | bool take_write = false; |
|---|
| 1632 | 1830 | struct fs_fte *fte; |
|---|
| 1633 | | - u64 version; |
|---|
| 1831 | + u64 version = 0; |
|---|
| 1634 | 1832 | int err; |
|---|
| 1635 | 1833 | |
|---|
| 1636 | | - fte = alloc_fte(ft, spec->match_value, flow_act); |
|---|
| 1834 | + fte = alloc_fte(ft, spec, flow_act); |
|---|
| 1637 | 1835 | if (IS_ERR(fte)) |
|---|
| 1638 | 1836 | return ERR_PTR(-ENOMEM); |
|---|
| 1639 | 1837 | |
|---|
| 1640 | 1838 | search_again_locked: |
|---|
| 1839 | + if (flow_act->flags & FLOW_ACT_NO_APPEND) |
|---|
| 1840 | + goto skip_search; |
|---|
| 1641 | 1841 | version = matched_fgs_get_version(match_head); |
|---|
| 1642 | | - /* Try to find a fg that already contains a matching fte */ |
|---|
| 1842 | + /* Try to find an fte with identical match value and attempt update its |
|---|
| 1843 | + * action. |
|---|
| 1844 | + */ |
|---|
| 1643 | 1845 | list_for_each_entry(iter, match_head, list) { |
|---|
| 1644 | 1846 | struct fs_fte *fte_tmp; |
|---|
| 1645 | 1847 | |
|---|
| .. | .. |
|---|
| 1647 | 1849 | fte_tmp = lookup_fte_locked(g, spec->match_value, take_write); |
|---|
| 1648 | 1850 | if (!fte_tmp) |
|---|
| 1649 | 1851 | continue; |
|---|
| 1650 | | - rule = add_rule_fg(g, spec->match_value, |
|---|
| 1651 | | - flow_act, dest, dest_num, fte_tmp); |
|---|
| 1652 | | - up_write_ref_node(&fte_tmp->node); |
|---|
| 1653 | | - tree_put_node(&fte_tmp->node); |
|---|
| 1852 | + rule = add_rule_fg(g, spec, flow_act, dest, dest_num, fte_tmp); |
|---|
| 1853 | + /* No error check needed here, because insert_fte() is not called */ |
|---|
| 1854 | + up_write_ref_node(&fte_tmp->node, false); |
|---|
| 1855 | + tree_put_node(&fte_tmp->node, false); |
|---|
| 1654 | 1856 | kmem_cache_free(steering->ftes_cache, fte); |
|---|
| 1655 | 1857 | return rule; |
|---|
| 1656 | 1858 | } |
|---|
| 1859 | + |
|---|
| 1860 | +skip_search: |
|---|
| 1861 | + /* No group with matching fte found, or we skipped the search. |
|---|
| 1862 | + * Try to add a new fte to any matching fg. |
|---|
| 1863 | + */ |
|---|
| 1657 | 1864 | |
|---|
| 1658 | 1865 | /* Check the ft version, for case that new flow group |
|---|
| 1659 | 1866 | * was added while the fgs weren't locked |
|---|
| .. | .. |
|---|
| 1663 | 1870 | goto out; |
|---|
| 1664 | 1871 | } |
|---|
| 1665 | 1872 | |
|---|
| 1666 | | - /* Check the fgs version, for case the new FTE with the |
|---|
| 1667 | | - * same values was added while the fgs weren't locked |
|---|
| 1873 | + /* Check the fgs version. If version have changed it could be that an |
|---|
| 1874 | + * FTE with the same match value was added while the fgs weren't |
|---|
| 1875 | + * locked. |
|---|
| 1668 | 1876 | */ |
|---|
| 1669 | | - if (version != matched_fgs_get_version(match_head)) { |
|---|
| 1877 | + if (!(flow_act->flags & FLOW_ACT_NO_APPEND) && |
|---|
| 1878 | + version != matched_fgs_get_version(match_head)) { |
|---|
| 1670 | 1879 | take_write = true; |
|---|
| 1671 | 1880 | goto search_again_locked; |
|---|
| 1672 | 1881 | } |
|---|
| .. | .. |
|---|
| 1674 | 1883 | list_for_each_entry(iter, match_head, list) { |
|---|
| 1675 | 1884 | g = iter->g; |
|---|
| 1676 | 1885 | |
|---|
| 1677 | | - if (!g->node.active) |
|---|
| 1678 | | - continue; |
|---|
| 1679 | | - |
|---|
| 1680 | 1886 | nested_down_write_ref_node(&g->node, FS_LOCK_PARENT); |
|---|
| 1887 | + |
|---|
| 1888 | + if (!g->node.active) { |
|---|
| 1889 | + up_write_ref_node(&g->node, false); |
|---|
| 1890 | + continue; |
|---|
| 1891 | + } |
|---|
| 1681 | 1892 | |
|---|
| 1682 | 1893 | err = insert_fte(g, fte); |
|---|
| 1683 | 1894 | if (err) { |
|---|
| 1684 | | - up_write_ref_node(&g->node); |
|---|
| 1895 | + up_write_ref_node(&g->node, false); |
|---|
| 1685 | 1896 | if (err == -ENOSPC) |
|---|
| 1686 | 1897 | continue; |
|---|
| 1687 | 1898 | kmem_cache_free(steering->ftes_cache, fte); |
|---|
| .. | .. |
|---|
| 1689 | 1900 | } |
|---|
| 1690 | 1901 | |
|---|
| 1691 | 1902 | nested_down_write_ref_node(&fte->node, FS_LOCK_CHILD); |
|---|
| 1692 | | - up_write_ref_node(&g->node); |
|---|
| 1693 | | - rule = add_rule_fg(g, spec->match_value, |
|---|
| 1694 | | - flow_act, dest, dest_num, fte); |
|---|
| 1695 | | - up_write_ref_node(&fte->node); |
|---|
| 1696 | | - tree_put_node(&fte->node); |
|---|
| 1903 | + up_write_ref_node(&g->node, false); |
|---|
| 1904 | + rule = add_rule_fg(g, spec, flow_act, dest, dest_num, fte); |
|---|
| 1905 | + up_write_ref_node(&fte->node, false); |
|---|
| 1906 | + if (IS_ERR(rule)) |
|---|
| 1907 | + tree_put_node(&fte->node, false); |
|---|
| 1697 | 1908 | return rule; |
|---|
| 1698 | 1909 | } |
|---|
| 1699 | 1910 | rule = ERR_PTR(-ENOENT); |
|---|
| .. | .. |
|---|
| 1704 | 1915 | |
|---|
| 1705 | 1916 | static struct mlx5_flow_handle * |
|---|
| 1706 | 1917 | _mlx5_add_flow_rules(struct mlx5_flow_table *ft, |
|---|
| 1707 | | - struct mlx5_flow_spec *spec, |
|---|
| 1918 | + const struct mlx5_flow_spec *spec, |
|---|
| 1708 | 1919 | struct mlx5_flow_act *flow_act, |
|---|
| 1709 | 1920 | struct mlx5_flow_destination *dest, |
|---|
| 1710 | 1921 | int dest_num) |
|---|
| 1711 | 1922 | |
|---|
| 1712 | 1923 | { |
|---|
| 1713 | 1924 | struct mlx5_flow_steering *steering = get_steering(&ft->node); |
|---|
| 1714 | | - struct mlx5_flow_group *g; |
|---|
| 1715 | 1925 | struct mlx5_flow_handle *rule; |
|---|
| 1716 | | - struct match_list_head match_head; |
|---|
| 1926 | + struct match_list match_head; |
|---|
| 1927 | + struct mlx5_flow_group *g; |
|---|
| 1717 | 1928 | bool take_write = false; |
|---|
| 1718 | 1929 | struct fs_fte *fte; |
|---|
| 1719 | 1930 | int version; |
|---|
| .. | .. |
|---|
| 1724 | 1935 | return ERR_PTR(-EINVAL); |
|---|
| 1725 | 1936 | |
|---|
| 1726 | 1937 | for (i = 0; i < dest_num; i++) { |
|---|
| 1727 | | - if (!dest_is_valid(&dest[i], flow_act->action, ft)) |
|---|
| 1938 | + if (!dest_is_valid(&dest[i], flow_act, ft)) |
|---|
| 1728 | 1939 | return ERR_PTR(-EINVAL); |
|---|
| 1729 | 1940 | } |
|---|
| 1730 | 1941 | nested_down_read_ref_node(&ft->node, FS_LOCK_GRANDPARENT); |
|---|
| .. | .. |
|---|
| 1732 | 1943 | version = atomic_read(&ft->node.version); |
|---|
| 1733 | 1944 | |
|---|
| 1734 | 1945 | /* Collect all fgs which has a matching match_criteria */ |
|---|
| 1735 | | - err = build_match_list(&match_head, ft, spec); |
|---|
| 1946 | + err = build_match_list(&match_head, ft, spec, take_write); |
|---|
| 1736 | 1947 | if (err) { |
|---|
| 1737 | 1948 | if (take_write) |
|---|
| 1738 | | - up_write_ref_node(&ft->node); |
|---|
| 1949 | + up_write_ref_node(&ft->node, false); |
|---|
| 1739 | 1950 | else |
|---|
| 1740 | 1951 | up_read_ref_node(&ft->node); |
|---|
| 1741 | 1952 | return ERR_PTR(err); |
|---|
| .. | .. |
|---|
| 1746 | 1957 | |
|---|
| 1747 | 1958 | rule = try_add_to_existing_fg(ft, &match_head.list, spec, flow_act, dest, |
|---|
| 1748 | 1959 | dest_num, version); |
|---|
| 1749 | | - free_match_list(&match_head); |
|---|
| 1960 | + free_match_list(&match_head, take_write); |
|---|
| 1750 | 1961 | if (!IS_ERR(rule) || |
|---|
| 1751 | 1962 | (PTR_ERR(rule) != -ENOENT && PTR_ERR(rule) != -EAGAIN)) { |
|---|
| 1752 | 1963 | if (take_write) |
|---|
| 1753 | | - up_write_ref_node(&ft->node); |
|---|
| 1964 | + up_write_ref_node(&ft->node, false); |
|---|
| 1754 | 1965 | return rule; |
|---|
| 1755 | 1966 | } |
|---|
| 1756 | 1967 | |
|---|
| .. | .. |
|---|
| 1766 | 1977 | g = alloc_auto_flow_group(ft, spec); |
|---|
| 1767 | 1978 | if (IS_ERR(g)) { |
|---|
| 1768 | 1979 | rule = ERR_CAST(g); |
|---|
| 1769 | | - up_write_ref_node(&ft->node); |
|---|
| 1980 | + up_write_ref_node(&ft->node, false); |
|---|
| 1770 | 1981 | return rule; |
|---|
| 1771 | 1982 | } |
|---|
| 1772 | 1983 | |
|---|
| 1984 | + fte = alloc_fte(ft, spec, flow_act); |
|---|
| 1985 | + if (IS_ERR(fte)) { |
|---|
| 1986 | + up_write_ref_node(&ft->node, false); |
|---|
| 1987 | + err = PTR_ERR(fte); |
|---|
| 1988 | + goto err_alloc_fte; |
|---|
| 1989 | + } |
|---|
| 1990 | + |
|---|
| 1773 | 1991 | nested_down_write_ref_node(&g->node, FS_LOCK_PARENT); |
|---|
| 1774 | | - up_write_ref_node(&ft->node); |
|---|
| 1992 | + up_write_ref_node(&ft->node, false); |
|---|
| 1775 | 1993 | |
|---|
| 1776 | 1994 | err = create_auto_flow_group(ft, g); |
|---|
| 1777 | 1995 | if (err) |
|---|
| 1778 | 1996 | goto err_release_fg; |
|---|
| 1779 | 1997 | |
|---|
| 1780 | | - fte = alloc_fte(ft, spec->match_value, flow_act); |
|---|
| 1781 | | - if (IS_ERR(fte)) { |
|---|
| 1782 | | - err = PTR_ERR(fte); |
|---|
| 1783 | | - goto err_release_fg; |
|---|
| 1784 | | - } |
|---|
| 1785 | | - |
|---|
| 1786 | 1998 | err = insert_fte(g, fte); |
|---|
| 1787 | | - if (err) { |
|---|
| 1788 | | - kmem_cache_free(steering->ftes_cache, fte); |
|---|
| 1999 | + if (err) |
|---|
| 1789 | 2000 | goto err_release_fg; |
|---|
| 1790 | | - } |
|---|
| 1791 | 2001 | |
|---|
| 1792 | 2002 | nested_down_write_ref_node(&fte->node, FS_LOCK_CHILD); |
|---|
| 1793 | | - up_write_ref_node(&g->node); |
|---|
| 1794 | | - rule = add_rule_fg(g, spec->match_value, flow_act, dest, |
|---|
| 1795 | | - dest_num, fte); |
|---|
| 1796 | | - up_write_ref_node(&fte->node); |
|---|
| 1797 | | - tree_put_node(&fte->node); |
|---|
| 1798 | | - tree_put_node(&g->node); |
|---|
| 2003 | + up_write_ref_node(&g->node, false); |
|---|
| 2004 | + rule = add_rule_fg(g, spec, flow_act, dest, dest_num, fte); |
|---|
| 2005 | + up_write_ref_node(&fte->node, false); |
|---|
| 2006 | + if (IS_ERR(rule)) |
|---|
| 2007 | + tree_put_node(&fte->node, false); |
|---|
| 2008 | + tree_put_node(&g->node, false); |
|---|
| 1799 | 2009 | return rule; |
|---|
| 1800 | 2010 | |
|---|
| 1801 | 2011 | err_release_fg: |
|---|
| 1802 | | - up_write_ref_node(&g->node); |
|---|
| 1803 | | - tree_put_node(&g->node); |
|---|
| 2012 | + up_write_ref_node(&g->node, false); |
|---|
| 2013 | + kmem_cache_free(steering->ftes_cache, fte); |
|---|
| 2014 | +err_alloc_fte: |
|---|
| 2015 | + tree_put_node(&g->node, false); |
|---|
| 1804 | 2016 | return ERR_PTR(err); |
|---|
| 1805 | 2017 | } |
|---|
| 1806 | 2018 | |
|---|
| .. | .. |
|---|
| 1812 | 2024 | |
|---|
| 1813 | 2025 | struct mlx5_flow_handle * |
|---|
| 1814 | 2026 | mlx5_add_flow_rules(struct mlx5_flow_table *ft, |
|---|
| 1815 | | - struct mlx5_flow_spec *spec, |
|---|
| 2027 | + const struct mlx5_flow_spec *spec, |
|---|
| 1816 | 2028 | struct mlx5_flow_act *flow_act, |
|---|
| 1817 | 2029 | struct mlx5_flow_destination *dest, |
|---|
| 1818 | 2030 | int num_dest) |
|---|
| 1819 | 2031 | { |
|---|
| 1820 | 2032 | struct mlx5_flow_root_namespace *root = find_root(&ft->node); |
|---|
| 1821 | | - struct mlx5_flow_destination gen_dest = {}; |
|---|
| 2033 | + static const struct mlx5_flow_spec zero_spec = {}; |
|---|
| 2034 | + struct mlx5_flow_destination *gen_dest = NULL; |
|---|
| 1822 | 2035 | struct mlx5_flow_table *next_ft = NULL; |
|---|
| 1823 | 2036 | struct mlx5_flow_handle *handle = NULL; |
|---|
| 1824 | 2037 | u32 sw_action = flow_act->action; |
|---|
| 1825 | | - struct fs_prio *prio; |
|---|
| 2038 | + int i; |
|---|
| 1826 | 2039 | |
|---|
| 1827 | | - fs_get_obj(prio, ft->node.parent); |
|---|
| 1828 | | - if (flow_act->action == MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_PRIO) { |
|---|
| 1829 | | - if (!fwd_next_prio_supported(ft)) |
|---|
| 1830 | | - return ERR_PTR(-EOPNOTSUPP); |
|---|
| 1831 | | - if (num_dest) |
|---|
| 1832 | | - return ERR_PTR(-EINVAL); |
|---|
| 1833 | | - mutex_lock(&root->chain_lock); |
|---|
| 1834 | | - next_ft = find_next_chained_ft(prio); |
|---|
| 1835 | | - if (next_ft) { |
|---|
| 1836 | | - gen_dest.type = MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE; |
|---|
| 1837 | | - gen_dest.ft = next_ft; |
|---|
| 1838 | | - dest = &gen_dest; |
|---|
| 1839 | | - num_dest = 1; |
|---|
| 1840 | | - flow_act->action = MLX5_FLOW_CONTEXT_ACTION_FWD_DEST; |
|---|
| 1841 | | - } else { |
|---|
| 1842 | | - mutex_unlock(&root->chain_lock); |
|---|
| 1843 | | - return ERR_PTR(-EOPNOTSUPP); |
|---|
| 1844 | | - } |
|---|
| 2040 | + if (!spec) |
|---|
| 2041 | + spec = &zero_spec; |
|---|
| 2042 | + |
|---|
| 2043 | + if (!is_fwd_next_action(sw_action)) |
|---|
| 2044 | + return _mlx5_add_flow_rules(ft, spec, flow_act, dest, num_dest); |
|---|
| 2045 | + |
|---|
| 2046 | + if (!fwd_next_prio_supported(ft)) |
|---|
| 2047 | + return ERR_PTR(-EOPNOTSUPP); |
|---|
| 2048 | + |
|---|
| 2049 | + mutex_lock(&root->chain_lock); |
|---|
| 2050 | + next_ft = find_next_fwd_ft(ft, flow_act); |
|---|
| 2051 | + if (!next_ft) { |
|---|
| 2052 | + handle = ERR_PTR(-EOPNOTSUPP); |
|---|
| 2053 | + goto unlock; |
|---|
| 1845 | 2054 | } |
|---|
| 1846 | 2055 | |
|---|
| 2056 | + gen_dest = kcalloc(num_dest + 1, sizeof(*dest), |
|---|
| 2057 | + GFP_KERNEL); |
|---|
| 2058 | + if (!gen_dest) { |
|---|
| 2059 | + handle = ERR_PTR(-ENOMEM); |
|---|
| 2060 | + goto unlock; |
|---|
| 2061 | + } |
|---|
| 2062 | + for (i = 0; i < num_dest; i++) |
|---|
| 2063 | + gen_dest[i] = dest[i]; |
|---|
| 2064 | + gen_dest[i].type = |
|---|
| 2065 | + MLX5_FLOW_DESTINATION_TYPE_FLOW_TABLE; |
|---|
| 2066 | + gen_dest[i].ft = next_ft; |
|---|
| 2067 | + dest = gen_dest; |
|---|
| 2068 | + num_dest++; |
|---|
| 2069 | + flow_act->action &= ~(MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_PRIO | |
|---|
| 2070 | + MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_NS); |
|---|
| 2071 | + flow_act->action |= MLX5_FLOW_CONTEXT_ACTION_FWD_DEST; |
|---|
| 1847 | 2072 | handle = _mlx5_add_flow_rules(ft, spec, flow_act, dest, num_dest); |
|---|
| 2073 | + if (IS_ERR(handle)) |
|---|
| 2074 | + goto unlock; |
|---|
| 1848 | 2075 | |
|---|
| 1849 | | - if (sw_action == MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_PRIO) { |
|---|
| 1850 | | - if (!IS_ERR_OR_NULL(handle) && |
|---|
| 1851 | | - (list_empty(&handle->rule[0]->next_ft))) { |
|---|
| 1852 | | - mutex_lock(&next_ft->lock); |
|---|
| 1853 | | - list_add(&handle->rule[0]->next_ft, |
|---|
| 1854 | | - &next_ft->fwd_rules); |
|---|
| 1855 | | - mutex_unlock(&next_ft->lock); |
|---|
| 1856 | | - handle->rule[0]->sw_action = MLX5_FLOW_CONTEXT_ACTION_FWD_NEXT_PRIO; |
|---|
| 1857 | | - } |
|---|
| 1858 | | - mutex_unlock(&root->chain_lock); |
|---|
| 2076 | + if (list_empty(&handle->rule[num_dest - 1]->next_ft)) { |
|---|
| 2077 | + mutex_lock(&next_ft->lock); |
|---|
| 2078 | + list_add(&handle->rule[num_dest - 1]->next_ft, |
|---|
| 2079 | + &next_ft->fwd_rules); |
|---|
| 2080 | + mutex_unlock(&next_ft->lock); |
|---|
| 2081 | + handle->rule[num_dest - 1]->sw_action = sw_action; |
|---|
| 2082 | + handle->rule[num_dest - 1]->ft = ft; |
|---|
| 1859 | 2083 | } |
|---|
| 2084 | +unlock: |
|---|
| 2085 | + mutex_unlock(&root->chain_lock); |
|---|
| 2086 | + kfree(gen_dest); |
|---|
| 1860 | 2087 | return handle; |
|---|
| 1861 | 2088 | } |
|---|
| 1862 | 2089 | EXPORT_SYMBOL(mlx5_add_flow_rules); |
|---|
| 1863 | 2090 | |
|---|
| 1864 | 2091 | void mlx5_del_flow_rules(struct mlx5_flow_handle *handle) |
|---|
| 1865 | 2092 | { |
|---|
| 2093 | + struct fs_fte *fte; |
|---|
| 1866 | 2094 | int i; |
|---|
| 1867 | 2095 | |
|---|
| 2096 | + /* In order to consolidate the HW changes we lock the FTE for other |
|---|
| 2097 | + * changes, and increase its refcount, in order not to perform the |
|---|
| 2098 | + * "del" functions of the FTE. Will handle them here. |
|---|
| 2099 | + * The removal of the rules is done under locked FTE. |
|---|
| 2100 | + * After removing all the handle's rules, if there are remaining |
|---|
| 2101 | + * rules, it means we just need to modify the FTE in FW, and |
|---|
| 2102 | + * unlock/decrease the refcount we increased before. |
|---|
| 2103 | + * Otherwise, it means the FTE should be deleted. First delete the |
|---|
| 2104 | + * FTE in FW. Then, unlock the FTE, and proceed the tree_put_node of |
|---|
| 2105 | + * the FTE, which will handle the last decrease of the refcount, as |
|---|
| 2106 | + * well as required handling of its parent. |
|---|
| 2107 | + */ |
|---|
| 2108 | + fs_get_obj(fte, handle->rule[0]->node.parent); |
|---|
| 2109 | + down_write_ref_node(&fte->node, false); |
|---|
| 1868 | 2110 | for (i = handle->num_rules - 1; i >= 0; i--) |
|---|
| 1869 | | - tree_remove_node(&handle->rule[i]->node); |
|---|
| 2111 | + tree_remove_node(&handle->rule[i]->node, true); |
|---|
| 2112 | + if (list_empty(&fte->node.children)) { |
|---|
| 2113 | + del_hw_fte(&fte->node); |
|---|
| 2114 | + /* Avoid double call to del_hw_fte */ |
|---|
| 2115 | + fte->node.del_hw_func = NULL; |
|---|
| 2116 | + up_write_ref_node(&fte->node, false); |
|---|
| 2117 | + tree_put_node(&fte->node, false); |
|---|
| 2118 | + } else if (fte->dests_size) { |
|---|
| 2119 | + if (fte->modify_mask) |
|---|
| 2120 | + modify_fte(fte); |
|---|
| 2121 | + up_write_ref_node(&fte->node, false); |
|---|
| 2122 | + } else { |
|---|
| 2123 | + up_write_ref_node(&fte->node, false); |
|---|
| 2124 | + } |
|---|
| 1870 | 2125 | kfree(handle); |
|---|
| 1871 | 2126 | } |
|---|
| 1872 | 2127 | EXPORT_SYMBOL(mlx5_del_flow_rules); |
|---|
| .. | .. |
|---|
| 1874 | 2129 | /* Assuming prio->node.children(flow tables) is sorted by level */ |
|---|
| 1875 | 2130 | static struct mlx5_flow_table *find_next_ft(struct mlx5_flow_table *ft) |
|---|
| 1876 | 2131 | { |
|---|
| 2132 | + struct fs_node *prio_parent, *child; |
|---|
| 1877 | 2133 | struct fs_prio *prio; |
|---|
| 1878 | 2134 | |
|---|
| 1879 | 2135 | fs_get_obj(prio, ft->node.parent); |
|---|
| 1880 | 2136 | |
|---|
| 1881 | 2137 | if (!list_is_last(&ft->node.list, &prio->node.children)) |
|---|
| 1882 | 2138 | return list_next_entry(ft, node.list); |
|---|
| 1883 | | - return find_next_chained_ft(prio); |
|---|
| 2139 | + |
|---|
| 2140 | + prio_parent = find_prio_chains_parent(&prio->node, &child); |
|---|
| 2141 | + |
|---|
| 2142 | + if (prio_parent && list_is_first(&child->list, &prio_parent->children)) |
|---|
| 2143 | + return find_closest_ft(&prio->node, false, false); |
|---|
| 2144 | + |
|---|
| 2145 | + return find_next_chained_ft(&prio->node); |
|---|
| 1884 | 2146 | } |
|---|
| 1885 | 2147 | |
|---|
| 1886 | 2148 | static int update_root_ft_destroy(struct mlx5_flow_table *ft) |
|---|
| .. | .. |
|---|
| 1903 | 2165 | if (list_empty(&root->underlay_qpns)) { |
|---|
| 1904 | 2166 | /* Don't set any QPN (zero) in case QPN list is empty */ |
|---|
| 1905 | 2167 | qpn = 0; |
|---|
| 1906 | | - err = root->cmds->update_root_ft(root->dev, new_root_ft, |
|---|
| 2168 | + err = root->cmds->update_root_ft(root, new_root_ft, |
|---|
| 1907 | 2169 | qpn, false); |
|---|
| 1908 | 2170 | } else { |
|---|
| 1909 | 2171 | list_for_each_entry(uqp, &root->underlay_qpns, list) { |
|---|
| 1910 | 2172 | qpn = uqp->qpn; |
|---|
| 1911 | | - err = root->cmds->update_root_ft(root->dev, |
|---|
| 2173 | + err = root->cmds->update_root_ft(root, |
|---|
| 1912 | 2174 | new_root_ft, qpn, |
|---|
| 1913 | 2175 | false); |
|---|
| 1914 | 2176 | if (err) |
|---|
| .. | .. |
|---|
| 1964 | 2226 | int err = 0; |
|---|
| 1965 | 2227 | |
|---|
| 1966 | 2228 | mutex_lock(&root->chain_lock); |
|---|
| 1967 | | - err = disconnect_flow_table(ft); |
|---|
| 2229 | + if (!(ft->flags & MLX5_FLOW_TABLE_UNMANAGED)) |
|---|
| 2230 | + err = disconnect_flow_table(ft); |
|---|
| 1968 | 2231 | if (err) { |
|---|
| 1969 | 2232 | mutex_unlock(&root->chain_lock); |
|---|
| 1970 | 2233 | return err; |
|---|
| 1971 | 2234 | } |
|---|
| 1972 | | - if (tree_remove_node(&ft->node)) |
|---|
| 2235 | + if (tree_remove_node(&ft->node, false)) |
|---|
| 1973 | 2236 | mlx5_core_warn(get_dev(&ft->node), "Flow table %d wasn't destroyed, refcount > 1\n", |
|---|
| 1974 | 2237 | ft->id); |
|---|
| 1975 | 2238 | mutex_unlock(&root->chain_lock); |
|---|
| .. | .. |
|---|
| 1980 | 2243 | |
|---|
| 1981 | 2244 | void mlx5_destroy_flow_group(struct mlx5_flow_group *fg) |
|---|
| 1982 | 2245 | { |
|---|
| 1983 | | - if (tree_remove_node(&fg->node)) |
|---|
| 2246 | + if (tree_remove_node(&fg->node, false)) |
|---|
| 1984 | 2247 | mlx5_core_warn(get_dev(&fg->node), "Flow group %d wasn't destroyed, refcount > 1\n", |
|---|
| 1985 | 2248 | fg->id); |
|---|
| 1986 | 2249 | } |
|---|
| 2250 | + |
|---|
| 2251 | +struct mlx5_flow_namespace *mlx5_get_fdb_sub_ns(struct mlx5_core_dev *dev, |
|---|
| 2252 | + int n) |
|---|
| 2253 | +{ |
|---|
| 2254 | + struct mlx5_flow_steering *steering = dev->priv.steering; |
|---|
| 2255 | + |
|---|
| 2256 | + if (!steering || !steering->fdb_sub_ns) |
|---|
| 2257 | + return NULL; |
|---|
| 2258 | + |
|---|
| 2259 | + return steering->fdb_sub_ns[n]; |
|---|
| 2260 | +} |
|---|
| 2261 | +EXPORT_SYMBOL(mlx5_get_fdb_sub_ns); |
|---|
| 1987 | 2262 | |
|---|
| 1988 | 2263 | struct mlx5_flow_namespace *mlx5_get_flow_namespace(struct mlx5_core_dev *dev, |
|---|
| 1989 | 2264 | enum mlx5_flow_namespace_type type) |
|---|
| 1990 | 2265 | { |
|---|
| 1991 | 2266 | struct mlx5_flow_steering *steering = dev->priv.steering; |
|---|
| 1992 | 2267 | struct mlx5_flow_root_namespace *root_ns; |
|---|
| 1993 | | - int prio; |
|---|
| 2268 | + int prio = 0; |
|---|
| 1994 | 2269 | struct fs_prio *fs_prio; |
|---|
| 1995 | 2270 | struct mlx5_flow_namespace *ns; |
|---|
| 1996 | 2271 | |
|---|
| .. | .. |
|---|
| 1998 | 2273 | return NULL; |
|---|
| 1999 | 2274 | |
|---|
| 2000 | 2275 | switch (type) { |
|---|
| 2001 | | - case MLX5_FLOW_NAMESPACE_BYPASS: |
|---|
| 2002 | | - case MLX5_FLOW_NAMESPACE_LAG: |
|---|
| 2003 | | - case MLX5_FLOW_NAMESPACE_OFFLOADS: |
|---|
| 2004 | | - case MLX5_FLOW_NAMESPACE_ETHTOOL: |
|---|
| 2005 | | - case MLX5_FLOW_NAMESPACE_KERNEL: |
|---|
| 2006 | | - case MLX5_FLOW_NAMESPACE_LEFTOVERS: |
|---|
| 2007 | | - case MLX5_FLOW_NAMESPACE_ANCHOR: |
|---|
| 2008 | | - prio = type; |
|---|
| 2009 | | - break; |
|---|
| 2010 | 2276 | case MLX5_FLOW_NAMESPACE_FDB: |
|---|
| 2011 | 2277 | if (steering->fdb_root_ns) |
|---|
| 2012 | 2278 | return &steering->fdb_root_ns->ns; |
|---|
| 2013 | | - else |
|---|
| 2014 | | - return NULL; |
|---|
| 2279 | + return NULL; |
|---|
| 2015 | 2280 | case MLX5_FLOW_NAMESPACE_SNIFFER_RX: |
|---|
| 2016 | 2281 | if (steering->sniffer_rx_root_ns) |
|---|
| 2017 | 2282 | return &steering->sniffer_rx_root_ns->ns; |
|---|
| 2018 | | - else |
|---|
| 2019 | | - return NULL; |
|---|
| 2283 | + return NULL; |
|---|
| 2020 | 2284 | case MLX5_FLOW_NAMESPACE_SNIFFER_TX: |
|---|
| 2021 | 2285 | if (steering->sniffer_tx_root_ns) |
|---|
| 2022 | 2286 | return &steering->sniffer_tx_root_ns->ns; |
|---|
| 2023 | | - else |
|---|
| 2024 | | - return NULL; |
|---|
| 2025 | | - case MLX5_FLOW_NAMESPACE_EGRESS: |
|---|
| 2026 | | - if (steering->egress_root_ns) |
|---|
| 2027 | | - return &steering->egress_root_ns->ns; |
|---|
| 2028 | | - else |
|---|
| 2029 | | - return NULL; |
|---|
| 2030 | | - default: |
|---|
| 2031 | 2287 | return NULL; |
|---|
| 2288 | + default: |
|---|
| 2289 | + break; |
|---|
| 2032 | 2290 | } |
|---|
| 2033 | 2291 | |
|---|
| 2034 | | - root_ns = steering->root_ns; |
|---|
| 2292 | + if (type == MLX5_FLOW_NAMESPACE_EGRESS || |
|---|
| 2293 | + type == MLX5_FLOW_NAMESPACE_EGRESS_KERNEL) { |
|---|
| 2294 | + root_ns = steering->egress_root_ns; |
|---|
| 2295 | + prio = type - MLX5_FLOW_NAMESPACE_EGRESS; |
|---|
| 2296 | + } else if (type == MLX5_FLOW_NAMESPACE_RDMA_RX) { |
|---|
| 2297 | + root_ns = steering->rdma_rx_root_ns; |
|---|
| 2298 | + prio = RDMA_RX_BYPASS_PRIO; |
|---|
| 2299 | + } else if (type == MLX5_FLOW_NAMESPACE_RDMA_RX_KERNEL) { |
|---|
| 2300 | + root_ns = steering->rdma_rx_root_ns; |
|---|
| 2301 | + prio = RDMA_RX_KERNEL_PRIO; |
|---|
| 2302 | + } else if (type == MLX5_FLOW_NAMESPACE_RDMA_TX) { |
|---|
| 2303 | + root_ns = steering->rdma_tx_root_ns; |
|---|
| 2304 | + } else { /* Must be NIC RX */ |
|---|
| 2305 | + root_ns = steering->root_ns; |
|---|
| 2306 | + prio = type; |
|---|
| 2307 | + } |
|---|
| 2308 | + |
|---|
| 2035 | 2309 | if (!root_ns) |
|---|
| 2036 | 2310 | return NULL; |
|---|
| 2037 | 2311 | |
|---|
| .. | .. |
|---|
| 2053 | 2327 | { |
|---|
| 2054 | 2328 | struct mlx5_flow_steering *steering = dev->priv.steering; |
|---|
| 2055 | 2329 | |
|---|
| 2056 | | - if (!steering || vport >= MLX5_TOTAL_VPORTS(dev)) |
|---|
| 2330 | + if (!steering || vport >= mlx5_eswitch_get_total_vports(dev)) |
|---|
| 2057 | 2331 | return NULL; |
|---|
| 2058 | 2332 | |
|---|
| 2059 | 2333 | switch (type) { |
|---|
| .. | .. |
|---|
| 2074 | 2348 | } |
|---|
| 2075 | 2349 | } |
|---|
| 2076 | 2350 | |
|---|
| 2077 | | -static struct fs_prio *fs_create_prio(struct mlx5_flow_namespace *ns, |
|---|
| 2078 | | - unsigned int prio, int num_levels) |
|---|
| 2351 | +static struct fs_prio *_fs_create_prio(struct mlx5_flow_namespace *ns, |
|---|
| 2352 | + unsigned int prio, |
|---|
| 2353 | + int num_levels, |
|---|
| 2354 | + enum fs_node_type type) |
|---|
| 2079 | 2355 | { |
|---|
| 2080 | 2356 | struct fs_prio *fs_prio; |
|---|
| 2081 | 2357 | |
|---|
| .. | .. |
|---|
| 2083 | 2359 | if (!fs_prio) |
|---|
| 2084 | 2360 | return ERR_PTR(-ENOMEM); |
|---|
| 2085 | 2361 | |
|---|
| 2086 | | - fs_prio->node.type = FS_TYPE_PRIO; |
|---|
| 2362 | + fs_prio->node.type = type; |
|---|
| 2087 | 2363 | tree_init_node(&fs_prio->node, NULL, del_sw_prio); |
|---|
| 2088 | 2364 | tree_add_node(&fs_prio->node, &ns->node); |
|---|
| 2089 | 2365 | fs_prio->num_levels = num_levels; |
|---|
| .. | .. |
|---|
| 2091 | 2367 | list_add_tail(&fs_prio->node.list, &ns->node.children); |
|---|
| 2092 | 2368 | |
|---|
| 2093 | 2369 | return fs_prio; |
|---|
| 2370 | +} |
|---|
| 2371 | + |
|---|
| 2372 | +static struct fs_prio *fs_create_prio_chained(struct mlx5_flow_namespace *ns, |
|---|
| 2373 | + unsigned int prio, |
|---|
| 2374 | + int num_levels) |
|---|
| 2375 | +{ |
|---|
| 2376 | + return _fs_create_prio(ns, prio, num_levels, FS_TYPE_PRIO_CHAINS); |
|---|
| 2377 | +} |
|---|
| 2378 | + |
|---|
| 2379 | +static struct fs_prio *fs_create_prio(struct mlx5_flow_namespace *ns, |
|---|
| 2380 | + unsigned int prio, int num_levels) |
|---|
| 2381 | +{ |
|---|
| 2382 | + return _fs_create_prio(ns, prio, num_levels, FS_TYPE_PRIO); |
|---|
| 2094 | 2383 | } |
|---|
| 2095 | 2384 | |
|---|
| 2096 | 2385 | static struct mlx5_flow_namespace *fs_init_namespace(struct mlx5_flow_namespace |
|---|
| .. | .. |
|---|
| 2101 | 2390 | return ns; |
|---|
| 2102 | 2391 | } |
|---|
| 2103 | 2392 | |
|---|
| 2104 | | -static struct mlx5_flow_namespace *fs_create_namespace(struct fs_prio *prio) |
|---|
| 2393 | +static struct mlx5_flow_namespace *fs_create_namespace(struct fs_prio *prio, |
|---|
| 2394 | + int def_miss_act) |
|---|
| 2105 | 2395 | { |
|---|
| 2106 | 2396 | struct mlx5_flow_namespace *ns; |
|---|
| 2107 | 2397 | |
|---|
| .. | .. |
|---|
| 2110 | 2400 | return ERR_PTR(-ENOMEM); |
|---|
| 2111 | 2401 | |
|---|
| 2112 | 2402 | fs_init_namespace(ns); |
|---|
| 2403 | + ns->def_miss_action = def_miss_act; |
|---|
| 2113 | 2404 | tree_init_node(&ns->node, NULL, del_sw_ns); |
|---|
| 2114 | 2405 | tree_add_node(&ns->node, &prio->node); |
|---|
| 2115 | 2406 | list_add_tail(&ns->node.list, &prio->node.children); |
|---|
| .. | .. |
|---|
| 2176 | 2467 | base = &fs_prio->node; |
|---|
| 2177 | 2468 | } else if (init_node->type == FS_TYPE_NAMESPACE) { |
|---|
| 2178 | 2469 | fs_get_obj(fs_prio, fs_parent_node); |
|---|
| 2179 | | - fs_ns = fs_create_namespace(fs_prio); |
|---|
| 2470 | + fs_ns = fs_create_namespace(fs_prio, init_node->def_miss_action); |
|---|
| 2180 | 2471 | if (IS_ERR(fs_ns)) |
|---|
| 2181 | 2472 | return PTR_ERR(fs_ns); |
|---|
| 2182 | 2473 | base = &fs_ns->node; |
|---|
| .. | .. |
|---|
| 2217 | 2508 | return 0; |
|---|
| 2218 | 2509 | } |
|---|
| 2219 | 2510 | |
|---|
| 2511 | +static void del_sw_root_ns(struct fs_node *node) |
|---|
| 2512 | +{ |
|---|
| 2513 | + struct mlx5_flow_root_namespace *root_ns; |
|---|
| 2514 | + struct mlx5_flow_namespace *ns; |
|---|
| 2515 | + |
|---|
| 2516 | + fs_get_obj(ns, node); |
|---|
| 2517 | + root_ns = container_of(ns, struct mlx5_flow_root_namespace, ns); |
|---|
| 2518 | + mutex_destroy(&root_ns->chain_lock); |
|---|
| 2519 | + kfree(node); |
|---|
| 2520 | +} |
|---|
| 2521 | + |
|---|
| 2220 | 2522 | static struct mlx5_flow_root_namespace |
|---|
| 2221 | 2523 | *create_root_ns(struct mlx5_flow_steering *steering, |
|---|
| 2222 | 2524 | enum fs_flow_table_type table_type) |
|---|
| .. | .. |
|---|
| 2225 | 2527 | struct mlx5_flow_root_namespace *root_ns; |
|---|
| 2226 | 2528 | struct mlx5_flow_namespace *ns; |
|---|
| 2227 | 2529 | |
|---|
| 2228 | | - if (mlx5_accel_ipsec_device_caps(steering->dev) & MLX5_ACCEL_IPSEC_CAP_DEVICE && |
|---|
| 2530 | + if (mlx5_fpga_ipsec_device_caps(steering->dev) & MLX5_ACCEL_IPSEC_CAP_DEVICE && |
|---|
| 2229 | 2531 | (table_type == FS_FT_NIC_RX || table_type == FS_FT_NIC_TX)) |
|---|
| 2230 | 2532 | cmds = mlx5_fs_cmd_get_default_ipsec_fpga_cmds(table_type); |
|---|
| 2231 | 2533 | |
|---|
| .. | .. |
|---|
| 2243 | 2545 | ns = &root_ns->ns; |
|---|
| 2244 | 2546 | fs_init_namespace(ns); |
|---|
| 2245 | 2547 | mutex_init(&root_ns->chain_lock); |
|---|
| 2246 | | - tree_init_node(&ns->node, NULL, NULL); |
|---|
| 2548 | + tree_init_node(&ns->node, NULL, del_sw_root_ns); |
|---|
| 2247 | 2549 | tree_add_node(&ns->node, NULL); |
|---|
| 2248 | 2550 | |
|---|
| 2249 | 2551 | return root_ns; |
|---|
| .. | .. |
|---|
| 2269 | 2571 | int acc_level_ns = acc_level; |
|---|
| 2270 | 2572 | |
|---|
| 2271 | 2573 | prio->start_level = acc_level; |
|---|
| 2272 | | - fs_for_each_ns(ns, prio) |
|---|
| 2574 | + fs_for_each_ns(ns, prio) { |
|---|
| 2273 | 2575 | /* This updates start_level and num_levels of ns's priority descendants */ |
|---|
| 2274 | 2576 | acc_level_ns = set_prio_attrs_in_ns(ns, acc_level); |
|---|
| 2577 | + |
|---|
| 2578 | + /* If this a prio with chains, and we can jump from one chain |
|---|
| 2579 | + * (namepsace) to another, so we accumulate the levels |
|---|
| 2580 | + */ |
|---|
| 2581 | + if (prio->node.type == FS_TYPE_PRIO_CHAINS) |
|---|
| 2582 | + acc_level = acc_level_ns; |
|---|
| 2583 | + } |
|---|
| 2584 | + |
|---|
| 2275 | 2585 | if (!prio->num_levels) |
|---|
| 2276 | 2586 | prio->num_levels = acc_level_ns - prio->start_level; |
|---|
| 2277 | 2587 | WARN_ON(prio->num_levels < acc_level_ns - prio->start_level); |
|---|
| .. | .. |
|---|
| 2348 | 2658 | tree_get_node(node); |
|---|
| 2349 | 2659 | list_for_each_entry_safe(iter, temp, &node->children, list) |
|---|
| 2350 | 2660 | clean_tree(iter); |
|---|
| 2351 | | - tree_put_node(node); |
|---|
| 2352 | | - tree_remove_node(node); |
|---|
| 2661 | + tree_put_node(node, false); |
|---|
| 2662 | + tree_remove_node(node, false); |
|---|
| 2353 | 2663 | } |
|---|
| 2354 | 2664 | } |
|---|
| 2355 | 2665 | |
|---|
| .. | .. |
|---|
| 2369 | 2679 | if (!steering->esw_egress_root_ns) |
|---|
| 2370 | 2680 | return; |
|---|
| 2371 | 2681 | |
|---|
| 2372 | | - for (i = 0; i < MLX5_TOTAL_VPORTS(dev); i++) |
|---|
| 2682 | + for (i = 0; i < mlx5_eswitch_get_total_vports(dev); i++) |
|---|
| 2373 | 2683 | cleanup_root_ns(steering->esw_egress_root_ns[i]); |
|---|
| 2374 | 2684 | |
|---|
| 2375 | 2685 | kfree(steering->esw_egress_root_ns); |
|---|
| .. | .. |
|---|
| 2384 | 2694 | if (!steering->esw_ingress_root_ns) |
|---|
| 2385 | 2695 | return; |
|---|
| 2386 | 2696 | |
|---|
| 2387 | | - for (i = 0; i < MLX5_TOTAL_VPORTS(dev); i++) |
|---|
| 2697 | + for (i = 0; i < mlx5_eswitch_get_total_vports(dev); i++) |
|---|
| 2388 | 2698 | cleanup_root_ns(steering->esw_ingress_root_ns[i]); |
|---|
| 2389 | 2699 | |
|---|
| 2390 | 2700 | kfree(steering->esw_ingress_root_ns); |
|---|
| .. | .. |
|---|
| 2399 | 2709 | cleanup_egress_acls_root_ns(dev); |
|---|
| 2400 | 2710 | cleanup_ingress_acls_root_ns(dev); |
|---|
| 2401 | 2711 | cleanup_root_ns(steering->fdb_root_ns); |
|---|
| 2712 | + steering->fdb_root_ns = NULL; |
|---|
| 2713 | + kfree(steering->fdb_sub_ns); |
|---|
| 2714 | + steering->fdb_sub_ns = NULL; |
|---|
| 2402 | 2715 | cleanup_root_ns(steering->sniffer_rx_root_ns); |
|---|
| 2403 | 2716 | cleanup_root_ns(steering->sniffer_tx_root_ns); |
|---|
| 2717 | + cleanup_root_ns(steering->rdma_rx_root_ns); |
|---|
| 2718 | + cleanup_root_ns(steering->rdma_tx_root_ns); |
|---|
| 2404 | 2719 | cleanup_root_ns(steering->egress_root_ns); |
|---|
| 2405 | 2720 | mlx5_cleanup_fc_stats(dev); |
|---|
| 2406 | 2721 | kmem_cache_destroy(steering->ftes_cache); |
|---|
| .. | .. |
|---|
| 2418 | 2733 | |
|---|
| 2419 | 2734 | /* Create single prio */ |
|---|
| 2420 | 2735 | prio = fs_create_prio(&steering->sniffer_tx_root_ns->ns, 0, 1); |
|---|
| 2421 | | - if (IS_ERR(prio)) { |
|---|
| 2422 | | - cleanup_root_ns(steering->sniffer_tx_root_ns); |
|---|
| 2423 | | - return PTR_ERR(prio); |
|---|
| 2424 | | - } |
|---|
| 2425 | | - return 0; |
|---|
| 2736 | + return PTR_ERR_OR_ZERO(prio); |
|---|
| 2426 | 2737 | } |
|---|
| 2427 | 2738 | |
|---|
| 2428 | 2739 | static int init_sniffer_rx_root_ns(struct mlx5_flow_steering *steering) |
|---|
| .. | .. |
|---|
| 2435 | 2746 | |
|---|
| 2436 | 2747 | /* Create single prio */ |
|---|
| 2437 | 2748 | prio = fs_create_prio(&steering->sniffer_rx_root_ns->ns, 0, 1); |
|---|
| 2438 | | - if (IS_ERR(prio)) { |
|---|
| 2439 | | - cleanup_root_ns(steering->sniffer_rx_root_ns); |
|---|
| 2440 | | - return PTR_ERR(prio); |
|---|
| 2749 | + return PTR_ERR_OR_ZERO(prio); |
|---|
| 2750 | +} |
|---|
| 2751 | + |
|---|
| 2752 | +static int init_rdma_rx_root_ns(struct mlx5_flow_steering *steering) |
|---|
| 2753 | +{ |
|---|
| 2754 | + int err; |
|---|
| 2755 | + |
|---|
| 2756 | + steering->rdma_rx_root_ns = create_root_ns(steering, FS_FT_RDMA_RX); |
|---|
| 2757 | + if (!steering->rdma_rx_root_ns) |
|---|
| 2758 | + return -ENOMEM; |
|---|
| 2759 | + |
|---|
| 2760 | + err = init_root_tree(steering, &rdma_rx_root_fs, |
|---|
| 2761 | + &steering->rdma_rx_root_ns->ns.node); |
|---|
| 2762 | + if (err) |
|---|
| 2763 | + goto out_err; |
|---|
| 2764 | + |
|---|
| 2765 | + set_prio_attrs(steering->rdma_rx_root_ns); |
|---|
| 2766 | + |
|---|
| 2767 | + return 0; |
|---|
| 2768 | + |
|---|
| 2769 | +out_err: |
|---|
| 2770 | + cleanup_root_ns(steering->rdma_rx_root_ns); |
|---|
| 2771 | + steering->rdma_rx_root_ns = NULL; |
|---|
| 2772 | + return err; |
|---|
| 2773 | +} |
|---|
| 2774 | + |
|---|
| 2775 | +static int init_rdma_tx_root_ns(struct mlx5_flow_steering *steering) |
|---|
| 2776 | +{ |
|---|
| 2777 | + int err; |
|---|
| 2778 | + |
|---|
| 2779 | + steering->rdma_tx_root_ns = create_root_ns(steering, FS_FT_RDMA_TX); |
|---|
| 2780 | + if (!steering->rdma_tx_root_ns) |
|---|
| 2781 | + return -ENOMEM; |
|---|
| 2782 | + |
|---|
| 2783 | + err = init_root_tree(steering, &rdma_tx_root_fs, |
|---|
| 2784 | + &steering->rdma_tx_root_ns->ns.node); |
|---|
| 2785 | + if (err) |
|---|
| 2786 | + goto out_err; |
|---|
| 2787 | + |
|---|
| 2788 | + set_prio_attrs(steering->rdma_tx_root_ns); |
|---|
| 2789 | + |
|---|
| 2790 | + return 0; |
|---|
| 2791 | + |
|---|
| 2792 | +out_err: |
|---|
| 2793 | + cleanup_root_ns(steering->rdma_tx_root_ns); |
|---|
| 2794 | + steering->rdma_tx_root_ns = NULL; |
|---|
| 2795 | + return err; |
|---|
| 2796 | +} |
|---|
| 2797 | + |
|---|
| 2798 | +/* FT and tc chains are stored in the same array so we can re-use the |
|---|
| 2799 | + * mlx5_get_fdb_sub_ns() and tc api for FT chains. |
|---|
| 2800 | + * When creating a new ns for each chain store it in the first available slot. |
|---|
| 2801 | + * Assume tc chains are created and stored first and only then the FT chain. |
|---|
| 2802 | + */ |
|---|
| 2803 | +static void store_fdb_sub_ns_prio_chain(struct mlx5_flow_steering *steering, |
|---|
| 2804 | + struct mlx5_flow_namespace *ns) |
|---|
| 2805 | +{ |
|---|
| 2806 | + int chain = 0; |
|---|
| 2807 | + |
|---|
| 2808 | + while (steering->fdb_sub_ns[chain]) |
|---|
| 2809 | + ++chain; |
|---|
| 2810 | + |
|---|
| 2811 | + steering->fdb_sub_ns[chain] = ns; |
|---|
| 2812 | +} |
|---|
| 2813 | + |
|---|
| 2814 | +static int create_fdb_sub_ns_prio_chain(struct mlx5_flow_steering *steering, |
|---|
| 2815 | + struct fs_prio *maj_prio) |
|---|
| 2816 | +{ |
|---|
| 2817 | + struct mlx5_flow_namespace *ns; |
|---|
| 2818 | + struct fs_prio *min_prio; |
|---|
| 2819 | + int prio; |
|---|
| 2820 | + |
|---|
| 2821 | + ns = fs_create_namespace(maj_prio, MLX5_FLOW_TABLE_MISS_ACTION_DEF); |
|---|
| 2822 | + if (IS_ERR(ns)) |
|---|
| 2823 | + return PTR_ERR(ns); |
|---|
| 2824 | + |
|---|
| 2825 | + for (prio = 0; prio < FDB_TC_MAX_PRIO; prio++) { |
|---|
| 2826 | + min_prio = fs_create_prio(ns, prio, FDB_TC_LEVELS_PER_PRIO); |
|---|
| 2827 | + if (IS_ERR(min_prio)) |
|---|
| 2828 | + return PTR_ERR(min_prio); |
|---|
| 2441 | 2829 | } |
|---|
| 2830 | + |
|---|
| 2831 | + store_fdb_sub_ns_prio_chain(steering, ns); |
|---|
| 2832 | + |
|---|
| 2833 | + return 0; |
|---|
| 2834 | +} |
|---|
| 2835 | + |
|---|
| 2836 | +static int create_fdb_chains(struct mlx5_flow_steering *steering, |
|---|
| 2837 | + int fs_prio, |
|---|
| 2838 | + int chains) |
|---|
| 2839 | +{ |
|---|
| 2840 | + struct fs_prio *maj_prio; |
|---|
| 2841 | + int levels; |
|---|
| 2842 | + int chain; |
|---|
| 2843 | + int err; |
|---|
| 2844 | + |
|---|
| 2845 | + levels = FDB_TC_LEVELS_PER_PRIO * FDB_TC_MAX_PRIO * chains; |
|---|
| 2846 | + maj_prio = fs_create_prio_chained(&steering->fdb_root_ns->ns, |
|---|
| 2847 | + fs_prio, |
|---|
| 2848 | + levels); |
|---|
| 2849 | + if (IS_ERR(maj_prio)) |
|---|
| 2850 | + return PTR_ERR(maj_prio); |
|---|
| 2851 | + |
|---|
| 2852 | + for (chain = 0; chain < chains; chain++) { |
|---|
| 2853 | + err = create_fdb_sub_ns_prio_chain(steering, maj_prio); |
|---|
| 2854 | + if (err) |
|---|
| 2855 | + return err; |
|---|
| 2856 | + } |
|---|
| 2857 | + |
|---|
| 2858 | + return 0; |
|---|
| 2859 | +} |
|---|
| 2860 | + |
|---|
| 2861 | +static int create_fdb_fast_path(struct mlx5_flow_steering *steering) |
|---|
| 2862 | +{ |
|---|
| 2863 | + int err; |
|---|
| 2864 | + |
|---|
| 2865 | + steering->fdb_sub_ns = kcalloc(FDB_NUM_CHAINS, |
|---|
| 2866 | + sizeof(*steering->fdb_sub_ns), |
|---|
| 2867 | + GFP_KERNEL); |
|---|
| 2868 | + if (!steering->fdb_sub_ns) |
|---|
| 2869 | + return -ENOMEM; |
|---|
| 2870 | + |
|---|
| 2871 | + err = create_fdb_chains(steering, FDB_TC_OFFLOAD, FDB_TC_MAX_CHAIN + 1); |
|---|
| 2872 | + if (err) |
|---|
| 2873 | + return err; |
|---|
| 2874 | + |
|---|
| 2875 | + err = create_fdb_chains(steering, FDB_FT_OFFLOAD, 1); |
|---|
| 2876 | + if (err) |
|---|
| 2877 | + return err; |
|---|
| 2878 | + |
|---|
| 2442 | 2879 | return 0; |
|---|
| 2443 | 2880 | } |
|---|
| 2444 | 2881 | |
|---|
| 2445 | 2882 | static int init_fdb_root_ns(struct mlx5_flow_steering *steering) |
|---|
| 2446 | 2883 | { |
|---|
| 2447 | | - struct fs_prio *prio; |
|---|
| 2884 | + struct fs_prio *maj_prio; |
|---|
| 2885 | + int err; |
|---|
| 2448 | 2886 | |
|---|
| 2449 | 2887 | steering->fdb_root_ns = create_root_ns(steering, FS_FT_FDB); |
|---|
| 2450 | 2888 | if (!steering->fdb_root_ns) |
|---|
| 2451 | 2889 | return -ENOMEM; |
|---|
| 2452 | 2890 | |
|---|
| 2453 | | - prio = fs_create_prio(&steering->fdb_root_ns->ns, 0, 2); |
|---|
| 2454 | | - if (IS_ERR(prio)) |
|---|
| 2891 | + maj_prio = fs_create_prio(&steering->fdb_root_ns->ns, FDB_BYPASS_PATH, |
|---|
| 2892 | + 1); |
|---|
| 2893 | + if (IS_ERR(maj_prio)) { |
|---|
| 2894 | + err = PTR_ERR(maj_prio); |
|---|
| 2895 | + goto out_err; |
|---|
| 2896 | + } |
|---|
| 2897 | + err = create_fdb_fast_path(steering); |
|---|
| 2898 | + if (err) |
|---|
| 2455 | 2899 | goto out_err; |
|---|
| 2456 | 2900 | |
|---|
| 2457 | | - prio = fs_create_prio(&steering->fdb_root_ns->ns, 1, 1); |
|---|
| 2458 | | - if (IS_ERR(prio)) |
|---|
| 2901 | + maj_prio = fs_create_prio(&steering->fdb_root_ns->ns, FDB_SLOW_PATH, 1); |
|---|
| 2902 | + if (IS_ERR(maj_prio)) { |
|---|
| 2903 | + err = PTR_ERR(maj_prio); |
|---|
| 2459 | 2904 | goto out_err; |
|---|
| 2905 | + } |
|---|
| 2906 | + |
|---|
| 2907 | + /* We put this priority last, knowing that nothing will get here |
|---|
| 2908 | + * unless explicitly forwarded to. This is possible because the |
|---|
| 2909 | + * slow path tables have catch all rules and nothing gets passed |
|---|
| 2910 | + * those tables. |
|---|
| 2911 | + */ |
|---|
| 2912 | + maj_prio = fs_create_prio(&steering->fdb_root_ns->ns, FDB_PER_VPORT, 1); |
|---|
| 2913 | + if (IS_ERR(maj_prio)) { |
|---|
| 2914 | + err = PTR_ERR(maj_prio); |
|---|
| 2915 | + goto out_err; |
|---|
| 2916 | + } |
|---|
| 2460 | 2917 | |
|---|
| 2461 | 2918 | set_prio_attrs(steering->fdb_root_ns); |
|---|
| 2462 | 2919 | return 0; |
|---|
| 2463 | 2920 | |
|---|
| 2464 | 2921 | out_err: |
|---|
| 2465 | 2922 | cleanup_root_ns(steering->fdb_root_ns); |
|---|
| 2923 | + kfree(steering->fdb_sub_ns); |
|---|
| 2924 | + steering->fdb_sub_ns = NULL; |
|---|
| 2466 | 2925 | steering->fdb_root_ns = NULL; |
|---|
| 2467 | | - return PTR_ERR(prio); |
|---|
| 2926 | + return err; |
|---|
| 2468 | 2927 | } |
|---|
| 2469 | 2928 | |
|---|
| 2470 | 2929 | static int init_egress_acl_root_ns(struct mlx5_flow_steering *steering, int vport) |
|---|
| .. | .. |
|---|
| 2496 | 2955 | static int init_egress_acls_root_ns(struct mlx5_core_dev *dev) |
|---|
| 2497 | 2956 | { |
|---|
| 2498 | 2957 | struct mlx5_flow_steering *steering = dev->priv.steering; |
|---|
| 2958 | + int total_vports = mlx5_eswitch_get_total_vports(dev); |
|---|
| 2499 | 2959 | int err; |
|---|
| 2500 | 2960 | int i; |
|---|
| 2501 | 2961 | |
|---|
| 2502 | | - steering->esw_egress_root_ns = kcalloc(MLX5_TOTAL_VPORTS(dev), |
|---|
| 2503 | | - sizeof(*steering->esw_egress_root_ns), |
|---|
| 2504 | | - GFP_KERNEL); |
|---|
| 2962 | + steering->esw_egress_root_ns = |
|---|
| 2963 | + kcalloc(total_vports, |
|---|
| 2964 | + sizeof(*steering->esw_egress_root_ns), |
|---|
| 2965 | + GFP_KERNEL); |
|---|
| 2505 | 2966 | if (!steering->esw_egress_root_ns) |
|---|
| 2506 | 2967 | return -ENOMEM; |
|---|
| 2507 | 2968 | |
|---|
| 2508 | | - for (i = 0; i < MLX5_TOTAL_VPORTS(dev); i++) { |
|---|
| 2969 | + for (i = 0; i < total_vports; i++) { |
|---|
| 2509 | 2970 | err = init_egress_acl_root_ns(steering, i); |
|---|
| 2510 | 2971 | if (err) |
|---|
| 2511 | 2972 | goto cleanup_root_ns; |
|---|
| .. | .. |
|---|
| 2524 | 2985 | static int init_ingress_acls_root_ns(struct mlx5_core_dev *dev) |
|---|
| 2525 | 2986 | { |
|---|
| 2526 | 2987 | struct mlx5_flow_steering *steering = dev->priv.steering; |
|---|
| 2988 | + int total_vports = mlx5_eswitch_get_total_vports(dev); |
|---|
| 2527 | 2989 | int err; |
|---|
| 2528 | 2990 | int i; |
|---|
| 2529 | 2991 | |
|---|
| 2530 | | - steering->esw_ingress_root_ns = kcalloc(MLX5_TOTAL_VPORTS(dev), |
|---|
| 2531 | | - sizeof(*steering->esw_ingress_root_ns), |
|---|
| 2532 | | - GFP_KERNEL); |
|---|
| 2992 | + steering->esw_ingress_root_ns = |
|---|
| 2993 | + kcalloc(total_vports, |
|---|
| 2994 | + sizeof(*steering->esw_ingress_root_ns), |
|---|
| 2995 | + GFP_KERNEL); |
|---|
| 2533 | 2996 | if (!steering->esw_ingress_root_ns) |
|---|
| 2534 | 2997 | return -ENOMEM; |
|---|
| 2535 | 2998 | |
|---|
| 2536 | | - for (i = 0; i < MLX5_TOTAL_VPORTS(dev); i++) { |
|---|
| 2999 | + for (i = 0; i < total_vports; i++) { |
|---|
| 2537 | 3000 | err = init_ingress_acl_root_ns(steering, i); |
|---|
| 2538 | 3001 | if (err) |
|---|
| 2539 | 3002 | goto cleanup_root_ns; |
|---|
| .. | .. |
|---|
| 2551 | 3014 | |
|---|
| 2552 | 3015 | static int init_egress_root_ns(struct mlx5_flow_steering *steering) |
|---|
| 2553 | 3016 | { |
|---|
| 2554 | | - struct fs_prio *prio; |
|---|
| 3017 | + int err; |
|---|
| 2555 | 3018 | |
|---|
| 2556 | 3019 | steering->egress_root_ns = create_root_ns(steering, |
|---|
| 2557 | 3020 | FS_FT_NIC_TX); |
|---|
| 2558 | 3021 | if (!steering->egress_root_ns) |
|---|
| 2559 | 3022 | return -ENOMEM; |
|---|
| 2560 | 3023 | |
|---|
| 2561 | | - /* create 1 prio*/ |
|---|
| 2562 | | - prio = fs_create_prio(&steering->egress_root_ns->ns, 0, 1); |
|---|
| 2563 | | - return PTR_ERR_OR_ZERO(prio); |
|---|
| 3024 | + err = init_root_tree(steering, &egress_root_fs, |
|---|
| 3025 | + &steering->egress_root_ns->ns.node); |
|---|
| 3026 | + if (err) |
|---|
| 3027 | + goto cleanup; |
|---|
| 3028 | + set_prio_attrs(steering->egress_root_ns); |
|---|
| 3029 | + return 0; |
|---|
| 3030 | +cleanup: |
|---|
| 3031 | + cleanup_root_ns(steering->egress_root_ns); |
|---|
| 3032 | + steering->egress_root_ns = NULL; |
|---|
| 3033 | + return err; |
|---|
| 2564 | 3034 | } |
|---|
| 2565 | 3035 | |
|---|
| 2566 | 3036 | int mlx5_init_fs(struct mlx5_core_dev *dev) |
|---|
| .. | .. |
|---|
| 2628 | 3098 | goto err; |
|---|
| 2629 | 3099 | } |
|---|
| 2630 | 3100 | |
|---|
| 2631 | | - if (MLX5_IPSEC_DEV(dev)) { |
|---|
| 3101 | + if (MLX5_CAP_FLOWTABLE_RDMA_RX(dev, ft_support) && |
|---|
| 3102 | + MLX5_CAP_FLOWTABLE_RDMA_RX(dev, table_miss_action_domain)) { |
|---|
| 3103 | + err = init_rdma_rx_root_ns(steering); |
|---|
| 3104 | + if (err) |
|---|
| 3105 | + goto err; |
|---|
| 3106 | + } |
|---|
| 3107 | + |
|---|
| 3108 | + if (MLX5_CAP_FLOWTABLE_RDMA_TX(dev, ft_support)) { |
|---|
| 3109 | + err = init_rdma_tx_root_ns(steering); |
|---|
| 3110 | + if (err) |
|---|
| 3111 | + goto err; |
|---|
| 3112 | + } |
|---|
| 3113 | + |
|---|
| 3114 | + if (mlx5_fpga_ipsec_device_caps(steering->dev) & MLX5_ACCEL_IPSEC_CAP_DEVICE || |
|---|
| 3115 | + MLX5_CAP_FLOWTABLE_NIC_TX(dev, ft_support)) { |
|---|
| 2632 | 3116 | err = init_egress_root_ns(steering); |
|---|
| 2633 | 3117 | if (err) |
|---|
| 2634 | 3118 | goto err; |
|---|
| .. | .. |
|---|
| 2657 | 3141 | goto update_ft_fail; |
|---|
| 2658 | 3142 | } |
|---|
| 2659 | 3143 | |
|---|
| 2660 | | - err = root->cmds->update_root_ft(dev, root->root_ft, underlay_qpn, |
|---|
| 3144 | + err = root->cmds->update_root_ft(root, root->root_ft, underlay_qpn, |
|---|
| 2661 | 3145 | false); |
|---|
| 2662 | 3146 | if (err) { |
|---|
| 2663 | 3147 | mlx5_core_warn(dev, "Failed adding underlay QPN (%u) to root FT err(%d)\n", |
|---|
| .. | .. |
|---|
| 2701 | 3185 | goto out; |
|---|
| 2702 | 3186 | } |
|---|
| 2703 | 3187 | |
|---|
| 2704 | | - err = root->cmds->update_root_ft(dev, root->root_ft, underlay_qpn, |
|---|
| 3188 | + err = root->cmds->update_root_ft(root, root->root_ft, underlay_qpn, |
|---|
| 2705 | 3189 | true); |
|---|
| 2706 | 3190 | if (err) |
|---|
| 2707 | 3191 | mlx5_core_warn(dev, "Failed removing underlay QPN (%u) from root FT err(%d)\n", |
|---|
| .. | .. |
|---|
| 2718 | 3202 | return err; |
|---|
| 2719 | 3203 | } |
|---|
| 2720 | 3204 | EXPORT_SYMBOL(mlx5_fs_remove_rx_underlay_qpn); |
|---|
| 3205 | + |
|---|
| 3206 | +static struct mlx5_flow_root_namespace |
|---|
| 3207 | +*get_root_namespace(struct mlx5_core_dev *dev, enum mlx5_flow_namespace_type ns_type) |
|---|
| 3208 | +{ |
|---|
| 3209 | + struct mlx5_flow_namespace *ns; |
|---|
| 3210 | + |
|---|
| 3211 | + if (ns_type == MLX5_FLOW_NAMESPACE_ESW_EGRESS || |
|---|
| 3212 | + ns_type == MLX5_FLOW_NAMESPACE_ESW_INGRESS) |
|---|
| 3213 | + ns = mlx5_get_flow_vport_acl_namespace(dev, ns_type, 0); |
|---|
| 3214 | + else |
|---|
| 3215 | + ns = mlx5_get_flow_namespace(dev, ns_type); |
|---|
| 3216 | + if (!ns) |
|---|
| 3217 | + return NULL; |
|---|
| 3218 | + |
|---|
| 3219 | + return find_root(&ns->node); |
|---|
| 3220 | +} |
|---|
| 3221 | + |
|---|
| 3222 | +struct mlx5_modify_hdr *mlx5_modify_header_alloc(struct mlx5_core_dev *dev, |
|---|
| 3223 | + u8 ns_type, u8 num_actions, |
|---|
| 3224 | + void *modify_actions) |
|---|
| 3225 | +{ |
|---|
| 3226 | + struct mlx5_flow_root_namespace *root; |
|---|
| 3227 | + struct mlx5_modify_hdr *modify_hdr; |
|---|
| 3228 | + int err; |
|---|
| 3229 | + |
|---|
| 3230 | + root = get_root_namespace(dev, ns_type); |
|---|
| 3231 | + if (!root) |
|---|
| 3232 | + return ERR_PTR(-EOPNOTSUPP); |
|---|
| 3233 | + |
|---|
| 3234 | + modify_hdr = kzalloc(sizeof(*modify_hdr), GFP_KERNEL); |
|---|
| 3235 | + if (!modify_hdr) |
|---|
| 3236 | + return ERR_PTR(-ENOMEM); |
|---|
| 3237 | + |
|---|
| 3238 | + modify_hdr->ns_type = ns_type; |
|---|
| 3239 | + err = root->cmds->modify_header_alloc(root, ns_type, num_actions, |
|---|
| 3240 | + modify_actions, modify_hdr); |
|---|
| 3241 | + if (err) { |
|---|
| 3242 | + kfree(modify_hdr); |
|---|
| 3243 | + return ERR_PTR(err); |
|---|
| 3244 | + } |
|---|
| 3245 | + |
|---|
| 3246 | + return modify_hdr; |
|---|
| 3247 | +} |
|---|
| 3248 | +EXPORT_SYMBOL(mlx5_modify_header_alloc); |
|---|
| 3249 | + |
|---|
| 3250 | +void mlx5_modify_header_dealloc(struct mlx5_core_dev *dev, |
|---|
| 3251 | + struct mlx5_modify_hdr *modify_hdr) |
|---|
| 3252 | +{ |
|---|
| 3253 | + struct mlx5_flow_root_namespace *root; |
|---|
| 3254 | + |
|---|
| 3255 | + root = get_root_namespace(dev, modify_hdr->ns_type); |
|---|
| 3256 | + if (WARN_ON(!root)) |
|---|
| 3257 | + return; |
|---|
| 3258 | + root->cmds->modify_header_dealloc(root, modify_hdr); |
|---|
| 3259 | + kfree(modify_hdr); |
|---|
| 3260 | +} |
|---|
| 3261 | +EXPORT_SYMBOL(mlx5_modify_header_dealloc); |
|---|
| 3262 | + |
|---|
| 3263 | +struct mlx5_pkt_reformat *mlx5_packet_reformat_alloc(struct mlx5_core_dev *dev, |
|---|
| 3264 | + int reformat_type, |
|---|
| 3265 | + size_t size, |
|---|
| 3266 | + void *reformat_data, |
|---|
| 3267 | + enum mlx5_flow_namespace_type ns_type) |
|---|
| 3268 | +{ |
|---|
| 3269 | + struct mlx5_pkt_reformat *pkt_reformat; |
|---|
| 3270 | + struct mlx5_flow_root_namespace *root; |
|---|
| 3271 | + int err; |
|---|
| 3272 | + |
|---|
| 3273 | + root = get_root_namespace(dev, ns_type); |
|---|
| 3274 | + if (!root) |
|---|
| 3275 | + return ERR_PTR(-EOPNOTSUPP); |
|---|
| 3276 | + |
|---|
| 3277 | + pkt_reformat = kzalloc(sizeof(*pkt_reformat), GFP_KERNEL); |
|---|
| 3278 | + if (!pkt_reformat) |
|---|
| 3279 | + return ERR_PTR(-ENOMEM); |
|---|
| 3280 | + |
|---|
| 3281 | + pkt_reformat->ns_type = ns_type; |
|---|
| 3282 | + pkt_reformat->reformat_type = reformat_type; |
|---|
| 3283 | + err = root->cmds->packet_reformat_alloc(root, reformat_type, size, |
|---|
| 3284 | + reformat_data, ns_type, |
|---|
| 3285 | + pkt_reformat); |
|---|
| 3286 | + if (err) { |
|---|
| 3287 | + kfree(pkt_reformat); |
|---|
| 3288 | + return ERR_PTR(err); |
|---|
| 3289 | + } |
|---|
| 3290 | + |
|---|
| 3291 | + return pkt_reformat; |
|---|
| 3292 | +} |
|---|
| 3293 | +EXPORT_SYMBOL(mlx5_packet_reformat_alloc); |
|---|
| 3294 | + |
|---|
| 3295 | +void mlx5_packet_reformat_dealloc(struct mlx5_core_dev *dev, |
|---|
| 3296 | + struct mlx5_pkt_reformat *pkt_reformat) |
|---|
| 3297 | +{ |
|---|
| 3298 | + struct mlx5_flow_root_namespace *root; |
|---|
| 3299 | + |
|---|
| 3300 | + root = get_root_namespace(dev, pkt_reformat->ns_type); |
|---|
| 3301 | + if (WARN_ON(!root)) |
|---|
| 3302 | + return; |
|---|
| 3303 | + root->cmds->packet_reformat_dealloc(root, pkt_reformat); |
|---|
| 3304 | + kfree(pkt_reformat); |
|---|
| 3305 | +} |
|---|
| 3306 | +EXPORT_SYMBOL(mlx5_packet_reformat_dealloc); |
|---|
| 3307 | + |
|---|
| 3308 | +int mlx5_flow_namespace_set_peer(struct mlx5_flow_root_namespace *ns, |
|---|
| 3309 | + struct mlx5_flow_root_namespace *peer_ns) |
|---|
| 3310 | +{ |
|---|
| 3311 | + if (peer_ns && ns->mode != peer_ns->mode) { |
|---|
| 3312 | + mlx5_core_err(ns->dev, |
|---|
| 3313 | + "Can't peer namespace of different steering mode\n"); |
|---|
| 3314 | + return -EINVAL; |
|---|
| 3315 | + } |
|---|
| 3316 | + |
|---|
| 3317 | + return ns->cmds->set_peer(ns, peer_ns); |
|---|
| 3318 | +} |
|---|
| 3319 | + |
|---|
| 3320 | +/* This function should be called only at init stage of the namespace. |
|---|
| 3321 | + * It is not safe to call this function while steering operations |
|---|
| 3322 | + * are executed in the namespace. |
|---|
| 3323 | + */ |
|---|
| 3324 | +int mlx5_flow_namespace_set_mode(struct mlx5_flow_namespace *ns, |
|---|
| 3325 | + enum mlx5_flow_steering_mode mode) |
|---|
| 3326 | +{ |
|---|
| 3327 | + struct mlx5_flow_root_namespace *root; |
|---|
| 3328 | + const struct mlx5_flow_cmds *cmds; |
|---|
| 3329 | + int err; |
|---|
| 3330 | + |
|---|
| 3331 | + root = find_root(&ns->node); |
|---|
| 3332 | + if (&root->ns != ns) |
|---|
| 3333 | + /* Can't set cmds to non root namespace */ |
|---|
| 3334 | + return -EINVAL; |
|---|
| 3335 | + |
|---|
| 3336 | + if (root->table_type != FS_FT_FDB) |
|---|
| 3337 | + return -EOPNOTSUPP; |
|---|
| 3338 | + |
|---|
| 3339 | + if (root->mode == mode) |
|---|
| 3340 | + return 0; |
|---|
| 3341 | + |
|---|
| 3342 | + if (mode == MLX5_FLOW_STEERING_MODE_SMFS) |
|---|
| 3343 | + cmds = mlx5_fs_cmd_get_dr_cmds(); |
|---|
| 3344 | + else |
|---|
| 3345 | + cmds = mlx5_fs_cmd_get_fw_cmds(); |
|---|
| 3346 | + if (!cmds) |
|---|
| 3347 | + return -EOPNOTSUPP; |
|---|
| 3348 | + |
|---|
| 3349 | + err = cmds->create_ns(root); |
|---|
| 3350 | + if (err) { |
|---|
| 3351 | + mlx5_core_err(root->dev, "Failed to create flow namespace (%d)\n", |
|---|
| 3352 | + err); |
|---|
| 3353 | + return err; |
|---|
| 3354 | + } |
|---|
| 3355 | + |
|---|
| 3356 | + root->cmds->destroy_ns(root); |
|---|
| 3357 | + root->cmds = cmds; |
|---|
| 3358 | + root->mode = mode; |
|---|
| 3359 | + |
|---|
| 3360 | + return 0; |
|---|
| 3361 | +} |
|---|