[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20111014205542.GA9707@mcarlson.broadcom.com>
Date: Fri, 14 Oct 2011 13:55:42 -0700
From: "Matt Carlson" <mcarlson@...adcom.com>
To: "Matt Carlson" <mcarlson@...adcom.com>
cc: "davem@...emloft.net" <davem@...emloft.net>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"bhutchings@...arflare.com" <bhutchings@...arflare.com>
Subject: Re: [PATCH RFC 0/2] Add extended pause query capability
On Fri, Oct 14, 2011 at 01:54:00PM -0700, Matt Carlson wrote:
> The current implementation of get_pauseparam allows userspace to query the
> flow control configuration, but not the flow control status. This patchset
> defines a new ethtool_pauseparamext structure and adds a new
> get_pauseparamext ethtool callback to support it. The new facilities allow
> the driver to report both config and status in the same query.
>
> Please note that Ben Hutchings' suggestion to deduce the flow control settings
> through the 'advertising' and 'lp_advertising' from ETHTOOL_GSET was considered,
> but rejected because there was no way to know if the flow control
> advertisements reported were valid.
Please add the following to the patches. Sorry Michael.
Signed-off-by: Michael Chan <mchan@...adcom.com>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists