netdev - Re: phy: SMSC LAN8710/LAN8720 high load

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

Date:	Fri, 15 Jan 2016 10:18:50 +0100
From:	Teresa Remmet <t.remmet@...tec.de>
To:	David Laight <David.Laight@...LAB.COM>
Cc:	Igor Plyatov <plyatov@...il.com>,
	"David S. Miller" <davem@...emloft.net>,
	Florian Fainelli <f.fainelli@...il.com>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: phy: SMSC LAN8710/LAN8720 high load

Hello David,

Am Dienstag, den 12.01.2016, 11:33 +0000 schrieb David Laight:
> From: Teresa Remmet
> > Sent: 11 January 2016 07:42
> > we have noticed load issues on our AM335x boards with a LAN8710 phy while
> > cpu is idling using 4.4-rc8.
> > 
> > CPU:   0% usr   0% sys   0% nic  99% idle   0% io   0% irq   0% sirq
> > Load average: 1.21 1.16 1.00 1/84 372
> 
> The 'Load average' value is mostly meaningless.
> It includes any process/thread that is sleeping uninterruptably - even
> if the process is never woken (or is only woken for short periods).
> 
> If you have a kernel thread that you don't want affected by signals
> (exits on some request to the driver) then you end up increasing the
> 'load average'.

thank you for your explanation. But I'm still worried that waiting so
long in this function might be a problem.

When I look at the function duration of lan87xx_read_status it takes in
average about 1277869 us. When disabling the the power detect mode the
time drops down to 555.326 us.

As disabling over device tree does not work correct for lan87xx a patch
follows.

Teresa

> 
> 	David
>