From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [RFC PATCH net-next 0/2] Configuring PFC stall prevention via ethtool Date: Thu, 16 Nov 2017 03:44:29 +0100 Message-ID: <20171116024429.GG2130@lunn.ch> References: <1510772411-17054-1-git-send-email-eranbe@mellanox.com> <20171116010722.GD2130@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, "David S. Miller" , "John W. Linville" , Saeed Mahameed , Gal Pressman , Ariel Almog , Inbar Karmy To: Eran Ben Elisha Return-path: Received: from vps0.lunn.ch ([185.16.172.187]:56181 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758331AbdKPCod (ORCPT ); Wed, 15 Nov 2017 21:44:33 -0500 Content-Disposition: inline In-Reply-To: <20171116010722.GD2130@lunn.ch> Sender: netdev-owner@vger.kernel.org List-ID: > What do other vendors support? Time? Number of pause frames sent? So i checked a few Marvell Switches. You can also specify a time. It is a little bit more complex than that, since the units of time depend on the link speed. But converting a time in ms to what the register wants is possible. So i'm thinking rather than a poorly defined 'Auto', passing a time would be better. Andrew