mbox series

[v5,0/2] Do not flush cache when it is disabled

Message ID 20210425060207.2591-1-avri.altman@wdc.com (mailing list archive)
Headers show
Series Do not flush cache when it is disabled | expand

Message

Avri Altman April 25, 2021, 6:02 a.m. UTC
v4 -> v5
 - patche "Update ext_csd.cache_ctrl if it was written" accepted
 - Add one more patch: "Add cache_enabled bus ops"
 - Add Acked-by tag
 - another rebase

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.

Avri Altman (2):
  mmc: Add cache_enabled bus ops
  mmc: block: Issue flush only if allowed

 drivers/mmc/core/block.c   | 9 +++++++++
 drivers/mmc/core/core.h    | 7 +++++++
 drivers/mmc/core/mmc.c     | 7 +++++++
 drivers/mmc/core/mmc_ops.c | 4 +---
 4 files changed, 24 insertions(+), 3 deletions(-)

Comments

Ulf Hansson April 26, 2021, 9:44 a.m. UTC | #1
On Sun, 25 Apr 2021 at 08:02, Avri Altman <avri.altman@wdc.com> wrote:
>
> v4 -> v5
>  - patche "Update ext_csd.cache_ctrl if it was written" accepted
>  - Add one more patch: "Add cache_enabled bus ops"
>  - Add Acked-by tag
>  - another rebase
>
> 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.
>
> Avri Altman (2):
>   mmc: Add cache_enabled bus ops
>   mmc: block: Issue flush only if allowed
>
>  drivers/mmc/core/block.c   | 9 +++++++++
>  drivers/mmc/core/core.h    | 7 +++++++
>  drivers/mmc/core/mmc.c     | 7 +++++++
>  drivers/mmc/core/mmc_ops.c | 4 +---
>  4 files changed, 24 insertions(+), 3 deletions(-)
>

Normally, I would appreciate this split of the patches, but since
these are material for stable kernels I decided to squash the patches.

I also took the opportunity of updating the commit message a bit and
did a minor cleanup in the code. Please have a look at my next branch
and yell at me if there is something you don't like.

So, applied for next and by adding a stable tag, thanks!

Kind regards
Uffe
Avri Altman April 26, 2021, 12:06 p.m. UTC | #2
> Normally, I would appreciate this split of the patches, but since
> these are material for stable kernels I decided to squash the patches.
> 
> I also took the opportunity of updating the commit message a bit and
> did a minor cleanup in the code. Please have a look at my next branch
> and yell at me if there is something you don't like.
Looks fine. Thanks a lot,
Avri

> 
> So, applied for next and by adding a stable tag, thanks!
> 
> Kind regards
> Uffe