linux-kernel - RE: [PATCH] mmc: block: enable cache-flushing when mmc cache is on

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]

Date:   Mon, 14 Mar 2022 07:26:26 +0000
From:   Avri Altman <Avri.Altman@....com>
To:     Michael Wu <michael@...winnertech.com>,
        "maintainer:MULTIMEDIA CARD (MMC), SECURE DIGITAL (SD) AND... 
        ,commit_signer:11/9=100%,authored:4/9=44%,added_lines:26/61=43% 
        ,removed_lines:25/35=71%" <ulf.hansson@...aro.org>,
        "commit_signer:3/9=33%,authored:4/9=44%,added_lines:26/61=43% 
        ,removed_lines:25/35=71%" <adrian.hunter@...el.com>,
        "commit_signer:1/9=11%,authored:4/9=44%,authored:1/9=11% 
        ,added_lines:26/61=43%,removed_lines:25/35=71%" <beanhuo@...ron.com>,
        "commit_signer:1/9=11%,authored:4/9=44%,authored:1/9=11% 
        ,added_lines:26/61=43%,added_lines:4/61=7%,removed_lines:25/35=71% 
        ,removed_lines:3/35=9%" <porzio@...il.com>
CC:     Ulf Hansson <ulf.hansson@...aro.org>,
        Adrian Hunter <adrian.hunter@...el.com>,
        Luca Porzio <porzio@...il.com>,
        lixiang <lixiang@...winnertech.com>,
        Bean Huo <beanhuo@...ron.com>,
        "open list:MULTIMEDIA CARD (MMC), SECURE DIGITAL (SD) AND..." 
        <linux-mmc@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] mmc: block: enable cache-flushing when mmc cache is on

Hi,
> The mmc core enable cache on default. But it only enables cache-flushing
> when host supports cmd23 and eMMC supports reliable write.
> For hosts which do not support cmd23 or eMMCs which do not support
> reliable write, the cache can not be flushed by `sync` command.
> This may leads to cache data lost.
> This patch enables cache-flushing as long as cache is enabled, no matter host
> supports cmd23 and/or eMMC supports reliable write or not.
I looked in the spec and indeed couldn't find why enabling cache is dependent of cmd23/reliable write.
Nor I was able to find the original commit log.

Please allow few days to ask internally.

Thanks,
Avri