From: Geert Uytterhoeven <geert+renesas@glider.be>
To: "David S . Miller" <davem@davemloft.net>,
Steve Glendinning <steve.glendinning@shawell.net>,
Andrew Lunn <andrew@lunn.ch>,
Florian Fainelli <f.fainelli@gmail.com>
Cc: Lukas Wunner <lukas@wunner.de>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
netdev@vger.kernel.org, linux-pm@vger.kernel.org,
linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
Geert Uytterhoeven <geert+renesas@glider.be>
Subject: [PATCH 0/2] net: Fix crashes due to activity during suspend
Date: Tue, 22 Aug 2017 20:37:24 +0200 [thread overview]
Message-ID: <1503427046-17618-1-git-send-email-geert+renesas@glider.be> (raw)
Hi all,
If an Ethernet device is used while the device is suspended, the system may
crash.
E.g. on sh73a0/kzm9g and r8a73a4/ape6evm, the external Ethernet chip is
driven by a PM controlled clock. If the Ethernet registers are accessed
while the clock is not running, the system will crash with an imprecise
external abort.
This patch series fixes two of such crashes:
1. The first patch prevents the PHY polling state machine from accessing
PHY registers while a device is suspended,
2. The second patch prevents the net core from trying to transmit packets
when an smsc911x device is suspended.
Both crashes can be reproduced on sh73a0/kzm9g and r8a73a4/ape6evm during
s2ram (rarely), or by using pm_test (more likely to trigger):
# echo 0 > /sys/module/printk/parameters/console_suspend
# echo platform > /sys/power/pm_test
# echo mem > /sys/power/state
With this series applied, my test systems survive a loop of 100 test
suspends.
Thanks for your comments!
Geert Uytterhoeven (2):
net: phy: Freeze PHY polling before suspending devices
net: smsc911x: Quiten netif during suspend
drivers/net/ethernet/smsc/smsc911x.c | 15 ++++++++++++++-
drivers/net/phy/phy.c | 12 +++++++-----
2 files changed, 21 insertions(+), 6 deletions(-)
--
2.7.4
Gr{oetje,eeting}s,
Geert
next reply other threads:[~2017-08-22 18:37 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-22 18:37 Geert Uytterhoeven [this message]
2017-08-22 18:37 ` [PATCH 1/2] net: phy: Freeze PHY polling before suspending devices Geert Uytterhoeven
2017-08-22 18:37 ` [PATCH 2/2] net: smsc911x: Quiten netif during suspend Geert Uytterhoeven
2017-08-22 20:17 ` Andrew Lunn
2017-08-22 18:49 ` [PATCH 0/2] net: Fix crashes due to activity " Florian Fainelli
2017-08-22 21:16 ` Geert Uytterhoeven
2017-08-23 11:45 ` Geert Uytterhoeven
2017-08-23 17:13 ` Florian Fainelli
2017-09-07 13:09 ` Florian Fainelli
2017-09-13 17:33 ` Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1503427046-17618-1-git-send-email-geert+renesas@glider.be \
--to=geert+renesas@glider.be \
--cc=andrew@lunn.ch \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=lukas@wunner.de \
--cc=netdev@vger.kernel.org \
--cc=rafael.j.wysocki@intel.com \
--cc=steve.glendinning@shawell.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).