linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Avri Altman <Avri.Altman@wdc.com>
To: Michael Wu <michael@allwinnertech.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@linaro.org>,
	"commit_signer:3/9=33%,authored:4/9=44%,added_lines:26/61=43% 
	,removed_lines:25/35=71%" <adrian.hunter@intel.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@micron.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@gmail.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>,
	Adrian Hunter <adrian.hunter@intel.com>,
	Luca Porzio <porzio@gmail.com>,
	lixiang <lixiang@allwinnertech.com>,
	Bean Huo <beanhuo@micron.com>, "open list:MULTIMEDIA CARD (MMC),
	SECURE DIGITAL (SD) AND..."  <linux-mmc@vger.kernel.org>,
	open list <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] mmc: block: enable cache-flushing when mmc cache is on
Date: Mon, 14 Mar 2022 07:26:26 +0000	[thread overview]
Message-ID: <DM6PR04MB6575D203B92955D9A913576CFC0F9@DM6PR04MB6575.namprd04.prod.outlook.com> (raw)
In-Reply-To: <20220312044315.7994-1-michael@allwinnertech.com>

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

  parent reply	other threads:[~2022-03-14  7:26 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-12  4:43 [PATCH] mmc: block: enable cache-flushing when mmc cache is on Michael Wu
2022-03-14  6:54 ` Adrian Hunter
2022-03-14  9:20   ` Michael Wu
2022-03-14  9:37     ` Avri Altman
2022-03-16  9:54       ` Michael Wu
2022-03-16 11:09         ` Avri Altman
2022-03-16 11:28           ` Adrian Hunter
2022-03-16 14:46             ` Christian Löhle
2022-03-16 16:05               ` Adrian Hunter
2022-03-17  9:14                 ` Ulf Hansson
2022-03-24 11:27                   ` Ulf Hansson
2022-03-25  5:45                     ` Michael Wu
2022-03-25 10:13                       ` Ulf Hansson
2022-03-28 10:11                         ` Michael Wu
2022-03-28 11:38                           ` Ulf Hansson
2022-03-29  9:08                             ` Michael Wu
2022-03-29  9:53                               ` Ulf Hansson
2022-03-27  8:09                 ` Avri Altman
2022-03-14 11:10     ` Avri Altman
2022-03-14  7:26 ` Avri Altman [this message]
2022-03-14 10:32   ` Adrian Hunter

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=DM6PR04MB6575D203B92955D9A913576CFC0F9@DM6PR04MB6575.namprd04.prod.outlook.com \
    --to=avri.altman@wdc.com \
    --cc=adrian.hunter@intel.com \
    --cc=beanhuo@micron.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=lixiang@allwinnertech.com \
    --cc=michael@allwinnertech.com \
    --cc=porzio@gmail.com \
    --cc=ulf.hansson@linaro.org \
    /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).