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

2016-08-21 Thread Michael Chan 
On Sun, Aug 21, 2016 at 7:44 AM, Yuval Mintz  wrote:
>> Ping?
>>
>> On Tue, Aug 02, 2016 at 03:49:57PM -0400, John W. Linville wrote:
>> > Are these revised patches still on their way?
>> >
>> > John
>> >
>> > On Sun, Jul 24, 2016 at 01:57:14AM -0700, Vidya Sagar Ravipati wrote:
>> > > Yuval,
>> > > I will try to resubmit the patches this week with updated
>> > > comments
>> > >
>> > > Thanks
>> > > Vidya Sagar
>
> I can pick this up; But I'd need someone to fill me on which of the newer
> speeds could support autoneg [if any].

All of the new speeds support autoneg as far as I know.

RE: [ethtool PATCH v2 4/4] ethtool: Enhancing link mode bits to support 25G/50G/100G

2016-08-21 Thread Yuval Mintz 
> Ping?
> 
> On Tue, Aug 02, 2016 at 03:49:57PM -0400, John W. Linville wrote:
> > Are these revised patches still on their way?
> >
> > John
> >
> > On Sun, Jul 24, 2016 at 01:57:14AM -0700, Vidya Sagar Ravipati wrote:
> > > Yuval,
> > > I will try to resubmit the patches this week with updated
> > > comments
> > >
> > > Thanks
> > > Vidya Sagar

I can pick this up; But I'd need someone to fill me on which of the newer
speeds could support autoneg [if any].

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

2016-08-18 Thread John W. Linville 
Ping?

On Tue, Aug 02, 2016 at 03:49:57PM -0400, John W. Linville wrote:
> Are these revised patches still on their way?
> 
> John
> 
> On Sun, Jul 24, 2016 at 01:57:14AM -0700, Vidya Sagar Ravipati wrote:
> > Yuval,
> > I will try to resubmit the patches this week with updated comments
> > 
> > Thanks
> > Vidya Sagar
> > 
> > On Sat, Jul 23, 2016 at 10:57 PM, Yuval Mintz 
> > wrote:
> > 
> > > > Enhancing link mode bits to support 25G/50G/100G for supported and
> > > > advertised speed mode bits
> > > >
> > > > Signed-off-by: Vidya Sagar Ravipati 
> > > > ---
> > > >  ethtool.c | 27 +++
> > > >  1 file changed, 27 insertions(+)
> > >
> > > Hi Vidya,
> > >
> > > Are you re-trying your series one anytime soon?
> > >
> > > If not, can we simply push this [and ethtool-copy.h], as those are needed
> > > for querying/setting the recently added new speeds.
> > >
> > >
> 
> -- 
> John W. Linville  Someday the world will need a hero, and you
> linvi...@tuxdriver.commight be all we have.  Be ready.
> 

-- 
John W. LinvilleSomeday the world will need a hero, and you
linvi...@tuxdriver.com  might be all we have.  Be ready.

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

2016-08-02 Thread John W. Linville 
Are these revised patches still on their way?

John

On Sun, Jul 24, 2016 at 01:57:14AM -0700, Vidya Sagar Ravipati wrote:
> Yuval,
> I will try to resubmit the patches this week with updated comments
> 
> Thanks
> Vidya Sagar
> 
> On Sat, Jul 23, 2016 at 10:57 PM, Yuval Mintz 
> wrote:
> 
> > > Enhancing link mode bits to support 25G/50G/100G for supported and
> > > advertised speed mode bits
> > >
> > > Signed-off-by: Vidya Sagar Ravipati 
> > > ---
> > >  ethtool.c | 27 +++
> > >  1 file changed, 27 insertions(+)
> >
> > Hi Vidya,
> >
> > Are you re-trying your series one anytime soon?
> >
> > If not, can we simply push this [and ethtool-copy.h], as those are needed
> > for querying/setting the recently added new speeds.
> >
> >

-- 
John W. LinvilleSomeday the world will need a hero, and you
linvi...@tuxdriver.com  might be all we have.  Be ready.

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

2016-07-24 Thread Vidya Sagar Ravipati 
Yuval,
I will try to resubmit the patches this week with updated comments

Thanks
Vidya Sagar

