[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5712c69ae37447c5b576d87b247f5756@hyperstone.com>
Date: Thu, 2 Mar 2023 15:07:56 +0000
From: Christian Löhle <CLoehle@...erstone.com>
To: Ulf Hansson <ulf.hansson@...aro.org>,
"linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
Jens Axboe <axboe@...nel.dk>
CC: Wenchao Chen <wenchao.chen666@...il.com>,
Adrian Hunter <adrian.hunter@...el.com>,
Avri Altman <avri.altman@....com>,
"linux-block@...r.kernel.org" <linux-block@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [RFC PATCH] mmc: core: Disable REQ_FUA if the eMMC supports an
internal cache
> Subject: [RFC PATCH] mmc: core: Disable REQ_FUA if the eMMC supports an internal cache
Hey Uffe,
I may have the setup to play with this.
(Powerfailing eMMC devices)
>
> REQ_FUA is in general supported for eMMC cards, which translates into so called "reliable writes". To support these write operations, the CMD23 (MMC_CAP_CMD23), needs to be supported by the mmc host too, which is common but not always the case.
>
> For some eMMC devices, it has been reported that reliable writes are quite costly, leading to performance degradations.
>
> In a way to improve the situation, let's avoid announcing REQ_FUA support if the eMMC supports an internal cache, as that allows us to rely solely on flush-requests (REQ_OP_FLUSH) instead, which seems to be a lot cheaper.
> Note that, those mmc hosts that lacks CMD23 support are already using this type of configuration, whatever that could mean.
Just note that reliable write is strictly weaker than turning cache off/flushing,
if card loses power during cache off/flush programming / busy, sector-wise atomicity is not mandated by the spec.
(And that is assuming cache off/flush is actually respected by the card as intended by the spec, should some cards be checked?)
Maybe some FS people can also chime in?
Regards,
Christian
Hyperstone GmbH | Reichenaustr. 39a | 78467 Konstanz
Managing Director: Dr. Jan Peter Berns.
Commercial register of local courts: Freiburg HRB381782
Powered by blists - more mailing lists