From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f170.google.com (mail-pf1-f170.google.com [209.85.210.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1DE6A3F074A for ; Wed, 18 Mar 2026 20:11:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773864674; cv=none; b=cTp7NbCKNQUBM4A+d6HQEYb/lzgSgYOM5WVpEEv7WhDLb9bblmvG51Pepn75RXaJ6gqeg72Nm+oqhML7tXAxG2sadRN9z5ct3df2GROp6Qh6Eoeod9FQd8rRzSiD9vH58xWA+MFqOjrh40Sqnb+77Pvo4CGggQp/juShWEBgeKg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773864674; c=relaxed/simple; bh=7V+tpJcHPHKkKArRby9B/AEvv+A6I5U8x69esMrCnRU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=TLhZNqHUGisYKq9Fs+4oRpbKj2EaGBkqYLF7uKX5UBJldhlUUBAQmxxxpZcBrVOpz37tO/oqhsVX+bvavhaMbL4cWTZtLxq2c77sOQZsZqMM6WxLzA3Q/ZQqpZ9AE2EuqjnFkFUXnfyYEgHu9/McODS5U6S4SN24PmpQyQuZ13c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=dama.to; spf=none smtp.mailfrom=dama.to; dkim=pass (2048-bit key) header.d=dama-to.20230601.gappssmtp.com header.i=@dama-to.20230601.gappssmtp.com header.b=DkWczbrN; arc=none smtp.client-ip=209.85.210.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=dama.to Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=dama.to Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=dama-to.20230601.gappssmtp.com header.i=@dama-to.20230601.gappssmtp.com header.b="DkWczbrN" Received: by mail-pf1-f170.google.com with SMTP id d2e1a72fcca58-82a7ebc729dso5500b3a.3 for ; Wed, 18 Mar 2026 13:11:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dama-to.20230601.gappssmtp.com; s=20230601; t=1773864672; x=1774469472; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=tcrjf6o6BBsskimuOBs0/X8mHwc2XWl98zNiH3etscg=; b=DkWczbrNIQVEjlE5M+pjSo5y5lJ+by1T8j93RepF68fSx2hJNsN3z/LP14spf2xTxD GjHNvBYIMtS9I1INUMEa9oAH70XyhYqcm42elsdox7gMSJoDGd57sSjTB5mSan9qvCsV eFlvtmd5JdQXQs1kjwSyZQPsYqDcKk4aFGRq7Dg4FgPkLrQBkMQ35JwbJbhSV8s2I33p ndebk4WsWZ/wndb1k/sYfJTDKIYBd//H3X325h17nxGmnatVLHabxX2djDTegEdlLaOq jvApSI+LinleIAO+nE8hXcutT4HnR+QMWsgbX152KBdlxgJt5fhGeLLe9LsBOYOLnIdH 05zw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773864672; x=1774469472; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=tcrjf6o6BBsskimuOBs0/X8mHwc2XWl98zNiH3etscg=; b=B4xaCPfUffCBLWfD2mc2w1Po5VQn/0VwIKi7neBQTwPf5HblsKID8WkwF9m6B6oWaJ kuOirsVf3vYLOKeD07XS5j37ygsUMilWaTQFzkpk2w0y5W0FeJuzeydtLEOS6jKZWlQc aoEmRjeWqTPFi1WSy0lOHFIrt45WxtpkdXo24dO8/t0akI5qkfvR+QtkeVyWuouf/pqf ZrTkQ0tkF+PyNui/0bN2QxhIG1thklHvCr6sumxssqVYss+gBvF33v6S/ga1N9aiMb9q 7hshX4kTqUcWVepxohAEXdriMYuTyzOg+JF9YYOfAQFLoxnLaNKu4pJU1bpNT3SSaybb yp3Q== X-Forwarded-Encrypted: i=1; AJvYcCWVxKkklpjwT0x5eYStB/3Wp7OcTDylY9n5G/VrnsedKRKVKUjmEbmee5kcZySkuyHM+4yUzY2ZcsluOXY=@vger.kernel.org X-Gm-Message-State: AOJu0YzTiAiCrXaybrnjDjSKnxP2riY7lIoHPmkNNRYgoH1sO5uHDB7s wXGxzNjNKx0uUkGgt/w6d2GpFpRnsz6u6VDVK689KTanQ8gG5I1cLV7hJ3o38kISrY4= X-Gm-Gg: ATEYQzwm9ft9gXSpHfXbXZuI0H0fo2zczQ4xRy0W7QXpLOZ9tb6TTK0mGOntlfrNEmG gdh/JO0bEB2sYmfoOkeXqZ9JjlpCetijZHJYe60UxfYlmODmZOodzylUCJyKEXBHocV8Jzm1Vfu CNK0Lwp8gpmGgGZ8ej8K6v9fc/HWXcoBCslr7pB9vpbN+H9UIOVlxSm0DCu+Xwnrwq/aDnKwM1g TIV8VIl4sWKfaQtrubCu9yCKqc7wpmCfUcQ9hIVXEVGRgcXNPCff2oO1PSjrxBnVlnan1q8UGrJ B85YUnMPDy6pwG7cIpgMsFdWCIdlYuO9h3yRN9EktW3+WeXjK5Z5XJBW2JxSMNlUd5mWL60++t1 izQoB43tDsof97P6QWSJD7Q5RWg8pPtH7DLJwljBXL8UTUr+wOr6Kcbuk1bBH4wUOJ2pJolU/x0 pWqLo= X-Received: by 2002:a05:6a00:2183:b0:829:9c9c:1720 with SMTP id d2e1a72fcca58-82a6ae70a0dmr4132616b3a.38.1773864672197; Wed, 18 Mar 2026 13:11:12 -0700 (PDT) Received: from localhost ([2a03:2880:2ff:8::]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82a6bbe1869sm4227445b3a.40.2026.03.18.13.11.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Mar 2026 13:11:11 -0700 (PDT) Date: Wed, 18 Mar 2026 13:11:11 -0700 From: Joe Damato To: Tariq Toukan Cc: Eric Dumazet , Jakub Kicinski , Paolo Abeni , Andrew Lunn , "David S. Miller" , Saeed Mahameed , Mark Bloch , Leon Romanovsky , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, Gal Pressman , Moshe Shemesh , Nimrod Oren , Carolina Jubran , Dragos Tatulea Subject: Re: [PATCH net-next] net/mlx5e: Add hds-thresh query support via ethtool Message-ID: Mail-Followup-To: Joe Damato , Tariq Toukan , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Andrew Lunn , "David S. Miller" , Saeed Mahameed , Mark Bloch , Leon Romanovsky , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, Gal Pressman , Moshe Shemesh , Nimrod Oren , Carolina Jubran , Dragos Tatulea References: <20260317104934.16124-1-tariqt@nvidia.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260317104934.16124-1-tariqt@nvidia.com> On Tue, Mar 17, 2026 at 12:49:34PM +0200, Tariq Toukan wrote: > From: Nimrod Oren > > Add support for reporting HDS (Header-Data Split) threshold via > ethtool. When applicable, mlx5 hardware splits packets of all sizes with > no configurable threshold, so report both hds-thresh and hds-thresh-max > as 0 (i.e. always split regardless of size). > > Signed-off-by: Nimrod Oren > Reviewed-by: Carolina Jubran > Reviewed-by: Dragos Tatulea > Signed-off-by: Tariq Toukan > --- > drivers/net/ethernet/mellanox/mlx5/core/en_ethtool.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_ethtool.c b/drivers/net/ethernet/mellanox/mlx5/core/en_ethtool.c > index 4a8dc85d5924..bb61e2179078 100644 > --- a/drivers/net/ethernet/mellanox/mlx5/core/en_ethtool.c > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_ethtool.c > @@ -371,6 +371,9 @@ void mlx5e_ethtool_get_ringparam(struct mlx5e_priv *priv, > param->tx_max_pending = 1 << MLX5E_PARAMS_MAXIMUM_LOG_SQ_SIZE; > param->rx_pending = 1 << priv->channels.params.log_rq_mtu_frames; > param->tx_pending = 1 << priv->channels.params.log_sq_size; > + > + kernel_param->hds_thresh = 0; I think this is populated by the core before the call to get_ringparam and it looks like it is defaulted to 0, so I think this assignment is unnecessary. But everything else seems fine, so: Reviewed-by: Joe Damato