From: Theodore Tso <tytso@mit.edu>
To: Andreas Dilger <adilger@sun.com>
Cc: Ric Wheeler <rwheeler@redhat.com>,
nicholas.dokos@hp.com, linux-fsdevel@vger.kernel.org,
Christoph Hellwig <hch@infradead.org>,
Douglas Shakshober <dshaks@redhat.com>,
Joshua Giles <jgiles@redhat.com>,
Valerie Aurora <vaurora@redhat.com>,
Eric Sandeen <esandeen@redhat.com>,
Steven Whitehouse <swhiteho@redhat.com>,
Edward Shishkin <edward@redhat.com>,
Josef Bacik <jbacik@redhat.com>, Jeff Moyer <jmoyer@redhat.com>,
Chris Mason <chris.mason@oracle.com>,
"Whitney, Eric" <eric.whitney@hp.com>
Subject: Re: large fs testing
Date: Tue, 26 May 2009 17:39:14 -0400 [thread overview]
Message-ID: <20090526213913.GH6986@mit.edu> (raw)
In-Reply-To: <20090526212132.GE3218@webber.adilger.int>
On Tue, May 26, 2009 at 03:21:32PM -0600, Andreas Dilger wrote:
> On May 26, 2009 13:47 -0400, Ric Wheeler wrote:
> > These runs were without lazy init, so I would expect to be a little more
> > than twice as slow as your second run (not the three times I saw)
> > assuming that it scales linearly.
>
> Making lazy_itable_init the default formatting option for ext4 is/was
> dependent upon the kernel doing the zeroing of the inode table blocks
> at first mount time. I'm not sure if that was implemented yet.
No, it hasn't been implemented yet. If someone would like to step
forward, it's not hard patch to write. The good news is that there's
no need to actually use the journal for most of the zero'ing;
basically, the design would be for each block group, if the inode
table hasn't been initialized, to call write_down on the per-block
group alloc_sem semphaore in ext4_group_info, initialize the inode
table in the unused portion of the block group (which can be calucated
from ext4_itable_unused_count()), then release the alloc_sem
semaphore, and then (under journal protection) set the
EXT4_BG_INODE_ZEROED flag in the block group descriptor.
If someone hurries, we could get this done before the next merge
window opens (probably in a week or two).
- Ted
next prev parent reply other threads:[~2009-05-26 21:39 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-23 13:53 large fs testing Ric Wheeler
2009-05-26 12:21 ` Joshua Giles
2009-05-26 12:28 ` Ric Wheeler
2009-05-26 17:39 ` Nick Dokos
2009-05-26 17:47 ` Ric Wheeler
2009-05-26 21:21 ` Andreas Dilger
2009-05-26 21:39 ` Theodore Tso [this message]
2009-05-26 22:17 ` Ric Wheeler
2009-05-28 6:30 ` Andreas Dilger
2009-05-28 10:52 ` Ric Wheeler
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090526213913.GH6986@mit.edu \
--to=tytso@mit.edu \
--cc=adilger@sun.com \
--cc=chris.mason@oracle.com \
--cc=dshaks@redhat.com \
--cc=edward@redhat.com \
--cc=eric.whitney@hp.com \
--cc=esandeen@redhat.com \
--cc=hch@infradead.org \
--cc=jbacik@redhat.com \
--cc=jgiles@redhat.com \
--cc=jmoyer@redhat.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=nicholas.dokos@hp.com \
--cc=rwheeler@redhat.com \
--cc=swhiteho@redhat.com \
--cc=vaurora@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).