Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 May 2019 17:23:36 +0000 (UTC)
From:      Hans Petter Selasky <hselasky@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   svn commit: r347813 - stable/11/sys/dev/mlx5/mlx5_en
Message-ID:  <201905161723.x4GHNaJ7004674@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: hselasky
Date: Thu May 16 17:23:36 2019
New Revision: 347813
URL: https://svnweb.freebsd.org/changeset/base/347813

Log:
  MFC r347263:
  Disable CQE zipping by default in mlx5en(4).
  
  After doing performance measurements, it seems like CQE zipping doesn't
  have any significant benefit.
  Moreover, we know that this feature is disabled by default on other
  operating systems (Linux for example).
  
  Submitted by:	slavash@
  Sponsored by:	Mellanox Technologies

Modified:
  stable/11/sys/dev/mlx5/mlx5_en/mlx5_en_main.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/dev/mlx5/mlx5_en/mlx5_en_main.c
==============================================================================
--- stable/11/sys/dev/mlx5/mlx5_en/mlx5_en_main.c	Thu May 16 17:22:57 2019	(r347812)
+++ stable/11/sys/dev/mlx5/mlx5_en/mlx5_en_main.c	Thu May 16 17:23:36 2019	(r347813)
@@ -3061,7 +3061,12 @@ mlx5e_build_ifp_priv(struct mlx5_core_dev *mdev,
 	priv->params.hw_lro_en = false;
 	priv->params.lro_wqe_sz = MLX5E_PARAMS_DEFAULT_LRO_WQE_SZ;
 
-	priv->params.cqe_zipping_en = !!MLX5_CAP_GEN(mdev, cqe_compression);
+	/*
+	 * CQE zipping is currently defaulted to off. when it won't
+	 * anymore we will consider the HW capability:
+	 * "!!MLX5_CAP_GEN(mdev, cqe_compression)"
+	 */
+	priv->params.cqe_zipping_en = false;
 
 	priv->mdev = mdev;
 	priv->params.num_channels = num_comp_vectors;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201905161723.x4GHNaJ7004674>