[v4,0/2] Do not flush cache when it is disabled
mbox series

Message ID 20210420134641.57343-1-avri.altman@wdc.com
Headers show
Series
  • Do not flush cache when it is disabled
Related show

Message

Avri Altman April 20, 2021, 1:46 p.m. UTC
v3 -> v4
 - Attend Adrian's comments
 - Add Acked-by tag

v2 -> v3:
 - rebase onto recent cache changes

v1 -> v2:
 - Attend Adrian's comments

Cache is a temporary storage space in an eMMC device. Volatile by
nature, the cache should in typical case reduce the access time compared
to an access to the main nonvolatile storage.

The cache function can be turned ON and OFF. Once OFF, the host is not
expected to issue a flush-cache command to the device.

Avri Altman (2):
  mmc: block: Issue flush only if allowed
  mmc: block: Update ext_csd.cache_ctrl if it was written

 drivers/mmc/core/block.c   | 21 +++++++++++++++++++++
 drivers/mmc/core/mmc.c     |  2 +-
 drivers/mmc/core/mmc_ops.h |  5 +++++
 3 files changed, 27 insertions(+), 1 deletion(-)

Comments

Avri Altman April 22, 2021, 9:41 a.m. UTC | #1
Ulf - Would you consider picking this sires up now?
I was hoping to close this bug.

Thanks,
Avri

> -----Original Message-----
> From: Avri Altman <avri.altman@wdc.com>
> Sent: Tuesday, 20 April 2021 16:47
> To: Ulf Hansson <ulf.hansson@linaro.org>; linux-mmc@vger.kernel.org; Adrian
> Hunter <adrian.hunter@intel.com>
> Cc: linux-kernel@vger.kernel.org; Avri Altman <Avri.Altman@wdc.com>
> Subject: [PATCH v4 0/2] Do not flush cache when it is disabled
> 
> v3 -> v4
>  - Attend Adrian's comments
>  - Add Acked-by tag
> 
> v2 -> v3:
>  - rebase onto recent cache changes
> 
> v1 -> v2:
>  - Attend Adrian's comments
> 
> Cache is a temporary storage space in an eMMC device. Volatile by
> nature, the cache should in typical case reduce the access time compared
> to an access to the main nonvolatile storage.
> 
> The cache function can be turned ON and OFF. Once OFF, the host is not
> expected to issue a flush-cache command to the device.
> 
> Avri Altman (2):
>   mmc: block: Issue flush only if allowed
>   mmc: block: Update ext_csd.cache_ctrl if it was written
> 
>  drivers/mmc/core/block.c   | 21 +++++++++++++++++++++
>  drivers/mmc/core/mmc.c     |  2 +-
>  drivers/mmc/core/mmc_ops.h |  5 +++++
>  3 files changed, 27 insertions(+), 1 deletion(-)
> 
> --
> 2.25.1