netdev - Re: [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 22 Aug 2016 15:26:06 -0400
From:   "John W. Linville" <linville@...driver.com>
To:     Vidya Sagar Ravipati <vidya@...ulusnetworks.com>
Cc:     netdev@...r.kernel.org, davem@...emloft.net, decot@...glers.com,
        bkenward@...arflare.com, daniel@...earbox.net, galp@...lanox.com,
        roopa@...ulusnetworks.com, gospo@...ulusnetworks.com,
        dustin@...ulusnetworks.com, ben@...adent.org.uk
Subject: Re: [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net

On Sun, Aug 21, 2016 at 04:44:01PM -0700, Vidya Sagar Ravipati wrote:
> From: Vidya Sagar Ravipati <vidya@...ulusnetworks.com>
> 
> This covers kernel changes upto:
> 
> commit 3851112e4737cd52aaeda0ce8d084be9ee128106
> Author: Vidya Sagar Ravipati <vidya@...ulusnetworks.com>
> Date:   Sun May 22 23:59:00 2016 -0700
> 
>     ethtool: add support for 25G/50G/100G speed modes
> 
>     This patch enhances ethtool link mode bitmap to include
>     25G/50G/100G speed along with interface modes

There is one more change since then:

commit 89da45b8b5b2187734a11038b8593714f964ffd1
Author: Gal Pressman <galp@...lanox.com>
Date:   Thu Jun 23 17:02:43 2016 +0300

    ethtool: Add 50G baseSR2 link mode
    
    Add ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT bit.
    
    Signed-off-by: Gal Pressman <galp@...lanox.com>
    Signed-off-by: Saeed Mahameed <saeedm@...lanox.com>
    Cc: Ben Hutchings <bwh@...nel.org>
    Cc: David Decotigny <decot@...glers.com>
    Acked-By: David Decotigny <decot@...glers.com>
    Signed-off-by: David S. Miller <davem@...emloft.net>

Might as well include that here and in the patch that uses the
new bits?

> Signed-off-by: Vidya Sagar Ravipati <vidya@...ulusnetworks.com>
> Signed-off-by: David S. Miller <davem@...emloft.net>
> ---
>  ethtool-copy.h | 17 +++++++++++++----
>  1 file changed, 13 insertions(+), 4 deletions(-)
> 
> diff --git a/ethtool-copy.h b/ethtool-copy.h
> index 7c581ea..7817779 100644
> --- a/ethtool-copy.h
> +++ b/ethtool-copy.h
> @@ -1351,6 +1351,15 @@ enum ethtool_link_mode_bit_indices {
>  	ETHTOOL_LINK_MODE_56000baseCR4_Full_BIT	= 28,
>  	ETHTOOL_LINK_MODE_56000baseSR4_Full_BIT	= 29,
>  	ETHTOOL_LINK_MODE_56000baseLR4_Full_BIT	= 30,
> +	ETHTOOL_LINK_MODE_25000baseCR_Full_BIT	= 31,
> +	ETHTOOL_LINK_MODE_25000baseKR_Full_BIT	= 32,
> +	ETHTOOL_LINK_MODE_25000baseSR_Full_BIT	= 33,
> +	ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT	= 34,
> +	ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT	= 35,
> +	ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT	= 36,
> +	ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT	= 37,
> +	ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT	= 38,
> +	ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT	= 39,
>  
>  	/* Last allowed bit for __ETHTOOL_LINK_MODE_LEGACY_MASK is bit
>  	 * 31. Please do NOT define any SUPPORTED_* or ADVERTISED_*
> @@ -1359,7 +1368,7 @@ enum ethtool_link_mode_bit_indices {
>  	 */
>  
>  	__ETHTOOL_LINK_MODE_LAST
> -	  = ETHTOOL_LINK_MODE_56000baseLR4_Full_BIT,
> +	  = ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
>  };
>  
>  #define __ETHTOOL_LINK_MODE_LEGACY_MASK(base_name)	\
> @@ -1646,9 +1655,9 @@ enum ethtool_reset_flags {
>   *	%ETHTOOL_GLINKSETTINGS: on entry, number of words passed by user
>   *	(>= 0); on return, if handshake in progress, negative if
>   *	request size unsupported by kernel: absolute value indicates
> - *	kernel recommended size and cmd field is 0, as well as all the
> - *	other fields; otherwise (handshake completed), strictly
> - *	positive to indicate size used by kernel and cmd field is
> + *	kernel expected size and all the other fields but cmd
> + *	are 0; otherwise (handshake completed), strictly positive
> + *	to indicate size used by kernel and cmd field stays
>   *	%ETHTOOL_GLINKSETTINGS, all other fields populated by driver. For
>   *	%ETHTOOL_SLINKSETTINGS: must be valid on entry, ie. a positive
>   *	value returned previously by %ETHTOOL_GLINKSETTINGS, otherwise
> -- 
> 2.1.4
> 
> 

-- 
John W. Linville		Someday the world will need a hero, and you
linville@...driver.com			might be all we have.  Be ready.