netdev - Re: [ethtool PATCH v3 4/4] ethtool: Enhancing link mode bits to support 25G/50G/100G

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] [day] [month] [year] [list]

Message-ID: <20160822192729.GC2628@tuxdriver.com>
Date:   Mon, 22 Aug 2016 15:27:30 -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 4/4] ethtool: Enhancing link mode bits to
 support 25G/50G/100G

On Sun, Aug 21, 2016 at 04:44:04PM -0700, Vidya Sagar Ravipati wrote:
> From: Vidya Sagar Ravipati <vidya@...ulusnetworks.com>
> 
> Enhancing link mode bits to support 25G/50G/100G
> for supported and advertised speed mode bits
> 
> Signed-off-by: Vidya Sagar Ravipati <vidya@...ulusnetworks.com>
> Acked-By: David Decotigny <decot@...glers.com>
> ---
>  ethtool.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/ethtool.c b/ethtool.c
> index 1d6564e..5c3c765 100644
> --- a/ethtool.c
> +++ b/ethtool.c
> @@ -512,6 +512,15 @@ static void init_global_link_mode_masks(void)
>  		ETHTOOL_LINK_MODE_56000baseCR4_Full_BIT,
>  		ETHTOOL_LINK_MODE_56000baseSR4_Full_BIT,
>  		ETHTOOL_LINK_MODE_56000baseLR4_Full_BIT,
> +		ETHTOOL_LINK_MODE_25000baseCR_Full_BIT,
> +		ETHTOOL_LINK_MODE_25000baseKR_Full_BIT,
> +		ETHTOOL_LINK_MODE_25000baseSR_Full_BIT,
> +		ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT,
> +		ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT,
> +		ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT,
> +		ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT,
> +		ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT,
> +		ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
>  	};
>  	static const enum ethtool_link_mode_bit_indices
>  		additional_advertised_flags_bits[] = {
> @@ -632,6 +641,24 @@ static void dump_link_caps(const char *prefix, const char *an_prefix,
>  		  "56000baseSR4/Full" },
>  		{ 0, ETHTOOL_LINK_MODE_56000baseLR4_Full_BIT,
>  		  "56000baseLR4/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_25000baseCR_Full_BIT,
> +		  "25000baseCR/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_25000baseKR_Full_BIT,
> +		  "25000baseKR/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_25000baseSR_Full_BIT,
> +		  "25000baseSR/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT,
> +		  "50000baseCR2/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT,
> +		  "50000baseKR2/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT,
> +		  "100000baseKR4/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT,
> +		  "100000baseSR4/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT,
> +		  "100000baseCR4/Full" },
> +		{ 0, ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
> +		  "100000baseLR4_ER4/Full" },
>  	};
>  	int indent;
>  	int did1, new_line_pend, i;

If my other comment wasn't clear, this is the patch I
to which I was suggesting that you could add support for
ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT.

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