From: "Stephen C. Tweedie" <sct@redhat.com>
To: Alexander Viro <viro@math.psu.edu>
Cc: "Stephen C. Tweedie" <sct@redhat.com>,
Andreas Dilger <adilger@enel.ucalgary.ca>,
Andreas Dilger <adilger@turbolinux.com>,
linux-kernel@vger.kernel.org, "Theodore Y. Ts'o" <tytso@mit.edu>,
Ext2 development mailing list <ext2-devel@lists.sourceforge.net>
Subject: Re: [Ext2-devel] Re: [RFC] ext2_new_block() behaviour
Date: Thu, 4 Jan 2001 22:04:33 +0000 [thread overview]
Message-ID: <20010104220433.T1290@redhat.com> (raw)
In-Reply-To: <20010103121609.C1290@redhat.com> <Pine.GSO.4.21.0101031051080.15658-100000@weyl.math.psu.edu>
In-Reply-To: <Pine.GSO.4.21.0101031051080.15658-100000@weyl.math.psu.edu>; from viro@math.psu.edu on Wed, Jan 03, 2001 at 11:12:48AM -0500
Hi,
On Wed, Jan 03, 2001 at 11:12:48AM -0500, Alexander Viro wrote:
>
> On Wed, 3 Jan 2001, Stephen C. Tweedie wrote:
>
> > Having preallocated blocks allocated immediately is deliberate:
> > directories grow slowly and remain closed most of the time, so the
> > normal preallocation regime of only preallocating open files and
> > discarding preallocation on close just doesn't work.
>
> Erm. For directories we would not have the call of discard_prealloc()
> on close(2) - they have NULL ->release() anyway and for them it would
> happen only on ext2_put_inode(), i.e. upon the final dput(). Which would
> not happen while some descendent would stay in dcache.
>
> IOW, if directory is really going to grow (which normally mean that we
> are busily writing into files in it or its subdirectories) we will not
> get discard_prealloc() until it's all over.
The problem with directories is that they don't always grow rapidly
like that. Spool directories are perfect examples of directories
which grow sporadically over a long time, which is why we wanted
persistent preallocation.
The most common instance where that happens to regular files is the
case of log files, but those tend to be held open even while idle so
the preallocation persists anyway.
--Stephen
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
next prev parent reply other threads:[~2001-01-04 22:08 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2000-11-30 22:13 [CFT][RFC] ext2_new_inode() fixes and cleanup Alexander Viro
2000-11-30 23:18 ` Theodore Y. Ts'o
2000-12-01 12:55 ` Alexander Viro
2000-12-01 8:27 ` Andreas Dilger
2000-12-01 11:28 ` Alexander Viro
2001-01-02 23:33 ` [RFC] ext2_new_block() behaviour Alexander Viro
2001-01-03 1:47 ` Andreas Dilger
2001-01-03 3:37 ` Alexander Viro
2001-01-03 12:16 ` [Ext2-devel] " Stephen C. Tweedie
2001-01-03 16:12 ` Alexander Viro
2001-01-04 4:42 ` Oliver Xymoron
2001-01-04 5:06 ` Alexander Viro
2001-01-04 5:15 ` Oliver Xymoron
2001-01-04 22:04 ` Stephen C. Tweedie [this message]
2001-01-04 22:31 ` Alexander Viro
2001-01-04 20:48 ` -bird tree " Marcelo Tosatti
2001-01-04 23:25 ` Stephen C. Tweedie
2001-01-05 7:06 ` Andreas Dilger
2001-01-05 11:29 ` Stephen C. Tweedie
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=20010104220433.T1290@redhat.com \
--to=sct@redhat.com \
--cc=adilger@enel.ucalgary.ca \
--cc=adilger@turbolinux.com \
--cc=ext2-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=viro@math.psu.edu \
/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