From: Vidya Sagar Ravipati <vi...@cumulusnetworks.com>

This patch enhances ethtool link mode bitmap to include
missing interface modes for 1G/10G speeds

Changes:
a) 1000baseX is the mode introduced to cover all 1G Fiber cases.
All modes under 1000BaseX i.e. 1000BASE-SX, 1000BASE-LX, 1000BASE-LX10
and 1000BASE-BX10 are not explicitly defined at this moment.
10G CR,SR,LR and ER link modes are included for 10G speed..
b) Update description regarding QSFP+/QSFP28 support

Issue:
ethtool on  1G/10G SFP port reports Base-T
as this port supports 1000baseX,10G CR, SR and LR modes.

root@tor-02$ ethtool swp1
Settings for swp1:
        Supported ports: [ FIBRE ]
        Supported link modes:   1000baseT/Full
                                10000baseT/Full
        Supported pause frame use: Symmetric Receive-only
        Supports auto-negotiation: Yes
        Advertised link modes:  1000baseT/Full
        Advertised pause frame use: No
        Advertised auto-negotiation: No
        Speed: 10000Mb/s
        Duplex: Full
        Port: FIBRE
        PHYAD: 0
        Transceiver: external
        Auto-negotiation: off
        Current message level: 0x00000000 (0)

        Link detected: yes

After fix:
root@tor-02$ ethtool swp1
Settings for swp1:
        Supported ports: [ FIBRE ]
        Supported link modes:   1000baseX/Full
                                10000baseCR/Full
                                10000baseSR/Full
                                10000baseLR/Full
                                10000baseER/Full
        Supported pause frame use: Symmetric Receive-only
        Supports auto-negotiation: Yes
        Advertised link modes:  1000baseX/Full
        Advertised pause frame use: No
        Advertised auto-negotiation: No
        Speed: 10000Mb/s
        Duplex: Full
        Port: FIBRE
        PHYAD: 0
        Transceiver: external
        Auto-negotiation: off
        Current message level: 0x00000000 (0)
        Link detected: yes

Signed-off-by: Vidya Sagar Ravipati <vi...@cumulusnetworks.com>
---
 ethtool.c | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/ethtool.c b/ethtool.c
index ab58c7f..6287b55 100644
--- a/ethtool.c
+++ b/ethtool.c
@@ -23,6 +23,7 @@
  * vmxnet3 support by Shrikrishna Khare <skh...@vmware.com>
  * Various features by Ben Hutchings <b...@decadent.org.uk>;
  *     Copyright 2008-2010, 2013-2016 Ben Hutchings
+ * QSFP+/QSFP28 DOM support by Vidya Sagar Ravipati <vi...@cumulusnetworks.com>
  *
  * TODO:
  *   * show settings for all devices
@@ -522,6 +523,12 @@ static void init_global_link_mode_masks(void)
                ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT,
                ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
                ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT,
+               ETHTOOL_LINK_MODE_1000baseX_Full_BIT,
+               ETHTOOL_LINK_MODE_10000baseCR_Full_BIT,
+               ETHTOOL_LINK_MODE_10000baseSR_Full_BIT,
+               ETHTOOL_LINK_MODE_10000baseLR_Full_BIT,
+               ETHTOOL_LINK_MODE_10000baseLRM_Full_BIT,
+               ETHTOOL_LINK_MODE_10000baseER_Full_BIT,
        };
        static const enum ethtool_link_mode_bit_indices
                additional_advertised_flags_bits[] = {
@@ -662,6 +669,18 @@ static void dump_link_caps(const char *prefix, const char 
*an_prefix,
                  "100000baseLR4_ER4/Full" },
                { 0, ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT,
                  "50000baseSR2/Full" },
+               { 0, ETHTOOL_LINK_MODE_1000baseX_Full_BIT,
+                 "1000baseX/Full" },
+               { 0, ETHTOOL_LINK_MODE_10000baseCR_Full_BIT,
+                 "10000baseCR/Full" },
+               { 0, ETHTOOL_LINK_MODE_10000baseSR_Full_BIT,
+                 "10000baseSR/Full" },
+               { 0, ETHTOOL_LINK_MODE_10000baseLR_Full_BIT,
+                 "10000baseLR/Full" },
+               { 0, ETHTOOL_LINK_MODE_10000baseLRM_Full_BIT,
+                 "10000baseLRM/Full" },
+               { 0, ETHTOOL_LINK_MODE_10000baseER_Full_BIT,
+                 "10000baseER/Full" },
        };
        int indent;
        int did1, new_line_pend, i;
-- 
2.1.4

Reply via email to