Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b6d9ccb1 authored by Mark Bloch's avatar Mark Bloch Committed by Leon Romanovsky
Browse files

net/mlx5: E-Switch, don't use hardcoded values for FDB prios 



When creating the FDB prios, use the enum values already defined and not
the hardcoded values.

Signed-off-by: default avatarMark Bloch <markb@mellanox.com>
Reviewed-by: default avatarMaor Gottlieb <maorg@mellanox.com>
Signed-off-by: default avatarLeon Romanovsky <leonro@mellanox.com>
parent aef6c443

drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c

+0 −5
Original line number Diff line number Diff line
@@ -41,11 +41,6 @@ 
#include "fs_core.h"

#include "lib/devcom.h"



enum {

	FDB_FAST_PATH = 0,

	FDB_SLOW_PATH

};



/* There are two match-all miss flows, one for unicast dst mac and

 * one for multicast.

 */

drivers/net/ethernet/mellanox/mlx5/core/fs_core.c

+3 −2
Original line number Diff line number Diff line
@@ -2479,7 +2479,8 @@ static int init_fdb_root_ns(struct mlx5_flow_steering *steering) 
		return -ENOMEM;



	levels = 2 * FDB_MAX_PRIO * (FDB_MAX_CHAIN + 1);

	maj_prio = fs_create_prio_chained(&steering->fdb_root_ns->ns, 0,

	maj_prio = fs_create_prio_chained(&steering->fdb_root_ns->ns,

					  FDB_FAST_PATH,

					  levels);

	if (IS_ERR(maj_prio)) {

		err = PTR_ERR(maj_prio);
@@ -2504,7 +2505,7 @@ static int init_fdb_root_ns(struct mlx5_flow_steering *steering) 
		steering->fdb_sub_ns[chain] = ns;

	}



	maj_prio = fs_create_prio(&steering->fdb_root_ns->ns, 1, 1);

	maj_prio = fs_create_prio(&steering->fdb_root_ns->ns, FDB_SLOW_PATH, 1);

	if (IS_ERR(maj_prio)) {

		err = PTR_ERR(maj_prio);

		goto out_err;

include/linux/mlx5/fs.h

+5 −0
Original line number Diff line number Diff line
@@ -75,6 +75,11 @@ enum mlx5_flow_namespace_type { 
	MLX5_FLOW_NAMESPACE_EGRESS,

};



enum {

	FDB_FAST_PATH,

	FDB_SLOW_PATH,

};



struct mlx5_flow_table;

struct mlx5_flow_group;

struct mlx5_flow_namespace;