diff for duplicates of <20180907062851.GA7336@super_plancton> diff --git a/a/1.txt b/N1/1.txt index 00c3af4..8a593b6 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,7 +1,7 @@ -Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit : +Le mercredi 08 août 2018 à 18:00:28 (+0300), Neftin, Sasha a écrit : > On 8/8/2018 17:24, Neftin, Sasha wrote: > > On 8/7/2018 09:42, Camille Bordignon wrote: -> > > Le lundi 06 ao?t 2018 ? 15:45:29 (-0700), Alexander Duyck a ?crit : +> > > Le lundi 06 août 2018 à 15:45:29 (-0700), Alexander Duyck a écrit : > > > > On Mon, Aug 6, 2018 at 4:59 AM, Camille Bordignon > > > > <camille.bordignon@easymile.com> wrote: > > > > > Hello, @@ -11,7 +11,7 @@ Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit : > > > > > It seems that after a wire reconnection, auto-negotation "fails" and > > > > > link speed drips to 10 Mbps. > > > > > -> > > > > ?From kernel logs: +> > > > > From kernel logs: > > > > > [17616.346150] e1000e: enp0s31f6 NIC Link is Down > > > > > [17627.003322] e1000e: enp0s31f6 NIC Link is Up 10 Mbps Full > > > > > Duplex, Flow Control: None @@ -21,31 +21,31 @@ Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit : > > > > > > > > > > $ethtool enp0s31f6 > > > > > Settings for enp0s31f6: -> > > > > ???????? Supported ports: [ TP ] -> > > > > ???????? Supported link modes:?? 10baseT/Half 10baseT/Full -> > > > > ???????????????????????????????? 100baseT/Half 100baseT/Full -> > > > > ???????????????????????????????? 1000baseT/Full -> > > > > ???????? Supported pause frame use: No -> > > > > ???????? Supports auto-negotiation: Yes -> > > > > ???????? Supported FEC modes: Not reported -> > > > > ???????? Advertised link modes:? 10baseT/Half 10baseT/Full -> > > > > ???????????????????????????????? 100baseT/Half 100baseT/Full -> > > > > ???????????????????????????????? 1000baseT/Full -> > > > > ???????? Advertised pause frame use: No -> > > > > ???????? Advertised auto-negotiation: Yes -> > > > > ???????? Advertised FEC modes: Not reported -> > > > > ???????? Speed: 10Mb/s -> > > > > ???????? Duplex: Full -> > > > > ???????? Port: Twisted Pair -> > > > > ???????? PHYAD: 1 -> > > > > ???????? Transceiver: internal -> > > > > ???????? Auto-negotiation: on -> > > > > ???????? MDI-X: on (auto) -> > > > > ???????? Supports Wake-on: pumbg -> > > > > ???????? Wake-on: g -> > > > > ???????? Current message level: 0x00000007 (7) -> > > > > ??????????????????????????????? drv probe link -> > > > > ???????? Link detected: yes +> > > > > Supported ports: [ TP ] +> > > > > Supported link modes: 10baseT/Half 10baseT/Full +> > > > > 100baseT/Half 100baseT/Full +> > > > > 1000baseT/Full +> > > > > Supported pause frame use: No +> > > > > Supports auto-negotiation: Yes +> > > > > Supported FEC modes: Not reported +> > > > > Advertised link modes: 10baseT/Half 10baseT/Full +> > > > > 100baseT/Half 100baseT/Full +> > > > > 1000baseT/Full +> > > > > Advertised pause frame use: No +> > > > > Advertised auto-negotiation: Yes +> > > > > Advertised FEC modes: Not reported +> > > > > Speed: 10Mb/s +> > > > > Duplex: Full +> > > > > Port: Twisted Pair +> > > > > PHYAD: 1 +> > > > > Transceiver: internal +> > > > > Auto-negotiation: on +> > > > > MDI-X: on (auto) +> > > > > Supports Wake-on: pumbg +> > > > > Wake-on: g +> > > > > Current message level: 0x00000007 (7) +> > > > > drv probe link +> > > > > Link detected: yes > > > > > > > > > > > > > > > Notice that if disconnection last less than about 5 seconds, @@ -67,14 +67,14 @@ Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit : > > > > > +++ b/drivers/net/ethernet/intel/e1000e/netdev.c > > > > > @@ -5069,7 +5069,7 @@ static bool e1000e_has_link(struct > > > > > e1000_adapter *adapter) -> > > > > ???????? case e1000_media_type_copper: -> > > > > ???????????????? if (hw->mac.get_link_status) { -> > > > > ???????????????????????? ret_val = hw->mac.ops.check_for_link(hw); -> > > > > -?????????????????????? link_active = !hw->mac.get_link_status; -> > > > > +?????????????????????? link_active = false; -> > > > > ???????????????? } else { -> > > > > ???????????????????????? link_active = true; -> > > > > ???????????????? } +> > > > > case e1000_media_type_copper: +> > > > > if (hw->mac.get_link_status) { +> > > > > ret_val = hw->mac.ops.check_for_link(hw); +> > > > > - link_active = !hw->mac.get_link_status; +> > > > > + link_active = false; +> > > > > } else { +> > > > > link_active = true; +> > > > > } > > > > > > > > > > Maybe this is related to watchdog task. > > > > > @@ -109,7 +109,7 @@ Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit : > > > Camille Bordignon > > > _______________________________________________ > > > Intel-wired-lan mailing list -> > > Intel-wired-lan at osuosl.org +> > > Intel-wired-lan@osuosl.org > > > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan > > > > > I've agree with Alex. Let's try add 2s delay after a link event. Please, @@ -118,7 +118,7 @@ Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit : > > see if you see same problem. > > _______________________________________________ > > Intel-wired-lan mailing list -> > Intel-wired-lan at osuosl.org +> > Intel-wired-lan@osuosl.org > > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan > Camille, > My apologies, I wrong understand Alex. Please, do not try add delay. Please, diff --git a/a/content_digest b/N1/content_digest index c94e440..deeeff5 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,15 +4,21 @@ "ref\0001556a4-c49c-b96b-0be8-b3c4be7bb09c@intel.com\0" "ref\0c1e2cf13-6bcc-498d-03ea-7c3792d675cb@intel.com\0" "From\0Camille Bordignon <camille.bordignon@easymile.com>\0" - "Subject\0[Intel-wired-lan] e1000e driver stuck at 10Mbps after reconnection\0" + "Subject\0Re: [Intel-wired-lan] e1000e driver stuck at 10Mbps after reconnection\0" "Date\0Fri, 7 Sep 2018 08:28:51 +0200\0" - "To\0intel-wired-lan@osuosl.org\0" + "To\0Neftin" + " Sasha <sasha.neftin@intel.com>\0" + "Cc\0Alexander Duyck <alexander.duyck@gmail.com>" + Netdev <netdev@vger.kernel.org> + intel-wired-lan <intel-wired-lan@lists.osuosl.org> + David S. Miller <davem@davemloft.net> + " linux-kernel@vger.kernel.org\0" "\00:1\0" "b\0" - "Le mercredi 08 ao?t 2018 ? 18:00:28 (+0300), Neftin, Sasha a ?crit :\n" + "Le mercredi 08 ao\303\273t 2018 \303\240 18:00:28 (+0300), Neftin, Sasha a \303\251crit :\n" "> On 8/8/2018 17:24, Neftin, Sasha wrote:\n" "> > On 8/7/2018 09:42, Camille Bordignon wrote:\n" - "> > > Le lundi 06 ao?t 2018 ? 15:45:29 (-0700), Alexander Duyck a ?crit :\n" + "> > > Le lundi 06 ao\303\273t 2018 \303\240 15:45:29 (-0700), Alexander Duyck a \303\251crit :\n" "> > > > On Mon, Aug 6, 2018 at 4:59 AM, Camille Bordignon\n" "> > > > <camille.bordignon@easymile.com> wrote:\n" "> > > > > Hello,\n" @@ -22,7 +28,7 @@ "> > > > > It seems that after a wire reconnection, auto-negotation \"fails\" and\n" "> > > > > link speed drips to 10 Mbps.\n" "> > > > > \n" - "> > > > > ?From kernel logs:\n" + "> > > > > \302\240From kernel logs:\n" "> > > > > [17616.346150] e1000e: enp0s31f6 NIC Link is Down\n" "> > > > > [17627.003322] e1000e: enp0s31f6 NIC Link is Up 10 Mbps Full\n" "> > > > > Duplex, Flow Control: None\n" @@ -32,31 +38,31 @@ "> > > > > \n" "> > > > > $ethtool enp0s31f6\n" "> > > > > Settings for enp0s31f6:\n" - "> > > > > ???????? Supported ports: [ TP ]\n" - "> > > > > ???????? Supported link modes:?? 10baseT/Half 10baseT/Full\n" - "> > > > > ???????????????????????????????? 100baseT/Half 100baseT/Full\n" - "> > > > > ???????????????????????????????? 1000baseT/Full\n" - "> > > > > ???????? Supported pause frame use: No\n" - "> > > > > ???????? Supports auto-negotiation: Yes\n" - "> > > > > ???????? Supported FEC modes: Not reported\n" - "> > > > > ???????? Advertised link modes:? 10baseT/Half 10baseT/Full\n" - "> > > > > ???????????????????????????????? 100baseT/Half 100baseT/Full\n" - "> > > > > ???????????????????????????????? 1000baseT/Full\n" - "> > > > > ???????? Advertised pause frame use: No\n" - "> > > > > ???????? Advertised auto-negotiation: Yes\n" - "> > > > > ???????? Advertised FEC modes: Not reported\n" - "> > > > > ???????? Speed: 10Mb/s\n" - "> > > > > ???????? Duplex: Full\n" - "> > > > > ???????? Port: Twisted Pair\n" - "> > > > > ???????? PHYAD: 1\n" - "> > > > > ???????? Transceiver: internal\n" - "> > > > > ???????? Auto-negotiation: on\n" - "> > > > > ???????? MDI-X: on (auto)\n" - "> > > > > ???????? Supports Wake-on: pumbg\n" - "> > > > > ???????? Wake-on: g\n" - "> > > > > ???????? Current message level: 0x00000007 (7)\n" - "> > > > > ??????????????????????????????? drv probe link\n" - "> > > > > ???????? Link detected: yes\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Supported ports: [ TP ]\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Supported link modes:\302\240\302\240 10baseT/Half 10baseT/Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 100baseT/Half 100baseT/Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 1000baseT/Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Supported pause frame use: No\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Supports auto-negotiation: Yes\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Supported FEC modes: Not reported\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Advertised link modes:\302\240 10baseT/Half 10baseT/Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 100baseT/Half 100baseT/Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 1000baseT/Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Advertised pause frame use: No\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Advertised auto-negotiation: Yes\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Advertised FEC modes: Not reported\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Speed: 10Mb/s\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Duplex: Full\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Port: Twisted Pair\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 PHYAD: 1\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Transceiver: internal\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Auto-negotiation: on\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 MDI-X: on (auto)\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Supports Wake-on: pumbg\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Wake-on: g\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Current message level: 0x00000007 (7)\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 drv probe link\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 Link detected: yes\n" "> > > > > \n" "> > > > > \n" "> > > > > Notice that if disconnection last less than about 5 seconds,\n" @@ -78,14 +84,14 @@ "> > > > > +++ b/drivers/net/ethernet/intel/e1000e/netdev.c\n" "> > > > > @@ -5069,7 +5069,7 @@ static bool e1000e_has_link(struct\n" "> > > > > e1000_adapter *adapter)\n" - "> > > > > ???????? case e1000_media_type_copper:\n" - "> > > > > ???????????????? if (hw->mac.get_link_status) {\n" - "> > > > > ???????????????????????? ret_val = hw->mac.ops.check_for_link(hw);\n" - "> > > > > -?????????????????????? link_active = !hw->mac.get_link_status;\n" - "> > > > > +?????????????????????? link_active = false;\n" - "> > > > > ???????????????? } else {\n" - "> > > > > ???????????????????????? link_active = true;\n" - "> > > > > ???????????????? }\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 case e1000_media_type_copper:\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 if (hw->mac.get_link_status) {\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 ret_val = hw->mac.ops.check_for_link(hw);\n" + "> > > > > -\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 link_active = !hw->mac.get_link_status;\n" + "> > > > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 link_active = false;\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 } else {\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 link_active = true;\n" + "> > > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 }\n" "> > > > > \n" "> > > > > Maybe this is related to watchdog task.\n" "> > > > > \n" @@ -120,7 +126,7 @@ "> > > Camille Bordignon\n" "> > > _______________________________________________\n" "> > > Intel-wired-lan mailing list\n" - "> > > Intel-wired-lan at osuosl.org\n" + "> > > Intel-wired-lan@osuosl.org\n" "> > > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan\n" "> > > \n" "> > I've agree with Alex. Let's try add 2s delay after a link event. Please,\n" @@ -129,7 +135,7 @@ "> > see if you see same problem.\n" "> > _______________________________________________\n" "> > Intel-wired-lan mailing list\n" - "> > Intel-wired-lan at osuosl.org\n" + "> > Intel-wired-lan@osuosl.org\n" "> > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan\n" "> Camille,\n" "> My apologies, I wrong understand Alex. Please, do not try add delay. Please,\n" @@ -155,4 +161,4 @@ "\n" Camille Bordignon -9a487e02c97eaa3171b76231f9de604158a5ad4296d877144cd097cd39603e74 +841336e5676a169f206b30d1a8cd6f3ddfc67bd374ff73c48b23420c67127017
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.