* [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net
@ 2016-08-21 23:44 Vidya Sagar Ravipati
2016-08-22 19:26 ` John W. Linville
0 siblings, 1 reply; 3+ messages in thread
From: Vidya Sagar Ravipati @ 2016-08-21 23:44 UTC (permalink / raw)
To: linville, netdev, davem, decot
Cc: bkenward, daniel, galp, roopa, gospo, dustin, ben
From: Vidya Sagar Ravipati <vidya@cumulusnetworks.com>
This covers kernel changes upto:
commit 3851112e4737cd52aaeda0ce8d084be9ee128106
Author: Vidya Sagar Ravipati <vidya@cumulusnetworks.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
Signed-off-by: Vidya Sagar Ravipati <vidya@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net
2016-08-21 23:44 [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net Vidya Sagar Ravipati
@ 2016-08-22 19:26 ` John W. Linville
2016-08-22 19:59 ` Vidya Sagar Ravipati
0 siblings, 1 reply; 3+ messages in thread
From: John W. Linville @ 2016-08-22 19:26 UTC (permalink / raw)
To: Vidya Sagar Ravipati
Cc: netdev, davem, decot, bkenward, daniel, galp, roopa, gospo, dustin, ben
On Sun, Aug 21, 2016 at 04:44:01PM -0700, Vidya Sagar Ravipati wrote:
> From: Vidya Sagar Ravipati <vidya@cumulusnetworks.com>
>
> This covers kernel changes upto:
>
> commit 3851112e4737cd52aaeda0ce8d084be9ee128106
> Author: Vidya Sagar Ravipati <vidya@cumulusnetworks.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@mellanox.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@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Cc: Ben Hutchings <bwh@kernel.org>
Cc: David Decotigny <decot@googlers.com>
Acked-By: David Decotigny <decot@googlers.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Might as well include that here and in the patch that uses the
new bits?
> Signed-off-by: Vidya Sagar Ravipati <vidya@cumulusnetworks.com>
> Signed-off-by: David S. Miller <davem@davemloft.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@tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net
2016-08-22 19:26 ` John W. Linville
@ 2016-08-22 19:59 ` Vidya Sagar Ravipati
0 siblings, 0 replies; 3+ messages in thread
From: Vidya Sagar Ravipati @ 2016-08-22 19:59 UTC (permalink / raw)
To: John W. Linville
Cc: netdev, David Miller, David Decotigny, bkenward, daniel,
Gal Pressman, Roopa Prabhu, Andy Gospodarek, Dustin Byford,
Ben Hutchings
John,
I will re-submit v4 patchset with these updated bits.
Thanks
Vidya
On Mon, Aug 22, 2016 at 12:26 PM, John W. Linville
<linville@tuxdriver.com> wrote:
> On Sun, Aug 21, 2016 at 04:44:01PM -0700, Vidya Sagar Ravipati wrote:
>> From: Vidya Sagar Ravipati <vidya@cumulusnetworks.com>
>>
>> This covers kernel changes upto:
>>
>> commit 3851112e4737cd52aaeda0ce8d084be9ee128106
>> Author: Vidya Sagar Ravipati <vidya@cumulusnetworks.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@mellanox.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@mellanox.com>
> Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
> Cc: Ben Hutchings <bwh@kernel.org>
> Cc: David Decotigny <decot@googlers.com>
> Acked-By: David Decotigny <decot@googlers.com>
> Signed-off-by: David S. Miller <davem@davemloft.net>
>
> Might as well include that here and in the patch that uses the
> new bits?
>
>> Signed-off-by: Vidya Sagar Ravipati <vidya@cumulusnetworks.com>
>> Signed-off-by: David S. Miller <davem@davemloft.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@tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-22 19:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-21 23:44 [ethtool PATCH v3 1/4] ethtool-copy.h:sync with net Vidya Sagar Ravipati
2016-08-22 19:26 ` John W. Linville
2016-08-22 19:59 ` Vidya Sagar Ravipati
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.