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
| ||
|
Date: Fri, 21 Jan 2022 19:01:23 +0300 From: Andrey Skvortsov <andrej.skvortzov@...il.com> To: Ulf Hansson <ulf.hansson@...aro.org> Cc: Arnaud Ferraris <arnaud.ferraris@...il.com>, Linus Walleij <linus.walleij@...aro.org>, Avri Altman <avri.altman@....com>, Shawn Lin <shawn.lin@...k-chips.com>, Huijin Park <huijin.park@...sung.com>, Christian Löhle <CLoehle@...erstone.com>, Yue Hu <huyue2@...ong.com>, linux-mmc@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] mmc: core: Wait for command setting 'Power Off Notification' bit to complete On 22-01-21 15:32, Ulf Hansson wrote: > On Sat, 15 Jan 2022 at 13:15, Andrey Skvortsov > <andrej.skvortzov@...il.com> wrote: > > > > SD card is allowed to signal busy on DAT0 up to 1s after the > > CMD49. According to SD spec (version 6.0 section 5.8.1.3) first host > > waits until busy of CMD49 is released and only then polls Power > > Management Status register up to 1s until the card indicates ready to > > power off. > > > > Without waiting for busy before polling status register sometimes card > > becomes unresponsive and system fails to suspend: > > > > [ 205.907459] Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. > > [ 206.421274] sunxi-mmc 1c0f000.mmc: data error, sending stop command > > [ 206.421321] sunxi-mmc 1c0f000.mmc: send stop command failed > > [ 206.421347] mmc0: error -110 reading status reg of PM func > > [ 206.421366] PM: dpm_run_callback(): mmc_bus_suspend+0x0/0x74 returns -110 > > [ 206.421402] mmcblk mmc0:aaaa: PM: failed to suspend async: error -110 > > [ 206.437064] PM: Some devices failed to suspend, or early wake event detected > > > > Tested with Sandisk Extreme PRO A2 64GB on Allwinner A64 system. > > > > Signed-off-by: Andrey Skvortsov <andrej.skvortzov@...il.com> > > I did some tests at my side too, using a Qcom platform and it worked > fine. However, I also used the same type of card as you, as I didn't > have any other try with at this moment. > So, applied for fixes and by adding a fixes/stable tag, thanks! Thank you very much. There is one another test report, that this patch fixes suspend with KLEVV CRAS 128GB (K128GUSD6U3-CA). [1] 1. https://gitlab.com/mobian1/issues/-/issues/389#note_816636364 > I also did a minor amendment to the patch, see below. > > Kind regards > Uffe -- Best regards, Andrey Skvortsov
Powered by blists - more mailing lists