Message-ID: <CAPDyKFpY+M_FVXCyeg+97jAgDSqhGDTNoND8CQDMWH-e09KGKQ@mail.gmail.com>
Date:   Tue, 8 Dec 2020 10:49:26 +0100
From:   Ulf Hansson <ulf.hansson@...aro.org>
To:     "Theodore Y. Ts'o" <tytso@....edu>,
        Michael Walle <michael@...le.cc>
Cc:     linux-ext4@...r.kernel.org,
        "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
        linux-block <linux-block@...r.kernel.org>
Subject: Re: discard feature, mkfs.ext4 and mmc default fallback to normal
 erase op

Hi Ted, Michael,

On Tue, 8 Dec 2020 at 03:41, Theodore Y. Ts'o <tytso@....edu> wrote:
>
> On Mon, Dec 07, 2020 at 09:39:32PM +0100, Michael Walle wrote:
> > > There are three different MMC commands which are defined:
> > >
> > > 1) DISCARD
> > > 2) ERASE
> > > 3) SECURE ERASE
> > >
> > > The first two are expected to be fast, since it only involves clearing
> > > some metadata fields in the Flash Translation Layer (FTL), so that the
> > > LBA's in the specified range are no longer mapped to a flash page.
> >
> > Mh, where is it specified that the erase command is fast? According
> > to the Physical Layer Simplified Specification Version 8.00:
> >
> >  The actual erase time may be quite long, and the host may issue CMD7
> >  to deselect thhe card or perform card disconnection, as described in
> >  the Block Write section, above.

Before I go into some more detail, of course I fully agree that
dealing with erase/discard from the eMMC/SD specifications (and other
types of devices) point of view isn't entirely easy. :-)

But I also think we can do better than currently, at least for eMMC/SD.

>
> I looked at the eMMC specification from JEDEC (JESD84-A44) and there,
> both the "erase" and "trim" are specified that the work is to be
> queued to be done at a time which is convenient to the controller
> (read: FTL).  This is in contrast to the "secure erase" and "secure
> trim" commands, where the erasing has to be done NOW NOW NOW for "high
> security applications".
>
> The only difference between "erase" and "trim" seems to be that erahse
> has to be done in units of the "erase groups" which is typically
> larger than the "write pages" which is the granularity required by the
> trim command.  There is also a comment that when you are erasing the
> entire partition, "erase" is preferred over "trim".  (Presumably
> because it is more convenient?  The spec is not clear.)
>
> Unfortunately, the SD Card spec and the eMMC spec both read like they
> were written by a standards committee stacked by hardware engineers.
> It doesn't look like they had file system engineers in the room,
> because the distinctions between "erase" and "trim" are pretty silly,
> and not well defined.  Aside from what I wrote, the spec is remarkably
> silent about what the host OS can depend upon.

Moreover, the specs have evolved over the years. Somehow, we need to
map a REQ_OP_DISCARD and REQ_OP_SECURE_ERASE to the best matching
operation that the currently inserted eMMC/SD card supports...

Long time time ago, both the SD and eMMC spec introduced support for
real discards commands, as being hints to the card without any
guarantees of what will happen to the data from a logical or a
physical point of view. If the card supports that, we should use it as
the first option for REQ_OP_DISCARD. Although, what should we pick as
the second best option, when the card doesn't support discard - that's
when it becomes more tricky. And the similar applies for
REQ_OP_SECURE_ERASE, or course.

If you have any suggestions for how we can improve in the above
decisions, feel free to suggest something.

Another issue that most likely is causing poor performance for
REQ_OP_DISCARD/REQ_OP_SECURE_ERASE for eMMC/SD, is that in
mmc_queue_setup_discard() we set up the maximum discard sectors
allowed per request and the discard granularity.

To find performance bottlenecks, I would start looking at what actual
eMMC/SD commands/args we end up mapping towards the
REQ_OP_DISCARD/REQ_OP_SECURE_ERASE requests. Then definitely, I would
also look at the values we end up picking as max discard sectors and
the discard granularity.

>
> From the fs perspective, what we care about is whether or not the
> command is a hint or a reliable way to zero a range of sectors.  A
> command could be a hint if the device is allowed to ignore it, or if
> the values of the sector are indeterminate, or if the sectors are
> zero'ed or not could change after a power cycle.  (I've seen an
> implementation where discard would result in the LBA's being read as
> zero --- but after a power cycle, reading from the same LBA would
> return the old data again.  This is standards complaint, but it's not
> terribly useful.)

:-)

>
> Assuming that the command is reliable, the next question is whether
> the erase operation is logical or physical --- which is to say, if an
> attacker has physical access to the die, with the ability to bypass
> the FTL and directly read the flash cells, could the attack retrieve
> the data, even if it required a distructive, physical attack on the
> hardware?  A logical erase would not require that the data be erased
> or otherwise made inaccessible against an attacker who bypasses the
> FTL; a physical erase would provide security guarantees that even if
> your phone has handed over to state-sponsored attacker, that nothing
> could be extracted after a physical erase.
>
> So if I were king, those would be the three levels of discard: "hint",
> "reliable logical", and "reliable physical", as those map to real use
> cases that are of actual use to a Host.  The challenge is mapping what
> we *actually* are given by different specs, which were written by
> hardware engineers and make distinctions that are not well defined so
> that multiple implementations can be "standard compliant", but have
> completely different performance profiles, thus making life easy for
> the marketing types, and hard for the file system engineers.  :-)

I agree, these are the three levels that make sense to support.

Honestly I haven't been paying enough attention to discussions for the
generic block layer around discards. However, considering what you
just stated above, we seem to be missing one request operation, don't
we?

[...]

Kind regards
Uffe

Powered by blists - more mailing lists