Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 10 Oct 2019 16:04:43 +0000 (UTC)
From:      Hans Petter Selasky <hselasky@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-releng@freebsd.org
Subject:   svn commit: r353407 - releng/12.1/sys/dev/mlx5/mlx5_en
Message-ID:  <201910101604.x9AG4hQB012977@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: hselasky
Date: Thu Oct 10 16:04:43 2019
New Revision: 353407
URL: https://svnweb.freebsd.org/changeset/base/353407

Log:
  MFS r353405:
  Fix regression issue after r352989:
  
  As noted by the commit message, callouts are now persistant
  and should not be in the auto-zero section of the RQ's and SQ's.
  This fixes an assert when using the TX completion event
  factor feature with mlx5en(4).
  
  Found by:	gallatin@
  Sponsored by:	Mellanox Technologies
  Approved by:	re (gjb)

Modified:
  releng/12.1/sys/dev/mlx5/mlx5_en/en.h
Directory Properties:
  releng/12.1/   (props changed)

Modified: releng/12.1/sys/dev/mlx5/mlx5_en/en.h
==============================================================================
--- releng/12.1/sys/dev/mlx5/mlx5_en/en.h	Thu Oct 10 15:38:08 2019	(r353406)
+++ releng/12.1/sys/dev/mlx5/mlx5_en/en.h	Thu Oct 10 16:04:43 2019	(r353407)
@@ -747,6 +747,7 @@ struct mlx5e_rq {
 	/* persistant fields */
 	struct mtx mtx;
 	struct mlx5e_rq_stats stats;
+	struct callout watchdog;
 
 	/* data path */
 #define	mlx5e_rq_zero_start wq
@@ -768,7 +769,6 @@ struct mlx5e_rq {
 	struct mlx5_wq_ctrl wq_ctrl;
 	u32	rqn;
 	struct mlx5e_channel *channel;
-	struct callout watchdog;
 } __aligned(MLX5E_CACHELINE_SIZE);
 
 struct mlx5e_sq_mbuf {
@@ -793,6 +793,7 @@ struct mlx5e_sq {
 	struct	mtx lock;
 	struct	mtx comp_lock;
 	struct	mlx5e_sq_stats stats;
+  	struct	callout cev_callout;
 
 	/* data path */
 #define	mlx5e_sq_zero_start dma_tag
@@ -811,7 +812,6 @@ struct mlx5e_sq {
 #define	MLX5E_CEV_STATE_SEND_NOPS 1	/* send NOPs */
 #define	MLX5E_CEV_STATE_HOLD_NOPS 2	/* don't send NOPs yet */
 	u16	running;		/* set if SQ is running */
-	struct callout cev_callout;
 	union {
 		u32	d32[2];
 		u64	d64;



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