netdev - RE: [PATCH] e1000e: workaround missing power down mii control bit on 82571

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

Message-ID: <8DD2590731AB5D4C9DBF71A877482A9001773F76C4@orsmsx509.amr.corp.intel.com>
Date:	Thu, 16 Dec 2010 11:28:18 -0800
From:	"Allan, Bruce W" <bruce.w.allan@...el.com>
To:	"Allan, Bruce W" <bruce.w.allan@...el.com>,
	Ben Hutchings <bhutchings@...arflare.com>,
	Arthur Jones <ajones@...erbed.com>
CC:	"Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [PATCH] e1000e: workaround missing power down mii control bit
 on 82571

>-----Original Message-----
>From: netdev-owner@...r.kernel.org [mailto:netdev-owner@...r.kernel.org] On
>Behalf Of Allan, Bruce W
>Sent: Thursday, December 16, 2010 11:04 AM
>To: Ben Hutchings; Arthur Jones
>Cc: Kirsher, Jeffrey T; netdev@...r.kernel.org
>Subject: RE: [PATCH] e1000e: workaround missing power down mii control bit on
>82571
>
>>-----Original Message-----
>>From: netdev-owner@...r.kernel.org [mailto:netdev-owner@...r.kernel.org] On
>>Behalf Of Ben Hutchings
>>Sent: Thursday, December 16, 2010 10:57 AM
>>To: Arthur Jones
>>Cc: Kirsher, Jeffrey T; netdev@...r.kernel.org
>>Subject: Re: [PATCH] e1000e: workaround missing power down mii control bit on
>>82571
>>
>>Adding this special case into MDIO access seems like a really nasty
>>hack.  Surely the callers that set the control register should take care
>>of this.
>>
>>Ben.
>
>Agreed.  I am setting up to repro now to see if it is an actual hardware
>issue or just a software bug; either way, this patch is not the correct
>approach and I'll follow up shortly.
>
>Bruce.

It's the reset in e1000_set_settings() which ignores that we had previously
powered off the Phy.  I'll go through the rest of the code and fix up this
and any other occurrences of similar issues properly.

Thanks for reporting this issue Arthur.

Bruce.