linux-ext4 - Re: [PATCH 1/2] HACK: ext3: mount fast even when recovering

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20090714143449.cae624c8.akpm@linux-foundation.org>
Date:	Tue, 14 Jul 2009 14:34:49 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Adrian Hunter <adrian.hunter@...ia.com>
Cc:	sct@...hat.com, adilger@....com, adrian.hunter@...ia.com,
	linux-ext4@...r.kernel.org, artem.bityutskiy@...ia.com
Subject: Re: [PATCH 1/2] HACK: ext3: mount fast even when recovering

On Tue, 14 Jul 2009 17:05:54 +0300
Adrian Hunter <adrian.hunter@...ia.com> wrote:

> Speed up ext3 recovery mount time by not sync'ing the
> block device.  Instead place all dirty buffers into the
> I/O queue and add a write barrier.  This ensures that
> no subsequent write will reach the disk before all the
> recovery writes, but that we do not have to wait for the
> I/O.
> 
> Note that ext3 reads sectors the correct way: through the
> buffer cache, so there is no risk of reading old metadata.

hm.  The change seems reasonable to me.  afaict it leaves no timing
windows during which another crash could muck things up.

As long as those write barriers actually work.  Do they?  For all
conceivable devices and IO schedulers?

It would be useful if you could quantify the benefits please - some
before-and-after timing results with both your funky hardware as well
as regular old disks would suit.

I'd suggest that if we're going to do this, we should aim to do it
unconditionally - no mount option needed.  We could leave the option
there for a while, for testing purposes (ie: we think the code might be
buggy).  But the new feature should perhaps default to "on", and we
plan to remove the mount option after a while.

Because there's no reason to retain the mount option in the long term.

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html