mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mingming Cao <cmm@us.ibm.com>
To: Andrew Morton <akpm@osdl.org>
Cc: Valerie Henson <val_henson@linux.intel.com>,
	pbadari@gmail.com, linux-kernel@vger.kernel.org,
	Ext2-devel@lists.sourceforge.net, arjan@linux.intel.com,
	tytso@mit.edu, zach.brown@oracle.com
Subject: Re: [Ext2-devel] [RFC] [PATCH] Reducing average ext2 fsck time through fs-wide dirty bit]
Date: Fri, 24 Mar 2006 11:13:19 -0800	[thread overview]
Message-ID: <1143227599.4561.139.camel@localhost.localdomain> (raw)
In-Reply-To: <20060324104818.0016c2f2.akpm@osdl.org>

On Fri, 2006-03-24 at 10:48 -0800, Andrew Morton wrote:
> Valerie Henson <val_henson@linux.intel.com> wrote:
> >
> > On Wed, Mar 22, 2006 at 05:55:03PM -0800, Andrew Morton wrote:
> > > Valerie Henson <val_henson@linux.intel.com> wrote:
> > > > 
> > > > ext2 is simpler and faster than ext3 in many cases.  This is sort of
> > > > cheating; ext2 is simpler and faster because it makes no effort to
> > > > maintain on-disk consistency and can skip annoying things like, oh,
> > > > reserving space in the journal.  I am looking for ways to make ext2
> > > > cheat even more.
> > > > 
> > > 
> > > But it might be feasible to knock up an ext3-- in which all the journal
> > > operations are stubbed out.
> > 
> > Hmm... Could we get the mark_buffer_dirty/mark_inode_dirty logic
> > right?
> 
> All things are possible ;) One might add a new
> ext3_minus_minus_mark_buffer_dirty(), for example, put that in all the
> right places.
> 
> >  Probably create a list in the stubbed journal functions and
> > then mark them dirty in the journal close?  However, half the reason
> > I'm working on ext2 is the simplicity of the code - stubbing it out
> > would solve the performance problem but not the complexity problem.
> 
> Well ext3-- won't do anything to simplify the ext3 codebase.  It was just a
> thought..
> 
> > Note that ext3's habit of clearing indirect blocks on truncate would
> > break some things I want to do in the future. (Insert secret plans
> > here.)
> 
> Ah.  I guess one would need to port the ext2 truncate code.
> 
> 

There are reasons for zeroing indirect blocks on truncate: 

      * There are limits to the size of a single journal transaction
        (1/4 of the journal size). When truncating a large fragmented
        file, it may require modifying so many block bitmaps and group
        descriptors that it forces a journal transaction to close out,
        stalling the unlink operation.
      * Because of this per-transaction limit, truncate needs to zero
        the [dt]indirect blocks starting from the end of the file, in
        case it needs to start a new transaction in the middle of the
        truncate (ext3 guarantees that a partially-completed truncate
        will be consistent/completed after a crash).
      * The read/write of the file's [dt]indirect blocks from the end of
        the file to the beginning can take a lot of time, as it does
        this in single-block chunks and the blocks are not contiguous.
> -------------------------------------------------------
> This SF.Net email is sponsored by xPML, a groundbreaking scripting language
> that extends applications into web and mobile media. Attend the live webcast
> and join the prime developer group breaking into this new coding territory!
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
> _______________________________________________
> Ext2-devel mailing list
> Ext2-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ext2-devel


  reply	other threads:[~2006-03-24 19:13 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-22  1:10 Valerie Henson
2006-03-22  8:40 ` Valerie Henson
2006-03-22 13:08 ` Alan Cox
2006-03-22 18:18   ` [Ext2-devel] " Mingming Cao
2006-03-22 18:16 ` [Ext2-devel] " Mingming Cao
     [not found]   ` <200603230011.53793.ioe-lkml@rameria.de>
2006-03-22 23:52     ` Mingming Cao
2006-03-22 19:09 ` Badari Pulavarty
2006-03-22 22:48   ` Valerie Henson
2006-03-23  1:55     ` Andrew Morton
2006-03-24 14:32       ` Valerie Henson
2006-03-24 15:35         ` Dave Kleikamp
2006-03-24 18:48         ` Andrew Morton
2006-03-24 19:13           ` Mingming Cao [this message]
2006-03-24 19:31             ` Andreas Dilger
2006-03-24 18:52         ` Theodore Ts'o
2006-03-24 19:14           ` Mingming Cao
2006-03-24 19:28         ` Andreas Dilger
2006-03-24 20:01           ` Theodore Ts'o
2006-03-24 21:00             ` Andreas Dilger
2006-03-24 21:39               ` Theodore Ts'o
2006-03-24 22:16                 ` Andreas Dilger
2006-03-25  5:13                   ` Suparna Bhattacharya
2006-03-25 17:38                   ` Ben Pfaff
2006-03-24 20:52           ` [Ext2-devel] " Matthew Wilcox
2006-03-24 21:23             ` Andreas Dilger

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=1143227599.4561.139.camel@localhost.localdomain \
    --to=cmm@us.ibm.com \
    --cc=Ext2-devel@lists.sourceforge.net \
    --cc=akpm@osdl.org \
    --cc=arjan@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pbadari@gmail.com \
    --cc=tytso@mit.edu \
    --cc=val_henson@linux.intel.com \
    --cc=zach.brown@oracle.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

Powered by JetHome