Add raw ethernet local loopback
Subsystems (5)
cover Add raw ethernet local loopback
Hi Doug,
The following patch set from Huy adds ability to disable/enable raw ethernet loopbacks based on actual number of user contexts.
The one or less user space context simply says that no actual consumer of this packets and they don't need to be forwarded to the vport.
After this patch set, the raw ethernet unicast and multicast local loopbacks are disabled by default and packets are forwarded to the next routing level (eswitch, or multihost switch, or out to the wire depending on the configuration).
Available in the "topic/local_lb" topic branch of this git repo:git://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git
Or for browsing:
https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/log/?h=topic/local_lb
Possible merge conflicts resolution is available in:
https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/log/?h=testing/queue-next
Huy Nguyen (3):
net/mlx5: Add raw ethernet local loopback firmware command IB/mlx5: Add raw ethernet local loopback support net/mlx5e: Enable local loopback in loopback selftest
drivers/infiniband/hw/mlx5/main.c | 51 ++++++++++++++++-- drivers/infiniband/hw/mlx5/mlx5_ib.h | 4 ++ .../net/ethernet/mellanox/mlx5/core/en_selftest.c | 13 +++++ drivers/net/ethernet/mellanox/mlx5/core/main.c | 19 +++++++ drivers/net/ethernet/mellanox/mlx5/core/vport.c | 62 ++++++++++++++++++++++ include/linux/mlx5/mlx5_ifc.h | 11 ++-- include/linux/mlx5/vport.h | 3 +- 7 files changed, 155 insertions(+), 8 deletions(-)
--
2.12.2
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
A message that is not in the archive (sent off-list or not yet received): 20170530064255.3745-1-leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
PATCH 1/3 net/mlx5: Add raw ethernet local loopback firmware command
From: Huy Nguyen <huyn-VPRAkNaXOzVWk0Htik3J/[email protected]>
Add support for raw ethernet local loopback firmware command.
--- drivers/net/ethernet/mellanox/mlx5/core/vport.c | 62 +++++++++++++++++++++++++ include/linux/mlx5/mlx5_ifc.h | 11 +++-- include/linux/mlx5/vport.h | 3 +- 3 files changed, 72 insertions(+), 4 deletions(-)
drivers/net/ethernet/mellanox/mlx5/core/vport.c +62 -0
@@ -901,6 +901,68 @@ int mlx5_modify_nic_vport_promisc(struct mlx5_core_dev *mdev, }EXPORT_SYMBOL_GPL (mlx5_modify_nic_vport_promisc); +enum { +UC_LOCAL_LB , +MC_LOCAL_LB +}; + +int mlx5_nic_vport_update_local_lb (struct mlx5_core_dev *mdev,bool enable) +{ +int inlen =MLX5_ST_SZ_BYTES (modify_nic_vport_context_in); +void *in; +int err; + +mlx5_core_dbg (mdev,"%s local_lb\n" , enable ?"enable" :"disable" ); + in =kvzalloc (inlen,GFP_KERNEL ); +if (!in) +return -ENOMEM ; + +MLX5_SET (modify_nic_vport_context_in, in, + field_select.disable_mc_local_lb,1 ); +MLX5_SET (modify_nic_vport_context_in, in, + nic_vport_context.disable_mc_local_lb, !enable); + +MLX5_SET (modify_nic_vport_context_in, in, + field_select.disable_uc_local_lb,1 ); +MLX5_SET (modify_nic_vport_context_in, in, + nic_vport_context.disable_uc_local_lb, !enable); + + err =mlx5_modify_nic_vport_context (mdev, in, inlen); + +kvfree (in); +return err; +} +EXPORT_SYMBOL_GPL (mlx5_nic_vport_update_local_lb); + +int mlx5_nic_vport_query_local_lb (struct mlx5_core_dev *mdev,bool *status) +{ +int outlen =MLX5_ST_SZ_BYTES (query_nic_vport_context_out); +u32 *out; +int value; +int err; + + out =kzalloc (outlen,GFP_KERNEL ); +if (!out) +return -ENOMEM ; + + err =mlx5_query_nic_vport_context (mdev,0 , out, outlen); +if (err) + goto out; + + value =MLX5_GET (query_nic_vport_context_out, out, + nic_vport_context.disable_mc_local_lb) <<MC_LOCAL_LB ; + + value |=MLX5_GET (query_nic_vport_context_out, out, + nic_vport_context.disable_uc_local_lb) <<UC_LOCAL_LB ; + + *status = !value; + +out: +kfree (out); +return err; +} +EXPORT_SYMBOL_GPL (mlx5_nic_vport_query_local_lb); +enum mlx5_vport_roce_state {MLX5_VPORT_ROCE_DISABLED =0 ,MLX5_VPORT_ROCE_ENABLED =1 ,
include/linux/mlx5/mlx5_ifc.h +8 -3
@@ -994,7 +994,8 @@ struct mlx5_ifc_cmd_hca_cap_bits {u8 log_max_wq_sz[0x5 ];u8 nic_vport_change_event[0x1 ]; -u8 reserved_at_3e1[0xa ]; +u8 disable_local_lb[0x1 ]; +u8 reserved_at_3e2[0x9 ];u8 log_max_vlan_list[0x5 ];u8 reserved_at_3f0[0x3 ];u8 log_max_current_mc_list[0x5 ];
@@ -2538,7 +2539,9 @@ struct mlx5_ifc_rmpc_bits {struct mlx5_ifc_nic_vport_context_bits {u8 reserved_at_0[0x5 ];u8 min_wqe_inline_mode[0x3 ]; -u8 reserved_at_8[0x17 ]; +u8 reserved_at_8[0x15 ]; +u8 disable_mc_local_lb[0x1 ]; +u8 disable_uc_local_lb[0x1 ];u8 roce_en[0x1 ];u8 arm_change_event[0x1 ];
@@ -5197,7 +5200,9 @@ struct mlx5_ifc_modify_nic_vport_context_out_bits { };struct mlx5_ifc_modify_nic_vport_field_select_bits { -u8 reserved_at_0[0x16 ]; +u8 reserved_at_0[0x14 ]; +u8 disable_uc_local_lb[0x1 ]; +u8 disable_mc_local_lb[0x1 ];u8 node_guid[0x1 ];u8 port_guid[0x1 ];u8 min_inline[0x1 ];
include/linux/mlx5/vport.h +2 -1
@@ -114,5 +114,6 @@ int mlx5_core_modify_hca_vport_context(struct mlx5_core_dev *dev, u8 other_vport, u8 port_num, int vf, struct mlx5_hca_vport_context *req); - +int mlx5_nic_vport_update_local_lb (struct mlx5_core_dev *mdev,bool enable); +int mlx5_nic_vport_query_local_lb (struct mlx5_core_dev *mdev,bool *status);#endif /* __MLX5_VPORT_H__ */
Signature · 6 lines
2.12.2 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
PATCH 2/3 IB/mlx5: Add raw ethernet local loopback support
In reply to a message not in the archiveFrom: Huy Nguyen <huyn-VPRAkNaXOzVWk0Htik3J/[email protected]>
Currently, unicast/multicast loopback raw ethernet (non-RDMA) packets are sent back to the vport. A unicast loopback packet is the packet with destination MAC address the same as the source MAC address. For multicast, the destination MAC address is in the vport's multicast filter list.
Moreover, the local loopback is not needed if
there is one or none user space context.
After this patch, the raw ethernet unicast and multicast local loopback are disabled by default. When there is more than one user space context, the local loopback is enabled.
Note that when local loopback is disabled, raw ethernet packets are not looped back to the vport and are forwarded to the next routing level (eswitch, or multihost switch, or out to the wire depending on the configuration).
--- drivers/infiniband/hw/mlx5/main.c | 51 ++++++++++++++++++++++++-- drivers/infiniband/hw/mlx5/mlx5_ib.h | 4 ++ drivers/net/ethernet/mellanox/mlx5/core/main.c | 19 ++++++++++ 3 files changed, 70 insertions(+), 4 deletions(-)
drivers/infiniband/hw/mlx5/main.c +47 -4
@@ -58,6 +58,7 @@#include <linux/mlx5/vport.h> #include "mlx5_ib.h" #include "cmd.h" +#include <linux/mlx5/vport.h> #define DRIVER_NAME "mlx5_ib"#define DRIVER_VERSION "2.2-1"
@@ -1201,6 +1202,45 @@ static int deallocate_uars(struct mlx5_ib_dev *dev, struct mlx5_ib_ucontext *conreturn 0 ; } +static int mlx5_ib_alloc_transport_domain (struct mlx5_ib_dev *dev,u32 *tdn) +{ +int err; + + err =mlx5_core_alloc_transport_domain (dev->mdev, tdn); +if (err) +return err; + +if ((MLX5_CAP_GEN (dev->mdev, port_type) !=MLX5_CAP_PORT_TYPE_ETH ) || + !MLX5_CAP_GEN (dev->mdev, disable_local_lb)) +return err; + +mutex_lock (&dev->lb_mutex); + dev->user_td++; + +if (dev->user_td ==2 ) + err =mlx5_nic_vport_update_local_lb (dev->mdev, true); + +mutex_unlock (&dev->lb_mutex); +return err; +} + +static void mlx5_ib_dealloc_transport_domain (struct mlx5_ib_dev *dev,u32 tdn) +{ +mlx5_core_dealloc_transport_domain (dev->mdev, tdn); + +if ((MLX5_CAP_GEN (dev->mdev, port_type) !=MLX5_CAP_PORT_TYPE_ETH ) || + !MLX5_CAP_GEN (dev->mdev, disable_local_lb)) +return ; + +mutex_lock (&dev->lb_mutex); + dev->user_td--; + +if (dev->user_td <2 ) +mlx5_nic_vport_update_local_lb (dev->mdev, false); + +mutex_unlock (&dev->lb_mutex); +} +static struct ib_ucontext *mlx5_ib_alloc_ucontext (struct ib_device *ibdev,struct ib_udata *udata) {
@@ -1309,8 +1349,7 @@ static struct ib_ucontext *mlx5_ib_alloc_ucontext(struct ib_device *ibdev,mutex_init (&context->upd_xlt_page_mutex);if (MLX5_CAP_GEN (dev->mdev, log_max_transport_domain)) { - err =mlx5_core_alloc_transport_domain (dev->mdev, - &context->tdn); + err =mlx5_ib_alloc_transport_domain (dev, &context->tdn);if (err) goto out_page; }
@@ -1376,7 +1415,7 @@ static struct ib_ucontext *mlx5_ib_alloc_ucontext(struct ib_device *ibdev, out_td:if (MLX5_CAP_GEN (dev->mdev, log_max_transport_domain)) -mlx5_core_dealloc_transport_domain (dev->mdev, context->tdn); +mlx5_ib_dealloc_transport_domain (dev, context->tdn); out_page:free_page (context->upd_xlt_page);
@@ -1404,7 +1443,7 @@ static int mlx5_ib_dealloc_ucontext(struct ib_ucontext *ibcontext) bfregi = &context->bfregi;if (MLX5_CAP_GEN (dev->mdev, log_max_transport_domain)) -mlx5_core_dealloc_transport_domain (dev->mdev, context->tdn); +mlx5_ib_dealloc_transport_domain (dev, context->tdn);free_page (context->upd_xlt_page);deallocate_uars (dev, context);
@@ -3788,6 +3827,10 @@ static void *mlx5_ib_add(struct mlx5_core_dev *mdev) goto err_umrc; } +if ((MLX5_CAP_GEN (mdev, port_type) ==MLX5_CAP_PORT_TYPE_ETH ) && +MLX5_CAP_GEN (mdev, disable_local_lb)) +mutex_init (&dev->lb_mutex); + dev->ib_active = true;return dev;
drivers/infiniband/hw/mlx5/mlx5_ib.h +4 -0
@@ -654,6 +654,10 @@ struct mlx5_ib_dev {struct mlx5_ib_port *port;struct mlx5_sq_bfreg bfreg;struct mlx5_sq_bfreg fp_bfreg; + +/* protect the user_td */ +struct mutex lb_mutex; +u32 user_td; };static inline struct mlx5_ib_cq *to_mibcq (struct mlx5_core_cq *mcq)
drivers/net/ethernet/mellanox/mlx5/core/main.c +19 -0
@@ -47,6 +47,7 @@#include <linux/debugfs.h> #include <linux/kmod.h> #include <linux/mlx5/mlx5_ifc.h> +#include <linux/mlx5/vport.h> #ifdef CONFIG_RFS_ACCEL #include <linux/cpu_rmap.h> #endif
@@ -574,6 +575,18 @@ static int set_hca_ctrl(struct mlx5_core_dev *dev)return err; } +static int mlx5_core_set_hca_defaults (struct mlx5_core_dev *dev) +{ +int ret =0 ; + +/* Disable local_lb by default */ +if ((MLX5_CAP_GEN (dev, port_type) ==MLX5_CAP_PORT_TYPE_ETH ) && +MLX5_CAP_GEN (dev, disable_local_lb)) + ret =mlx5_nic_vport_update_local_lb (dev, false); + +return ret; +} +int mlx5_core_enable_hca (struct mlx5_core_dev *dev,u16 func_id) {u32 out[MLX5_ST_SZ_DW (enable_hca_out)] = {0 };
@@ -1137,6 +1150,12 @@ static int mlx5_load_one(struct mlx5_core_dev *dev, struct mlx5_priv *priv, goto err_fs; } + err =mlx5_core_set_hca_defaults (dev); +if (err) { +dev_err (&pdev->dev,"Failed to set hca defaults\n" ); + goto err_fs; + } +#ifdef CONFIG_MLX5_CORE_EN mlx5_eswitch_attach (dev->priv.eswitch);#endif
--
2.12.2
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
PATCH 3/3 net/mlx5e: Enable local loopback in loopback selftest
In reply to a message not in the archiveFrom: Huy Nguyen <huyn-VPRAkNaXOzVWk0Htik3J/[email protected]>
Before running the ethtool's loopback selftest, we need to make sure that the local loopback is enabled.
--- drivers/net/ethernet/mellanox/mlx5/core/en_selftest.c | 13 +++++++++++++ 1 file changed, 13 insertions(+)
drivers/net/ethernet/mellanox/mlx5/core/en_selftest.c +13 -0
@@ -189,6 +189,7 @@ struct mlx5e_lbt_priv {struct packet_type pt;struct completion comp;bool loopback_ok; +bool local_lb; }; static int
@@ -236,6 +237,13 @@ static int mlx5e_test_loopback_setup(struct mlx5e_priv *priv, {int err =0 ; +/* Temporarily enable local_lb */ +if (MLX5_CAP_GEN (priv->mdev, disable_local_lb)) { +mlx5_nic_vport_query_local_lb (priv->mdev, &lbtp->local_lb); +if (!lbtp->local_lb) +mlx5_nic_vport_update_local_lb (priv->mdev, true); + } + err =mlx5e_refresh_tirs (priv, true);if (err)return err;
@@ -254,6 +262,11 @@ static int mlx5e_test_loopback_setup(struct mlx5e_priv *priv,static void mlx5e_test_loopback_cleanup (struct mlx5e_priv *priv,struct mlx5e_lbt_priv *lbtp) { +if (MLX5_CAP_GEN (priv->mdev, disable_local_lb)) { +if (!lbtp->local_lb) +mlx5_nic_vport_update_local_lb (priv->mdev, false); + } +dev_remove_pack (&lbtp->pt);mlx5e_refresh_tirs (priv, false); }
Signature · 6 lines
2.12.2 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
Doug Ledford
AppliedQuoted 9 lines · Leon Romanovsky
Hi Doug,
The following patch set from Huy adds ability to disable/enable raw ethernet loopbacks based on actual number of user contexts.
The one or less user space context simply says that no actual consumer
of this packets and they don't need to be forwarded to the vport.
Quoted 8 lines
After this patch set, the raw ethernet unicast and multicast local loopbacks are disabled by default and packets are forwarded to the next routing level (eswitch, or multihost switch, or out to the
wire depending on the configuration).Available in the "topic/local_lb" topic branch of this git repo:
git://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git
Thanks, applied.
Signature · 8 lines
Doug Ledford <[email protected]> GPG KeyID: B826A3330E572FDD Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html