Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210425060207.2591-1-avri.altman@wdc.com>
In-Reply-To: <20210425060207.2591-1-avri.altman@wdc.com>
From:   Ulf Hansson <ulf.hansson@linaro.org>
Date:   Mon, 26 Apr 2021 11:44:03 +0200
Message-ID: <CAPDyKFpZhk-kDpuA1m-E4RXHMXbG72S=OW07XrLzP0-9Q_Gu9g@mail.gmail.com>
Subject: Re: [PATCH v5 0/2] Do not flush cache when it is disabled
To:     Avri Altman <avri.altman@wdc.com>
Cc:     linux-mmc <linux-mmc@vger.kernel.org>,
        Adrian Hunter <adrian.hunter@intel.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

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