On Sat, Jul 23, 2016 at 10:57 PM, Yuval Mintz  wrote:
>> Enhancing link mode bits to support 25G/50G/100G for supported and
>> advertised speed mode bits
>>
>> Signed-off-by: Vidya Sagar Ravipati 
>> ---
>>  ethtool.c | 27 +++
>>  1 file changed, 27 insertions(+)
>
> Hi Vidya,
>
> Are you re-trying your series one anytime soon?
>
> If not, can we simply push this [and ethtool-copy.h], as those are needed
> for querying/setting the recently added new speeds.
>

RE: [ethtool PATCH v2 4/4] ethtool: Enhancing link mode bits to support 25G/50G/100G

2016-07-24 Thread Yuval Mintz 
> Enhancing link mode bits to support 25G/50G/100G for supported and
> advertised speed mode bits
> 
> Signed-off-by: Vidya Sagar Ravipati 
> ---
>  ethtool.c | 27 +++
>  1 file changed, 27 insertions(+)

Hi Vidya,

Are you re-trying your series one anytime soon?

If not, can we simply push this [and ethtool-copy.h], as those are needed
for querying/setting the recently added new speeds.

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

2016-06-27 Thread David Decotigny 
On Sun, Jun 26, 2016 at 12:45 PM, Vidya Sagar Ravipati
 wrote:
> From: Vidya Sagar Ravipati 
>
> Enhancing link mode bits to support 25G/50G/100G
> for supported and advertised speed mode bits
>
> Signed-off-by: Vidya Sagar Ravipati 
> ---
>  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_5baseCR2_Full_BIT,
> +   ETHTOOL_LINK_MODE_5baseKR2_Full_BIT,
> +   ETHTOOL_LINK_MODE_10baseKR4_Full_BIT,
> +   ETHTOOL_LINK_MODE_10baseSR4_Full_BIT,
> +   ETHTOOL_LINK_MODE_10baseCR4_Full_BIT,
> +   ETHTOOL_LINK_MODE_10baseLR4_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_5baseCR2_Full_BIT,
> + "5baseCR2/Full" },
> +   { 0, ETHTOOL_LINK_MODE_5baseKR2_Full_BIT,
> + "5baseKR2/Full" },
> +   { 0, ETHTOOL_LINK_MODE_10baseKR4_Full_BIT,
> + "10baseKR4/Full" },
> +   { 0, ETHTOOL_LINK_MODE_10baseSR4_Full_BIT,
> + "10baseSR4/Full" },
> +   { 0, ETHTOOL_LINK_MODE_10baseCR4_Full_BIT,
> + "10baseCR4/Full" },
> +   { 0, ETHTOOL_LINK_MODE_10baseLR4_ER4_Full_BIT,
> + "10baseLR4_ER4/Full" },
> };
> int indent;
> int did1, new_line_pend, i;
> --
> 2.1.4
>

Acked-By: David Decotigny

[ethtool PATCH v2 4/4] ethtool: Enhancing link mode bits to support 25G/50G/100G

2016-06-26 Thread Vidya Sagar Ravipati 
From: Vidya Sagar Ravipati 

Enhancing link mode bits to support 25G/50G/100G
for supported and advertised speed mode bits

Signed-off-by: Vidya Sagar Ravipati 
---
 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_5baseCR2_Full_BIT,
+   ETHTOOL_LINK_MODE_5baseKR2_Full_BIT,
+   ETHTOOL_LINK_MODE_10baseKR4_Full_BIT,
+   ETHTOOL_LINK_MODE_10baseSR4_Full_BIT,
+   ETHTOOL_LINK_MODE_10baseCR4_Full_BIT,
+   ETHTOOL_LINK_MODE_10baseLR4_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_5baseCR2_Full_BIT,
+ "5baseCR2/Full" },
+   { 0, ETHTOOL_LINK_MODE_5baseKR2_Full_BIT,
+ "5baseKR2/Full" },
+   { 0, ETHTOOL_LINK_MODE_10baseKR4_Full_BIT,
+ "10baseKR4/Full" },
+   { 0, ETHTOOL_LINK_MODE_10baseSR4_Full_BIT,
+ "10baseSR4/Full" },
+   { 0, ETHTOOL_LINK_MODE_10baseCR4_Full_BIT,
+ "10baseCR4/Full" },
+   { 0, ETHTOOL_LINK_MODE_10baseLR4_ER4_Full_BIT,
+ "10baseLR4_ER4/Full" },
};
int indent;
int did1, new_line_pend, i;
-- 
2.1.4